[kudu-CR] KUDU-734. Add test coverage for encodings and strings

2017-01-04 Thread Jun He (Code Review)
Jun He has posted comments on this change. Change subject: KUDU-734. Add test coverage for encodings and strings .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5588/1//COMMIT_MSG Commit Message: Line 9:

[kudu-CR] KUDU-90. Add a header checksum to our RPC protocol

2017-01-04 Thread Jun He (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5578 to look at the new patch set (#4). Change subject: KUDU-90. Add a header checksum to our RPC protocol .. KUDU-90.

[kudu-CR] [KUDU-90] Add a header checksum to our RPC protocol Added 4-byte checksum into the RPC protocol. Checksum is inserted immediately after total size.

2017-01-04 Thread Jun He (Code Review)
Jun He has posted comments on this change. Change subject: [KUDU-90] Add a header checksum to our RPC protocol Added 4-byte checksum into the RPC protocol. Checksum is inserted immediately after total_size. .. Patch Set 3:

[kudu-CR] KUDU-1733. Update consistency semantics doc

2017-01-04 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, Adar Dembo, John Russell, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5605 to review the following change. Change subject: KUDU-1733. Update consistency semantics doc

[kudu-CR] Initial draft of release notes and doc updates for 1.2

2017-01-04 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, Adar Dembo, John Russell, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5604 to look at the new patch set (#2). Change subject: Initial draft of release notes and doc updates for 1.2

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

2017-01-04 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5603 to look at the new patch set (#3). Change subject: thirdparty: Link curl against el6 workaround openssl ..

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

2017-01-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: thirdparty: Link curl against el6 workaround openssl .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5603/2/thirdparty/build-definitions.sh File

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

2017-01-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: Link curl against el6 workaround openssl .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5603/2/thirdparty/build-definitions.sh File

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

2017-01-04 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5603 to look at the new patch set (#2). Change subject: thirdparty: Link curl against el6 workaround openssl ..

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

2017-01-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: thirdparty: Link curl against el6 workaround openssl .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5603/1//COMMIT_MSG Commit Message: Line 7:

[kudu-CR] Restructure release notes in preparation for 1.2 release

2017-01-04 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, Adar Dembo, John Russell, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5602 to look at the new patch set (#2). Change subject: Restructure release notes in preparation for 1.2 release

[kudu-CR] WIP: release notes for 1.2

2017-01-04 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, Adar Dembo, John Russell, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5604 to review the following change. Change subject: WIP: release notes for 1.2

[kudu-CR] WIP [jepsen.kudu] run tests from clojure-maven-plugin

2017-01-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: WIP [jepsen.kudu] run tests from clojure-maven-plugin .. Abandoned Merged into a single patch: https://gerrit.cloudera.org/#/c/5500 -- To view, visit

[kudu-CR] [jepsen.kudu] run tests from clojure-maven-plugin

2017-01-04 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5500 to look at the new patch set (#7). Change subject: [jepsen.kudu] run tests from clojure-maven-plugin ..

[kudu-CR] Restructure release notes in preparation for 1.2 release

2017-01-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Restructure release notes in preparation for 1.2 release .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5602/1/docs/known_issues.adoc File

[kudu-CR] Restructure release notes in preparation for 1.2 release

2017-01-04 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: Restructure release notes in preparation for 1.2 release .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/5602/1/docs/known_issues.adoc File

[kudu-CR] Restructure release notes in preparation for 1.2 release

2017-01-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Restructure release notes in preparation for 1.2 release .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5602/1/docs/prior_release_notes.adoc

[kudu-CR] WIP [jepsen.kudu] run tests from clojure-maven-plugin

2017-01-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: WIP [jepsen.kudu] run tests from clojure-maven-plugin .. Patch Set 7: (6 comments) http://gerrit.cloudera.org:8080/#/c/5551/7/java/kudu-jepsen/.gitignore File

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

2017-01-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: Link curl against el6 workaround openssl .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5603/1//COMMIT_MSG Commit Message: Line 7:

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

2017-01-04 Thread Mike Percy (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5603 to review the following change. Change subject: thirdparty: Link curl against el6 workaround openssl ..

[kudu-CR] Restructure release notes in preparation for 1.2 release

2017-01-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Restructure release notes in preparation for 1.2 release .. Patch Set 1: (3 comments) Mind also generating site we can look at while reviewing?

[kudu-CR] Restructure release notes in preparation for 1.2 release

2017-01-04 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, John Russell, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5602 to review the following change. Change subject: Restructure release notes in preparation for 1.2 release

[kudu-CR](gh-pages) KUDU-1818. Add link to Cloudera training in FAQ

2017-01-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1818. Add link to Cloudera training in FAQ .. KUDU-1818. Add link to Cloudera training in FAQ Additionally I snuck in a formatting fix I

[kudu-CR](gh-pages) KUDU-1818. Add link to Cloudera training in FAQ

2017-01-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1818. Add link to Cloudera training in FAQ .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5601 To

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (3 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/build-support/jenkins/build-and-test.sh File

[kudu-CR](gh-pages) KUDU-1818. Add link to Cloudera training in FAQ

2017-01-04 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5601 to review the following change. Change subject: KUDU-1818. Add link to Cloudera training in FAQ .. KUDU-1818.

[kudu-CR](branch-1.2.x) KUDU-1655. Update docs for ASF maven repository coordinates

2017-01-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1655. Update docs for ASF maven repository coordinates .. KUDU-1655. Update docs for ASF maven repository coordinates - Update Maven Artifacts -

[kudu-CR](branch-1.2.x) KUDU-1655. Update docs for ASF maven repository coordinates

2017-01-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1655. Update docs for ASF maven repository coordinates .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5600 To

[kudu-CR] KUDU-1655. Update docs for ASF maven repository coordinates

2017-01-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1655. Update docs for ASF maven repository coordinates .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5581 To unsubscribe,

[kudu-CR] KUDU-1655. Update docs for ASF maven repository coordinates

2017-01-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1655. Update docs for ASF maven repository coordinates .. KUDU-1655. Update docs for ASF maven repository coordinates - Update Maven Artifacts -

[kudu-CR] KUDU-1741: Make MiniCluster and ExternalMiniCluster follow one semantic for Restart

2017-01-04 Thread Dinesh Bhat (Code Review)
Hello Mike Percy, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5598 to review the following change. Change subject: KUDU-1741: Make MiniCluster and ExternalMiniCluster follow one semantic for Restart

[kudu-CR] KUDU-1760. Add test coverage of reading pre-REINSERT after ALTER

2017-01-04 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5488 to look at the new patch set (#2). Change subject: KUDU-1760. Add test coverage of reading pre-REINSERT after ALTER

[kudu-CR] [jepsen.kudu] some updates

2017-01-04 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5500 to look at the new patch set (#6). Change subject: [jepsen.kudu] some updates .. [jepsen.kudu] some updates

[kudu-CR] [jepsen.kudu] some updates

2017-01-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [jepsen.kudu] some updates .. Patch Set 5: (8 comments) http://gerrit.cloudera.org:8080/#/c/5500/5/java/kudu-jepsen/README.md File java/kudu-jepsen/README.md: