{"action":"created","issue":{"url":"https://api.github.com/repos/druid-io/druid/issues/1639","labels_url":"https://api.github.com/repos/druid-io/druid/issues/1639/labels{/name}","comments_url":"https://api.github.com/repos/druid-io/druid/issues/1639/comments","events_url":"https://api.github.com/repos/druid-io/druid/issues/1639/events","html_url":"https://github.com/druid-io/druid/pull/1639","id":101709878,"number":1639,"title":"New plumber","user":{"login":"gianm","id":1214075,"avatar_url":"https://avatars.githubusercontent.com/u/1214075?v=3","gravatar_id":"","url":"https://api.github.com/users/gianm","html_url":"https://github.com/gianm","followers_url":"https://api.github.com/users/gianm/followers","following_url":"https://api.github.com/users/gianm/following{/other_user}","gists_url":"https://api.github.com/users/gianm/gists{/gist_id}","starred_url":"https://api.github.com/users/gianm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gianm/subscriptions","organizations_url":"https://api.github.com/users/gianm/orgs","repos_url":"https://api.github.com/users/gianm/repos","events_url":"https://api.github.com/users/gianm/events{/privacy}","received_events_url":"https://api.github.com/users/gianm/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/druid-io/druid/labels/Discuss","name":"Discuss","color":"ff00ff"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-08-18T17:40:37Z","updated_at":"2015-08-28T15:45:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/druid-io/druid/pulls/1639","html_url":"https://github.com/druid-io/druid/pull/1639","diff_url":"https://github.com/druid-io/druid/pull/1639.diff","patch_url":"https://github.com/druid-io/druid/pull/1639.patch"},"body":"Modify Plumbers in these ways,\r\n\r\n1) Persist using Committer instead of Runnable. (Although the metadata object\r\n   is ignored in this patch)\r\n\r\n2) Remove the getSink method.\r\n\r\n3) Plumbers are now responsible for time-based and hydrant-full-based periodic\r\n   committing. (FireChief, RealtimeIndexTask, and IndexTask used to do this)\r\n\r\nThe purpose of these changes is to make it possible to implement Plumbers using\r\nAppenderators, which is a hopefully more useful interface for things like transactional\r\ningestion and windowPeriodless ingestion. Thinking of something like this for\r\nthe interface:\r\n\r\n  https://gist.github.com/gianm/7d8f83619aada126fcd9"},"comment":{"url":"https://api.github.com/repos/druid-io/druid/issues/comments/135811018","html_url":"https://github.com/druid-io/druid/pull/1639#issuecomment-135811018","issue_url":"https://api.github.com/repos/druid-io/druid/issues/1639","id":135811018,"user":{"login":"gianm","id":1214075,"avatar_url":"https://avatars.githubusercontent.com/u/1214075?v=3","gravatar_id":"","url":"https://api.github.com/users/gianm","html_url":"https://github.com/gianm","followers_url":"https://api.github.com/users/gianm/followers","following_url":"https://api.github.com/users/gianm/following{/other_user}","gists_url":"https://api.github.com/users/gianm/gists{/gist_id}","starred_url":"https://api.github.com/users/gianm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gianm/subscriptions","organizations_url":"https://api.github.com/users/gianm/orgs","repos_url":"https://api.github.com/users/gianm/repos","events_url":"https://api.github.com/users/gianm/events{/privacy}","received_events_url":"https://api.github.com/users/gianm/received_events","type":"User","site_admin":false},"created_at":"2015-08-28T15:45:51Z","updated_at":"2015-08-28T15:45:51Z","body":"pushed a couple changes,\r\n\r\n1) Rebased on master (post #1609)\r\n2) Split the Indexer interface removal into a separate PR (#1683)"}}