[kudu-CR](branch-1.3.x) java: fix incorrect hashmap usage in Statistics

2017-06-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: java: fix incorrect hashmap usage in Statistics .. java: fix incorrect hashmap usage in Statistics 65cb2edf5661599f689e28f4eec161fe062f7cb5 changed

[kudu-CR](branch-1.3.x) java: fix incorrect hashmap usage in Statistics

2017-06-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: fix incorrect hashmap usage in Statistics .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7339 To unsubscribe, visit

[kudu-CR](branch-1.4.x) java: fix incorrect hashmap usage in Statistics

2017-06-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: fix incorrect hashmap usage in Statistics .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7338 To unsubscribe, visit

[kudu-CR] [java] add BaseKuduTest.doSetup() method

2017-06-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] add BaseKuduTest.doSetup() method .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/7336/1/java/kudu-client/src/test/java/org/apache/

[kudu-CR] java: fix incorrect hashmap usage in Statistics

2017-06-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: fix incorrect hashmap usage in Statistics .. Patch Set 1: Code-Review+2 Should we backport? -- To view, visit http://gerrit.cloudera.org:8080/7335 T

[kudu-CR] [docs] Add an upgrade note about MM threads

2017-06-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Add an upgrade note about MM threads .. [docs] Add an upgrade note about MM threads Some users configure MM threads to high values to work a

[kudu-CR] [docs] Add an upgrade note about MM threads

2017-06-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Add an upgrade note about MM threads .. Patch Set 2: Verified+1 Overriding Jenkins since this is a docs change. -- To view, visit http://gerrit.clo

[kudu-CR] [docs] Add an upgrade note about MM threads

2017-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Add an upgrade note about MM threads .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/7283/1/docs/release_notes.adoc File docs/relea

[kudu-CR] [docs] Add an upgrade note about MM threads

2017-06-23 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7283 to look at the new patch set (#2). Change subject: [docs] Add an upgrade note about MM threads .. [docs]

[kudu-CR] [docs] Add an upgrade note about MM threads

2017-06-23 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7283 to review the following change. Change subject: [docs] Add an upgrade note about MM threads .. [docs] Add

[kudu-CR] [java-client] separating Connection

2017-06-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] separating Connection .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/7146/10/java/kudu-client/src/main/java/org/apache/kudu

[kudu-CR] Skip running Java integration tests twice

2017-06-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Skip running Java integration tests twice .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7195 To unsubscribe, visit http:/

[kudu-CR] [java-client] separating Connection

2017-06-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] separating Connection .. Patch Set 8: (3 comments) http://gerrit.cloudera.org:8080/#/c/7146/8/java/kudu-client/src/main/java/org/apache/kudu/

[kudu-CR](gh-pages) Add 1.4.0 release docs and links

2017-06-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add 1.4.0 release docs and links .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7169 To unsubscribe, visit http

[kudu-CR](gh-pages) Move some old download links to archive

2017-06-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Move some old download links to archive .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7174 To unsubscribe, vis

[kudu-CR](branch-1.4.x) Bump version to 1.4.1-SNAPSHOT

2017-06-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump version to 1.4.1-SNAPSHOT .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7170 To unsubscribe, visit http://gerrit.clo

[kudu-CR] KUDU-2003. Fix fd-cache related tests on high-core systems

2017-06-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-2003. Fix fd-cache related tests on high-core systems .. Patch Set 1: Code-Review+1 Seems fine but I really don't know that part of the code very well.

[kudu-CR] Undefined behavior in TlsSocket::Writev()

2017-06-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Undefined behavior in TlsSocket::Writev() .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/7141/8//COMMIT_MSG Commit Message: Line 7: Undef

[kudu-CR] Undefined behavior in TlsSocket::Writev()

2017-06-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Undefined behavior in TlsSocket::Writev() .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/7141/8//COMMIT_MSG Commit Message: Line 7: Undef

[kudu-CR] Temporary workaround for KUDU-1959 (race when selecting rowsets)

