{"action":"created","issue":{"url":"https://api.github.com/repos/linkedin/pinot/issues/2787","repository_url":"https://api.github.com/repos/linkedin/pinot","labels_url":"https://api.github.com/repos/linkedin/pinot/issues/2787/labels{/name}","comments_url":"https://api.github.com/repos/linkedin/pinot/issues/2787/comments","events_url":"https://api.github.com/repos/linkedin/pinot/issues/2787/events","html_url":"https://github.com/linkedin/pinot/pull/2787","id":321289792,"number":2787,"title":"Purge task optimization","user":{"login":"jenniferdai","id":5035129,"avatar_url":"https://avatars2.githubusercontent.com/u/5035129?v=4","gravatar_id":"","url":"https://api.github.com/users/jenniferdai","html_url":"https://github.com/jenniferdai","followers_url":"https://api.github.com/users/jenniferdai/followers","following_url":"https://api.github.com/users/jenniferdai/following{/other_user}","gists_url":"https://api.github.com/users/jenniferdai/gists{/gist_id}","starred_url":"https://api.github.com/users/jenniferdai/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jenniferdai/subscriptions","organizations_url":"https://api.github.com/users/jenniferdai/orgs","repos_url":"https://api.github.com/users/jenniferdai/repos","events_url":"https://api.github.com/users/jenniferdai/events{/privacy}","received_events_url":"https://api.github.com/users/jenniferdai/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-05-08T18:08:24Z","updated_at":"2018-05-08T18:35:46Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/linkedin/pinot/pulls/2787","html_url":"https://github.com/linkedin/pinot/pull/2787","diff_url":"https://github.com/linkedin/pinot/pull/2787.diff","patch_url":"https://github.com/linkedin/pinot/pull/2787.patch"},"body":"* Don't purge segment unless there are records to be purged or modified"},"comment":{"url":"https://api.github.com/repos/linkedin/pinot/issues/comments/387500144","html_url":"https://github.com/linkedin/pinot/pull/2787#issuecomment-387500144","issue_url":"https://api.github.com/repos/linkedin/pinot/issues/2787","id":387500144,"user":{"login":"codecov-io","id":8655789,"avatar_url":"https://avatars0.githubusercontent.com/u/8655789?v=4","gravatar_id":"","url":"https://api.github.com/users/codecov-io","html_url":"https://github.com/codecov-io","followers_url":"https://api.github.com/users/codecov-io/followers","following_url":"https://api.github.com/users/codecov-io/following{/other_user}","gists_url":"https://api.github.com/users/codecov-io/gists{/gist_id}","starred_url":"https://api.github.com/users/codecov-io/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codecov-io/subscriptions","organizations_url":"https://api.github.com/users/codecov-io/orgs","repos_url":"https://api.github.com/users/codecov-io/repos","events_url":"https://api.github.com/users/codecov-io/events{/privacy}","received_events_url":"https://api.github.com/users/codecov-io/received_events","type":"User","site_admin":false},"created_at":"2018-05-08T18:35:46Z","updated_at":"2018-05-08T18:35:46Z","author_association":"NONE","body":"# [Codecov](https://codecov.io/gh/linkedin/pinot/pull/2787?src=pr&el=h1) Report\n> Merging [#2787](https://codecov.io/gh/linkedin/pinot/pull/2787?src=pr&el=desc) into [master](https://codecov.io/gh/linkedin/pinot/commit/754bcbbf6932663ac98f0c3e7faf8990d6e9f4f3?src=pr&el=desc) will **increase** coverage by `0.08%`.\n> The diff coverage is `68.75%`.\n\n[![Impacted file tree graph](https://codecov.io/gh/linkedin/pinot/pull/2787/graphs/tree.svg?height=150&token=4ibza2ugkz&src=pr&width=650)](https://codecov.io/gh/linkedin/pinot/pull/2787?src=pr&el=tree)\n\n```diff\n@@            Coverage Diff            @@\n##           master   #2787      +/-   ##\n=========================================\n+ Coverage   69.12%   69.2%   +0.08%     \n=========================================\n  Files         905     905              \n  Lines       43041   43055      +14     \n  Branches     5835    5837       +2     \n=========================================\n+ Hits        29751   29797      +46     \n+ Misses      11314   11293      -21     \n+ Partials     1976    1965      -11\n```\n\n\n| [Impacted Files](https://codecov.io/gh/linkedin/pinot/pull/2787?src=pr&el=tree) | Coverage Δ | |\n|---|---|---|\n| [...kedin/pinot/minion/executor/PurgeTaskExecutor.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtbWluaW9uL3NyYy9tYWluL2phdmEvY29tL2xpbmtlZGluL3Bpbm90L21pbmlvbi9leGVjdXRvci9QdXJnZVRhc2tFeGVjdXRvci5qYXZh) | `76% <66.66%> (-2.27%)` | :arrow_down: |\n| [.../com/linkedin/pinot/core/minion/SegmentPurger.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL2NvbS9saW5rZWRpbi9waW5vdC9jb3JlL21pbmlvbi9TZWdtZW50UHVyZ2VyLmphdmE=) | `78.82% <68.96%> (-3.37%)` | :arrow_down: |\n| [...ore/query/scheduler/resources/ResourceManager.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL2NvbS9saW5rZWRpbi9waW5vdC9jb3JlL3F1ZXJ5L3NjaGVkdWxlci9yZXNvdXJjZXMvUmVzb3VyY2VNYW5hZ2VyLmphdmE=) | `87.09% <0%> (-9.68%)` | :arrow_down: |\n| [...e/operator/dociditerators/BitmapDocIdIterator.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL2NvbS9saW5rZWRpbi9waW5vdC9jb3JlL29wZXJhdG9yL2RvY2lkaXRlcmF0b3JzL0JpdG1hcERvY0lkSXRlcmF0b3IuamF2YQ==) | `60.71% <0%> (-3.58%)` | :arrow_down: |\n| [...not/broker/broker/helix/ClusterChangeMediator.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtYnJva2VyL3NyYy9tYWluL2phdmEvY29tL2xpbmtlZGluL3Bpbm90L2Jyb2tlci9icm9rZXIvaGVsaXgvQ2x1c3RlckNoYW5nZU1lZGlhdG9yLmphdmE=) | `62.71% <0%> (-3.39%)` | :arrow_down: |\n| [...e/operator/dociditerators/MVScanDocIdIterator.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL2NvbS9saW5rZWRpbi9waW5vdC9jb3JlL29wZXJhdG9yL2RvY2lkaXRlcmF0b3JzL01WU2NhbkRvY0lkSXRlcmF0b3IuamF2YQ==) | `63.63% <0%> (-3.04%)` | :arrow_down: |\n| [.../pinot/core/query/scheduler/PriorityScheduler.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL2NvbS9saW5rZWRpbi9waW5vdC9jb3JlL3F1ZXJ5L3NjaGVkdWxlci9Qcmlvcml0eVNjaGVkdWxlci5qYXZh) | `78.08% <0%> (-2.74%)` | :arrow_down: |\n| [...lix/core/realtime/PinotRealtimeSegmentManager.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtY29udHJvbGxlci9zcmMvbWFpbi9qYXZhL2NvbS9saW5rZWRpbi9waW5vdC9jb250cm9sbGVyL2hlbGl4L2NvcmUvcmVhbHRpbWUvUGlub3RSZWFsdGltZVNlZ21lbnRNYW5hZ2VyLmphdmE=) | `73.54% <0%> (-1.06%)` | :arrow_down: |\n| [.../controller/helix/core/SegmentDeletionManager.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtY29udHJvbGxlci9zcmMvbWFpbi9qYXZhL2NvbS9saW5rZWRpbi9waW5vdC9jb250cm9sbGVyL2hlbGl4L2NvcmUvU2VnbWVudERlbGV0aW9uTWFuYWdlci5qYXZh) | `76.72% <0%> (-0.87%)` | :arrow_down: |\n| [...ot/common/protocols/SegmentCompletionProtocol.java](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree#diff-cGlub3QtY29tbW9uL3NyYy9tYWluL2phdmEvY29tL2xpbmtlZGluL3Bpbm90L2NvbW1vbi9wcm90b2NvbHMvU2VnbWVudENvbXBsZXRpb25Qcm90b2NvbC5qYXZh) | `92.57% <0%> (-0.58%)` | :arrow_down: |\n| ... and [19 more](https://codecov.io/gh/linkedin/pinot/pull/2787/diff?src=pr&el=tree-more) | |\n\n------\n\n[Continue to review full report at Codecov](https://codecov.io/gh/linkedin/pinot/pull/2787?src=pr&el=continue).\n> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)\n> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`\n> Powered by [Codecov](https://codecov.io/gh/linkedin/pinot/pull/2787?src=pr&el=footer). Last update [754bcbb...8266354](https://codecov.io/gh/linkedin/pinot/pull/2787?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).\n"}}
{"actor":{"display_login":"codecov-io"}}