{"action":"created","issue":{"url":"https://api.github.com/repos/druid-io/druid/issues/1576","repository_url":"https://api.github.com/repos/druid-io/druid","labels_url":"https://api.github.com/repos/druid-io/druid/issues/1576/labels{/name}","comments_url":"https://api.github.com/repos/druid-io/druid/issues/1576/comments","events_url":"https://api.github.com/repos/druid-io/druid/issues/1576/events","html_url":"https://github.com/druid-io/druid/pull/1576","id":97980003,"number":1576,"title":"Query time lookup cluster wide config","user":{"login":"drcrallen","id":8213081,"avatar_url":"https://avatars.githubusercontent.com/u/8213081?v=3","gravatar_id":"","url":"https://api.github.com/users/drcrallen","html_url":"https://github.com/drcrallen","followers_url":"https://api.github.com/users/drcrallen/followers","following_url":"https://api.github.com/users/drcrallen/following{/other_user}","gists_url":"https://api.github.com/users/drcrallen/gists{/gist_id}","starred_url":"https://api.github.com/users/drcrallen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drcrallen/subscriptions","organizations_url":"https://api.github.com/users/drcrallen/orgs","repos_url":"https://api.github.com/users/drcrallen/repos","events_url":"https://api.github.com/users/drcrallen/events{/privacy}","received_events_url":"https://api.github.com/users/drcrallen/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/druid-io/druid/labels/Discuss","name":"Discuss","color":"ff00ff"},{"url":"https://api.github.com/repos/druid-io/druid/labels/Feature","name":"Feature","color":"2ceefb"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/druid-io/druid/milestones/18","html_url":"https://github.com/druid-io/druid/milestones/0.9.1","labels_url":"https://api.github.com/repos/druid-io/druid/milestones/18/labels","id":1526819,"number":18,"title":"0.9.1","description":"","creator":{"login":"fjy","id":428325,"avatar_url":"https://avatars.githubusercontent.com/u/428325?v=3","gravatar_id":"","url":"https://api.github.com/users/fjy","html_url":"https://github.com/fjy","followers_url":"https://api.github.com/users/fjy/followers","following_url":"https://api.github.com/users/fjy/following{/other_user}","gists_url":"https://api.github.com/users/fjy/gists{/gist_id}","starred_url":"https://api.github.com/users/fjy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fjy/subscriptions","organizations_url":"https://api.github.com/users/fjy/orgs","repos_url":"https://api.github.com/users/fjy/repos","events_url":"https://api.github.com/users/fjy/events{/privacy}","received_events_url":"https://api.github.com/users/fjy/received_events","type":"User","site_admin":false},"open_issues":30,"closed_issues":10,"state":"open","created_at":"2016-01-20T19:04:22Z","updated_at":"2016-02-17T21:03:01Z","due_on":null,"closed_at":null},"comments":11,"created_at":"2015-07-29T16:52:41Z","updated_at":"2016-02-17T22:07:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/druid-io/druid/pulls/1576","html_url":"https://github.com/druid-io/druid/pull/1576","diff_url":"https://github.com/druid-io/druid/pull/1576.diff","patch_url":"https://github.com/druid-io/druid/pull/1576.patch"},"body":"This is the next PR for QTL where cluster wide configuration is possible.\r\n\r\nThis approach uses POSTs from the coordinator to populate QTL information in all interested nodes. Interested nodes \"register\" via the Announcer (currently ZK based). But all communication occurs through HTTP directly between the coordinator and the node.\r\n\r\nThe PR lists over 4k lines, but a LOT of that is tests.\r\n\r\nTODO:\r\n\r\n* ~~Proposal for coordinator and node endpoints~~\r\n* ~~Squash~~\r\n* ~~Docs~~ Punted until extensions are updated\r\n* ~~Migrate coordinator code to a module.~~\r\n* ~~Figure out a better way to handle the listening handling. (see https://github.com/druid-io/druid/pull/2286 )~~ added in this PR in core\r\n* ~~Migrate lookup config to auditable config type.~~\r\n* ~~Rebase on https://github.com/druid-io/druid/pull/2291~~\r\n* ~~Move core stuff and extension stuff to its own commit~~\r\n* Make sure put functions as expected. (I don't  think deleting actually works in this version)\r\n* Make sure initialization behaves as expected\r\n* Actually run the thing locally\r\n* Reconcile need for static config vs dynamic config. Ex: what happens if someone requests a delete or get of a lookup defined in runtime.properties?"},"comment":{"url":"https://api.github.com/repos/druid-io/druid/issues/comments/185429031","html_url":"https://github.com/druid-io/druid/pull/1576#issuecomment-185429031","issue_url":"https://api.github.com/repos/druid-io/druid/issues/1576","id":185429031,"user":{"login":"b-slim","id":7376396,"avatar_url":"https://avatars.githubusercontent.com/u/7376396?v=3","gravatar_id":"","url":"https://api.github.com/users/b-slim","html_url":"https://github.com/b-slim","followers_url":"https://api.github.com/users/b-slim/followers","following_url":"https://api.github.com/users/b-slim/following{/other_user}","gists_url":"https://api.github.com/users/b-slim/gists{/gist_id}","starred_url":"https://api.github.com/users/b-slim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/b-slim/subscriptions","organizations_url":"https://api.github.com/users/b-slim/orgs","repos_url":"https://api.github.com/users/b-slim/repos","events_url":"https://api.github.com/users/b-slim/events{/privacy}","received_events_url":"https://api.github.com/users/b-slim/received_events","type":"User","site_admin":false},"created_at":"2016-02-17T22:07:01Z","updated_at":"2016-02-17T22:07:01Z","body":"@drcrallen First i think we can not fix the problem of Authorization, it is beyond this work. If in your use case you want to hide a pass/login, you can always use a password provider.\r\nIf you really feel strong about having a static config. we can have a mode where  we allow delete during runtime and if the process restart, the config will be added again.    "}}