2017-06-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Temporary workaround for KUDU-1959 (race when selecting rowsets) .. Patch Set 2: Just leaving a comment to say that the jira should be KUDU-1956, not 1959.

[kudu-CR] Release notes for Kudu 1.4

2017-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for Kudu 1.4 .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7108 To unsubscribe, visit http://gerrit.clouder

[kudu-CR] Release notes for Kudu 1.4

2017-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for Kudu 1.4 .. Patch Set 1: Nice exhaustive list! Will +2 once the other issues are taken care of. -- To view, visit http://gerrit.cloudera

[kudu-CR](branch-1.3.x) [java-client] update protoc maven plugin

2017-06-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] update protoc maven plugin .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7070 To unsubscribe, visit http://

[kudu-CR](branch-1.3.x) [flags] fixed typo in group flag validation logic

2017-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [flags] fixed typo in group flag validation logic .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6864 To unsubscribe, visi

[kudu-CR](branch-1.3.x) KUDU-1941: more validation for RPC auth flags

2017-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1941: more validation for RPC auth flags .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6862 To unsubscribe, visit ht

[kudu-CR](branch-1.3.x) [flags] fixed typo in group flag validation logic

2017-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [flags] fixed typo in group flag validation logic .. [flags] fixed typo in group flag validation logic This is a follow-up for e7334c2e6607ac0fa778

[kudu-CR](branch-1.3.x) KUDU-1941: more validation for RPC auth flags

2017-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1941: more validation for RPC auth flags .. KUDU-1941: more validation for RPC auth flags With this patch, both master and tserver refuse to s

[kudu-CR] [doc] Remove beta upgrade reference

2017-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [doc] Remove beta upgrade reference .. [doc] Remove beta upgrade reference Change-Id: Ibde3132f3bffd1ca81d249fa9401d408dd47ff21 Reviewed-on: http:/

[kudu-CR] [doc] Remove beta upgrade reference

2017-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [doc] Remove beta upgrade reference .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6858 To unsubscribe, visit http://gerri

[kudu-CR] spark: add support for fault tolerant scanner

2017-05-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: spark: add support for fault tolerant scanner .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/6782/6/java/kudu-spark/src/main/scala/org/apa

[kudu-CR] WIP [java] re-acquire authn token if expired

2017-05-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: WIP [java] re-acquire authn token if expired .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/6816/2/java/kudu-client/src/main/java/org/apa

[kudu-CR](branch-1.3.x) KUDU-1993: fixed validation of 'grouped' gflags

2017-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1993: fixed validation of 'grouped' gflags .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6806 To unsubscribe, visit

[kudu-CR](branch-1.3.x) KUDU-1993: fixed validation of 'grouped' gflags

2017-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1993: fixed validation of 'grouped' gflags .. KUDU-1993: fixed validation of 'grouped' gflags Added generic implementation for grouped gflags

[kudu-CR](branch-1.3.x) KUDU-1981 Kudu should run at hosts len(FQDN) > 64

2017-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1981 Kudu should run at hosts len(FQDN) > 64 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6805 To unsubscribe, visi

[kudu-CR](branch-1.3.x) KUDU-1964. security: avoid calling ERR clear error() defensively

2017-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1964. security: avoid calling ERR_clear_error() defensively .. KUDU-1964. security: avoid calling ERR_clear_error() defensively This changes o

[kudu-CR](branch-1.3.x) KUDU-1964. security: avoid calling ERR clear error() defensively

2017-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1964. security: avoid calling ERR_clear_error() defensively .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6801 To un

[kudu-CR] spark: add support for fault tolerant scanner

2017-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: spark: add support for fault tolerant scanner .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/6782/1/java/kudu-spark/src/main/scala/org/ap

[kudu-CR] mapreduce: add support for fault tolerant scanner

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: mapreduce: add support for fault tolerant scanner .. mapreduce: add support for fault tolerant scanner This adds support to use fault tolerant scan

[kudu-CR] mapreduce: add support for fault tolerant scanner

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: mapreduce: add support for fault tolerant scanner .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6745 To unsubscribe, visi

