{"action":"created","issue":{"url":"https://api.github.com/repos/palantir/atlasdb/issues/2374","repository_url":"https://api.github.com/repos/palantir/atlasdb","labels_url":"https://api.github.com/repos/palantir/atlasdb/issues/2374/labels{/name}","comments_url":"https://api.github.com/repos/palantir/atlasdb/issues/2374/comments","events_url":"https://api.github.com/repos/palantir/atlasdb/issues/2374/events","html_url":"https://github.com/palantir/atlasdb/pull/2374","id":259234316,"number":2374,"title":"[Startup Independence]Part 6: Enable initialization with a quorum of nodes","user":{"login":"gmaretic","id":23168027,"avatar_url":"https://avatars0.githubusercontent.com/u/23168027?v=4","gravatar_id":"","url":"https://api.github.com/users/gmaretic","html_url":"https://github.com/gmaretic","followers_url":"https://api.github.com/users/gmaretic/followers","following_url":"https://api.github.com/users/gmaretic/following{/other_user}","gists_url":"https://api.github.com/users/gmaretic/gists{/gist_id}","starred_url":"https://api.github.com/users/gmaretic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gmaretic/subscriptions","organizations_url":"https://api.github.com/users/gmaretic/orgs","repos_url":"https://api.github.com/users/gmaretic/repos","events_url":"https://api.github.com/users/gmaretic/events{/privacy}","received_events_url":"https://api.github.com/users/gmaretic/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-09-20T17:00:01Z","updated_at":"2017-09-21T09:58:04Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/palantir/atlasdb/pulls/2374","html_url":"https://github.com/palantir/atlasdb/pull/2374","diff_url":"https://github.com/palantir/atlasdb/pull/2374.diff","patch_url":"https://github.com/palantir/atlasdb/pull/2374.patch"},"body":"**Goals (and why)**:\r\nEnable initialization with a quorum of C* nodes\r\n\r\n**Implementation Description (bullets)**:\r\nWe allowed tables (and keyspaces) to be created with a quorum of nodes\r\n\r\n**Concerns (what feedback would you like?)**:\r\nThis should be safe, but @ashrayjain to double-check?\r\n\r\n**Where should we start reviewing?**:\r\nSmall\r\n\r\n**Priority (whenever / two weeks / yesterday)**:\r\nAsap\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/palantir/atlasdb/2374)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/palantir/atlasdb/issues/comments/331110565","html_url":"https://github.com/palantir/atlasdb/pull/2374#issuecomment-331110565","issue_url":"https://api.github.com/repos/palantir/atlasdb/issues/2374","id":331110565,"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":"2017-09-21T09:58:04Z","updated_at":"2017-09-21T09:58:04Z","author_association":"NONE","body":"# [Codecov](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=h1) Report\n> :exclamation: No coverage uploaded for pull request base (`feature/startup-ordering@9ab8c6a`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit).\n> The diff coverage is `49.19%`.\n\n[![Impacted file tree graph](https://codecov.io/gh/palantir/atlasdb/pull/2374/graphs/tree.svg?token=41qO7xCQsU&width=650&src=pr&height=150)](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree)\n\n```diff\n@@                     Coverage Diff                     @@\n##             feature/startup-ordering    #2374   +/-   ##\n===========================================================\n  Coverage                            ?   59.13%           \n  Complexity                          ?     4319           \n===========================================================\n  Files                               ?      824           \n  Lines                               ?    38613           \n  Branches                            ?     4002           \n===========================================================\n  Hits                                ?    22832           \n  Misses                              ?    14336           \n  Partials                            ?     1445\n```\n\n\n| [Impacted Files](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree) | Coverage Δ | Complexity Δ | |\n|---|---|---|---|\n| [...yvalue/cassandra/CassandraTimestampBoundStore.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1jYXNzYW5kcmEvc3JjL21haW4vamF2YS9jb20vcGFsYW50aXIvYXRsYXNkYi9rZXl2YWx1ZS9jYXNzYW5kcmEvQ2Fzc2FuZHJhVGltZXN0YW1wQm91bmRTdG9yZS5qYXZh) | `0% <0%> (ø)` | `0 <0> (?)` | |\n| [...b/transaction/impl/AbstractTransactionManager.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1jbGllbnQvc3JjL21haW4vamF2YS9jb20vcGFsYW50aXIvYXRsYXNkYi90cmFuc2FjdGlvbi9pbXBsL0Fic3RyYWN0VHJhbnNhY3Rpb25NYW5hZ2VyLmphdmE=) | `60.78% <0%> (ø)` | `9 <0> (?)` | |\n| [...yvalue/cassandra/CassandraKeyValueServiceImpl.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1jYXNzYW5kcmEvc3JjL21haW4vamF2YS9jb20vcGFsYW50aXIvYXRsYXNkYi9rZXl2YWx1ZS9jYXNzYW5kcmEvQ2Fzc2FuZHJhS2V5VmFsdWVTZXJ2aWNlSW1wbC5qYXZh) | `0.29% <10.52%> (ø)` | `0 <0> (?)` | |\n| [...tlasdb/factory/TransactionManagersInitializer.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1jb25maWcvc3JjL21haW4vamF2YS9jb20vcGFsYW50aXIvYXRsYXNkYi9mYWN0b3J5L1RyYW5zYWN0aW9uTWFuYWdlcnNJbml0aWFsaXplci5qYXZh) | `100% <100%> (ø)` | `0 <0> (?)` | |\n| [...db/keyvalue/cassandra/CassandraClientPoolImpl.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1jYXNzYW5kcmEvc3JjL21haW4vamF2YS9jb20vcGFsYW50aXIvYXRsYXNkYi9rZXl2YWx1ZS9jYXNzYW5kcmEvQ2Fzc2FuZHJhQ2xpZW50UG9vbEltcGwuamF2YQ==) | `42.55% <50%> (ø)` | `0 <0> (?)` | |\n| [...m/palantir/async/initializer/AsyncInitializer.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1jb21tb25zL3NyYy9tYWluL2phdmEvY29tL3BhbGFudGlyL2FzeW5jL2luaXRpYWxpemVyL0FzeW5jSW5pdGlhbGl6ZXIuamF2YQ==) | `61.22% <59.45%> (ø)` | `6 <6> (?)` | |\n| [...palantir/atlasdb/persistentlock/LockStoreImpl.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1pbXBsLXNoYXJlZC9zcmMvbWFpbi9qYXZhL2NvbS9wYWxhbnRpci9hdGxhc2RiL3BlcnNpc3RlbnRsb2NrL0xvY2tTdG9yZUltcGwuamF2YQ==) | `93.33% <62.5%> (ø)` | `12 <1> (?)` | |\n| [...lasdb/keyvalue/impl/StaticTableMappingService.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1pbXBsLXNoYXJlZC9zcmMvbWFpbi9qYXZhL2NvbS9wYWxhbnRpci9hdGxhc2RiL2tleXZhbHVlL2ltcGwvU3RhdGljVGFibGVNYXBwaW5nU2VydmljZS5qYXZh) | `87.5% <62.5%> (ø)` | `7 <1> (?)` | |\n| [...ntir/timestamp/PersistentTimestampServiceImpl.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-dGltZXN0YW1wLWltcGwvc3JjL21haW4vamF2YS9jb20vcGFsYW50aXIvdGltZXN0YW1wL1BlcnNpc3RlbnRUaW1lc3RhbXBTZXJ2aWNlSW1wbC5qYXZh) | `85% <66.66%> (ø)` | `0 <0> (?)` | |\n| [.../atlasdb/cleaner/KeyValueServiceScrubberStore.java](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree#diff-YXRsYXNkYi1pbXBsLXNoYXJlZC9zcmMvbWFpbi9qYXZhL2NvbS9wYWxhbnRpci9hdGxhc2RiL2NsZWFuZXIvS2V5VmFsdWVTZXJ2aWNlU2NydWJiZXJTdG9yZS5qYXZh) | `89.1% <66.66%> (ø)` | `20 <1> (?)` | |\n| ... and [1 more](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=tree-more) | |\n\n------\n\n[Continue to review full report at Codecov](https://codecov.io/gh/palantir/atlasdb/pull/2374?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/palantir/atlasdb/pull/2374?src=pr&el=footer). Last update [9ab8c6a...e8365e7](https://codecov.io/gh/palantir/atlasdb/pull/2374?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).\n"}}
{"actor":{"display_login":"codecov-io"}}