[kudu-CR] mapreduce: add support for fault tolerant scanner

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: mapreduce: add support for fault tolerant scanner .. Patch Set 5: (2 comments) some nits and then we're good to go http://gerrit.cloudera.org:8080/#/c/674

[kudu-CR] [java client] improve flaky test ITClient

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] improve flaky test ITClient .. [java client] improve flaky test ITClient This changes ITClient test to use fault tolerant scanner. An

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/6630/13/src/kudu/cfile/cfile_reader.cc File src/k

[kudu-CR](branch-1.3.x) [doc] corrected superuser ACL flag name

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [doc] corrected superuser ACL flag name .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6788 To unsubscribe, visit http://gerr

[kudu-CR](branch-1.3.x) [doc] corrected superuser ACL flag name

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [doc] corrected superuser ACL flag name .. [doc] corrected superuser ACL flag name Change-Id: I514d32b1a631220746b778fd19b2c2db382f8cf8 Reviewed-on

[kudu-CR](branch-1.3.x) [doc] corrected superuser ACL flag name

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [doc] corrected superuser ACL flag name .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6788 To unsubscribe, visit http://g

[kudu-CR](branch-1.3.x) [docs] Add troubleshooting for KuduStorageHandler

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Add troubleshooting for KuduStorageHandler .. [docs] Add troubleshooting for KuduStorageHandler Change-Id: I80e028a6f827269d97f26ec7a2cf4b8c

[kudu-CR](branch-1.3.x) [docs] Refresh and augment the known issues

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augment the known issues We've learned a lot about Kudu since people have started

[kudu-CR] [java client] improve flaky test ITClient

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] improve flaky test ITClient .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6776 To unsubscribe, visit http:/

[kudu-CR](branch-1.3.x) [docs] Refresh and augment the known issues

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6786 to review the following change. Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augm

[kudu-CR] [docs] Refresh and augment the known issues

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has abandoned this change. Change subject: [docs] Refresh and augment the known issues .. Abandoned I always do this thing where I push against the master branch. -- To view, visit http://gerrit.cloudera.or

[kudu-CR](branch-1.3.x) [docs] Add troubleshooting for KuduStorageHandler

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/6785 Change subject: [docs] Add troubleshooting for KuduStorageHandler .. [docs] Add troubleshooting for KuduStorageHandler Change-Id: I

[kudu-CR] [docs] Refresh and augment the known issues

2017-05-03 Thread Jean-Daniel Cryans (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6784 to review the following change. Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augm

[kudu-CR] [java client] improve flaky test ITClient

2017-05-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] improve flaky test ITClient .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6776/2/java/kudu-client/src/test/java/org/apache/

[kudu-CR] [java client] improve flaky test ITClient

2017-05-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] improve flaky test ITClient .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6776/2/java/kudu-client/src/main/java/org/apache

[kudu-CR] mapreduce: add support for fault tolerant scanner

2017-04-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: mapreduce: add support for fault tolerant scanner .. Patch Set 2: The test failure is related to the fault tolerant scanner patch: 01:09:34.954 [INFO - kudu

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-579 [java_client] Scanner fault tolerance .. KUDU-579 [java_client] Scanner fault tolerance This patch adds java client support to restart sca

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-579 [java_client] Scanner fault tolerance .. Patch Set 10: Code-Review+2 Good job on the new test. -- To view, visit http://gerrit.cloudera.org:8080/

[kudu-CR] [docs] Add troubleshooting for KuduStorageHandler

2017-04-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Add troubleshooting for KuduStorageHandler .. [docs] Add troubleshooting for KuduStorageHandler Change-Id: I80e028a6f827269d97f26ec7a2cf4b8c

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-579 [java_client] Scanner fault tolerance .. Patch Set 8: (5 comments) http://gerrit.cloudera.org:8080/#/c/6566/8/java/kudu-client/src/main/java/org/a

[kudu-CR] Improve unsupported application feature flags error status

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Improve unsupported application feature flags error status .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6722 To unsubscr

[kudu-CR] Improve unsupported application feature flags error status

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Improve unsupported application feature flags error status .. Improve unsupported application feature flags error status It can be helpful to have

[kudu-CR] KUDU-861 Support changing column defaults and storage attributes

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-861 Support changing column defaults and storage attributes .. Patch Set 3: Code-Review+1 Carrying my +1 -- To view, visit http://gerrit.cloudera.org

[kudu-CR] [docs] Add troubleshooting for KuduStorageHandler

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6738 to review the following change. Change subject: [docs] Add troubleshooting for KuduStorageHandler .. [docs] Add troubl

[kudu-CR] KUDU-861 Support changing column defaults and storage attributes

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-861 Support changing column defaults and storage attributes .. Patch Set 2: Code-Review+1 Java side looks good. -- To view, visit http://gerrit.cloud

[kudu-CR](gh-pages) FAQ refresh

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: FAQ refresh .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6697/1/faq.md File faq.md: Line 194 > Why was this removed? Don't we still hav

[kudu-CR] [docs] Refresh and augment the known issues

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augment the known issues We've learned a lot about Kudu since people have started

[kudu-CR] [docs] Refresh and augment the known issues

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6699 to look at the new patch set (#5). Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh an

[kudu-CR] [docs] Refresh and augment the known issues

2017-04-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Refresh and augment the known issues .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6699/4/docs/developing.adoc File docs/developin

[kudu-CR] [docs] Refresh and augment the known issues

2017-04-24 Thread Jean-Daniel Cryans (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6699 to look at the new patch set (#4). Change subject: [docs] Refresh and augment the known issues .. [doc

[kudu-CR] [docs] Refresh and augment the known issues

2017-04-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Refresh and augment the known issues .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/6699/3/docs/developing.adoc File docs/developi

[kudu-CR] [docs] Refresh and augment the known issues

2017-04-20 Thread Jean-Daniel Cryans (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6699 to look at the new patch set (#3). Change subject: [docs] Refresh and augment the known issues .. [doc

[kudu-CR] [docs] Refresh and augment the known issues

2017-04-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Refresh and augment the known issues .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6699/2/docs/known_issues.adoc File docs/known_

[kudu-CR](gh-pages) FAQ refresh

2017-04-20 Thread Jean-Daniel Cryans (Code Review)
Hello Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6697 to look at the new patch set (#2). Change subject: FAQ refresh .. FAQ refresh Change-Id: Idd8a79e3067bedbef71b0dcf9

[kudu-CR](gh-pages) FAQ refresh

2017-04-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: FAQ refresh .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6697/1/faq.md File faq.md: PS1, Line 250: Kudu hasn't been officially tested w

[kudu-CR] [docs] Refresh and augment the known issues

2017-04-20 Thread Jean-Daniel Cryans (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6699 to look at the new patch set (#2). Change subject: [docs] Refresh and augment the known issues .. [doc

[kudu-CR] make site: only build necessary binaries

2017-04-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: make_site: only build necessary binaries .. make_site: only build necessary binaries make_site relies on these binaries to build docs for them, but

[kudu-CR] [docs] Refresh and augment the know issues

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Refresh and augment the know issues .. Patch Set 1: Poor rendering: https://github.com/jdcryans/kudu/blob/master/docs/known_issues.adoc -- To view

[kudu-CR] [docs] Refresh and augment the know issues

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6699 to review the following change. Change subject: [docs] Refresh and augment the know issues .. [docs] Refresh and augme

[kudu-CR](gh-pages) FAQ refresh

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6697 to review the following change. Change subject: FAQ refresh .. FAQ refresh Change-Id: Idd8a79e3067bedbef71b0dcf9921be

[kudu-CR] cache: add a benchmark

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: cache: add a benchmark .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6696/1/src/kudu/util/cache-bench.cc File src/kudu/util/cache-bench.c

[kudu-CR] make site: only build necessary binaries

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: make_site: only build necessary binaries .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6695 To unsubscribe, visit http://

[kudu-CR] cache: add a benchmark

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: cache: add a benchmark .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6696/1/src/kudu/util/cache-bench.cc File src/kudu/util/cache-bench.c

[kudu-CR](branch-1.3.x) KUDU-1708. Document Kudu command-line tools

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1708. Document Kudu command-line tools .. KUDU-1708. Document Kudu command-line tools - Adds helpxml flag support to the kudu binary - Adds do

[kudu-CR](branch-1.3.x) KUDU-1708. Document Kudu command-line tools

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1708. Document Kudu command-line tools .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6645 To unsubscribe, visit http

[kudu-CR](branch-1.3.x) [docs] Add admin workflow for recovering from disk failure

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Add admin workflow for recovering from disk failure .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6677 To unsubscr

[kudu-CR](gh-pages) Add blog post for 1.3.1

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add blog post for 1.3.1 .. Add blog post for 1.3.1 Change-Id: If8a94c8b75d4ba39fdee151e775a65ed34f5ee79 Reviewed-on: http://gerrit.cloudera.org:808

[kudu-CR](gh-pages) Add blog post for 1.3.1

2017-04-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add blog post for 1.3.1 .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6685 To unsubscribe, visit http://gerrit

[kudu-CR](branch-1.3.x) [docs] Add security guide

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Add security guide .. [docs] Add security guide Change-Id: Iabf60804975dc105243626be48d3a141c9a4dab5 Reviewed-on: http://gerrit.cloudera.org

[kudu-CR](branch-1.3.x) [docs] Add security guide

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Add security guide .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6647 To unsubscribe, visit http://gerrit.cloudera

[kudu-CR](branch-1.3.x) Add ksck section to admin guide common workflows

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add ksck section to admin guide common workflows .. Add ksck section to admin guide common workflows I've often wanted this when helping people thr

[kudu-CR](branch-1.3.x) Add ksck section to admin guide common workflows

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add ksck section to admin guide common workflows .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6646 To unsubscribe, visit

[kudu-CR](branch-1.3.x) docs: Fix axis lable for hash-range-partitioning

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: docs: Fix axis lable for hash-range-partitioning .. docs: Fix axis lable for hash-range-partitioning Horizontal axis label should show range partit

[kudu-CR](branch-1.3.x) docs: Fix axis lable for hash-range-partitioning

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: Fix axis lable for hash-range-partitioning .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6644 To unsubscribe, visit

[kudu-CR](branch-1.3.x) docs: Fix "make site"

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: docs: Fix "make site" .. docs: Fix "make site" * "make site" was broken due to changed command-line argments on "docs/support/scripts/make_site.s

[kudu-CR](branch-1.2.x) KUDU-1933. consensus: Avoid and repair integer overflow in log index

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1933. consensus: Avoid and repair integer overflow in log index .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6660

[kudu-CR](branch-1.2.x) KUDU-1933. consensus: Avoid and repair integer overflow in log index

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1933. consensus: Avoid and repair integer overflow in log index .. KUDU-1933. consensus: Avoid and repair integer overflow in log index We ob

[kudu-CR](branch-1.2.x) KUDU-1607. Unpin tablet flush after failed bootstrap

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1607. Unpin tablet flush after failed bootstrap .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6572/3//COMMIT_MSG Commi

[kudu-CR](branch-1.2.x) KUDU-1607. Unpin tablet flush after failed bootstrap

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1607. Unpin tablet flush after failed bootstrap .. KUDU-1607. Unpin tablet flush after failed bootstrap We have heard reports that, in certain

[kudu-CR](branch-1.3.x) Bump version to 1.3.2-SNAPSHOT

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Bump version to 1.3.2-SNAPSHOT .. Bump version to 1.3.2-SNAPSHOT Change-Id: Ie4c48aa6ec08a40500d85a1a53ccb48b64d2d4ef Reviewed-on: http://gerrit.cl

[kudu-CR](branch-1.3.x) Bump version to 1.3.2-SNAPSHOT

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6669 to review the following change. Change subject: Bump version to 1.3.2-SNAPSHOT .. Bump version to 1.3.2-SNAPSHOT Chan

<    1   2   3   4   5   6   7   8   9   10   >