[kudu-CR](branch-1.2.x) KUDU-1852. KuduTableAlterer should not crash with nullptr arguments

2017-03-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1852. KuduTableAlterer should not crash with nullptr arguments .. KUDU-1852. KuduTableAlterer should not crash with nullptr arguments Change-

[kudu-CR](branch-1.2.x) KUDU-1821. Noisy warning from catalog manager

2017-03-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1821. Noisy warning from catalog manager .. KUDU-1821. Noisy warning from catalog manager This commit changes the LOG if the catalog manager i

[kudu-CR](branch-1.2.x) KUDU-1821. Noisy warning from catalog manager

2017-03-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1821. Noisy warning from catalog manager .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6205 To unsubscribe, visit ht

[kudu-CR](branch-1.2.x) KUDU-1904 Don't seek on empty RLE blocks

2017-03-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1904 Don't seek on empty RLE blocks .. KUDU-1904 Don't seek on empty RLE blocks Seeking in RLE blocks enforces that the seek position is less

[kudu-CR](branch-1.2.x) KUDU-1904 Don't seek on empty RLE blocks

2017-03-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1904 Don't seek on empty RLE blocks .. Patch Set 2: Code-Review+2 Verified+1 Test failure isn't related, although it's an odd one. -- To view, visit

[kudu-CR] Java Client: authenticate with login username when using SASL PLAIN

2017-03-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Java Client: authenticate with login username when using SASL PLAIN .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6209 T

[kudu-CR] Update version to 1.4.0-SNAPSHOT

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Update version to 1.4.0-SNAPSHOT .. Update version to 1.4.0-SNAPSHOT Change-Id: I128157c46f6c8f77ecee604591619917f5d1ce42 Reviewed-on: http://gerri

[kudu-CR] Update version to 1.4.0-SNAPSHOT

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Update version to 1.4.0-SNAPSHOT .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6223 To unsubscribe, visit http://gerrit.c

[kudu-CR] KUDU-1906. Fix lost callback for scanner path

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1906. Fix lost callback for scanner path .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6239 To unsubscribe, visit ht

[kudu-CR] ITBLL: avoid creating client many times during Generator step

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ITBLL: avoid creating client many times during Generator step .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6236 To unsub

[kudu-CR] mapreduce: support for running on secure clusters

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: mapreduce: support for running on secure clusters .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6237 To unsubscribe, visi

[kudu-CR] KUDU-1906. Fix lost callback for scanner path

2017-03-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1906. Fix lost callback for scanner path .. KUDU-1906. Fix lost callback for scanner path Fixes another case similar to KUDU-1888 in which we

[kudu-CR] KUDU-1906. Fix lost callback for scanner path

2017-03-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1906. Fix lost callback for scanner path .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6239 To unsubscribe, visit ht

[kudu-CR](branch-1.3.x) KUDU-1897: disable Kerberos replay cache

2017-03-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1897: disable Kerberos replay cache .. KUDU-1897: disable Kerberos replay cache Change-Id: Ifbce55a0b12682fdf69e7b2c361c6336495db64d Reviewed-

[kudu-CR] [exactly once rpc-test] Reduce flakyness of TestExactlyOnceSemanticsGarbageCollection

2017-03-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [exactly_once_rpc-test] Reduce flakyness of TestExactlyOnceSemanticsGarbageCollection .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloude

[kudu-CR] [exactly once rpc-test] Reduce flakyness of TestExactlyOnceSemanticsGarbageCollection

2017-03-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [exactly_once_rpc-test] Reduce flakyness of TestExactlyOnceSemanticsGarbageCollection .. [exactly_once_rpc-test] Reduce flakyness of TestExactlyOn

[kudu-CR] java: avoid spewing ClosedChannelException on client-initiated disconnects

2017-03-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: avoid spewing ClosedChannelException on client-initiated disconnects .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR](branch-1.3.x) [rpc] Do not send error reply for invalid connection context

2017-03-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [rpc] Do not send error reply for invalid connection context .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6304 To unsubs

[kudu-CR](branch-1.3.x) env: Add support for getting FS capacity

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: env: Add support for getting FS capacity .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6277 To unsubscribe, visit http://

[kudu-CR](branch-1.3.x) env: Add support for getting FS capacity

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: env: Add support for getting FS capacity .. env: Add support for getting FS capacity We need a way to get FS capacity in a follow-up patch. This ch

[kudu-CR](branch-1.3.x) Ignore SIGPIPE earlier in startup process

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Ignore SIGPIPE earlier in startup process .. Ignore SIGPIPE earlier in startup process This change resolves a race during startup where we are not

[kudu-CR](branch-1.3.x) Ignore SIGPIPE earlier in startup process

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Ignore SIGPIPE earlier in startup process .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6276 To unsubscribe, visit http:/

[kudu-CR](branch-1.3.x) Reserve 1% of disk space by default

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Reserve 1% of disk space by default .. Reserve 1% of disk space by default Many people are not aware that Kudu is able to reserve a certain amount

[kudu-CR](branch-1.3.x) Reserve 1% of disk space by default

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Reserve 1% of disk space by default .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6278 To unsubscribe, visit http://gerri

[kudu-CR](branch-1.3.x) [rpc] Do not send error reply for invalid connection context

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [rpc] Do not send error reply for invalid connection context .. Patch Set 1: Verified+1 Build likely failed due to KUDU-1894 which we really need to fix. -

[kudu-CR](branch-1.3.x) [rpc] Do not send error reply for invalid connection context

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [rpc] Do not send error reply for invalid connection context .. [rpc] Do not send error reply for invalid connection context We recently introduced

[kudu-CR](branch-1.3.x) java: avoid spewing ClosedChannelException on client-initiated disconnects

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: java: avoid spewing ClosedChannelException on client-initiated disconnects .. java: avoid spewing ClosedChannelException on client-initiated discon

[kudu-CR](branch-1.3.x) java: avoid spewing ClosedChannelException on client-initiated disconnects

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: avoid spewing ClosedChannelException on client-initiated disconnects .. Patch Set 1: Code-Review+2 Verified+1 Test failure is not related but it's re

[kudu-CR] WIP: java: more error message cleanup

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: WIP: java: more error message cleanup .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6321/1/java/kudu-client/src/main/java/org/apache/kud

[kudu-CR](branch-1.3.x) Fix Webserver option 'password file' to pass correct Squeasel option 'global auth file' for enabling HTTP authorization.

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Fix Webserver option 'password_file' to pass correct Squeasel option 'global_auth_file' for enabling HTTP authorization. .. Patch Set 1: Code-Review+2 --

[kudu-CR](branch-1.3.x) [sys catalog] no copying into intermediate KuduPartialRow

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [sys_catalog] no copying into intermediate KuduPartialRow .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6324 To unsubscri

[kudu-CR](branch-1.3.x) Fix Webserver option 'password file' to pass correct Squeasel option 'global auth file' for enabling HTTP authorization.

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Fix Webserver option 'password_file' to pass correct Squeasel option 'global_auth_file' for enabling HTTP authorization. .. Fix Webserver option 'p

[kudu-CR](branch-1.3.x) [sys catalog] no copying into intermediate KuduPartialRow

2017-03-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [sys_catalog] no copying into intermediate KuduPartialRow .. [sys_catalog] no copying into intermediate KuduPartialRow Use KuduPartialRow::SetStrin

[kudu-CR](branch-1.3.x) Change version to 1.3.0 (non-SNAPSHOT)

2017-03-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Change version to 1.3.0 (non-SNAPSHOT) .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6331 To unsubscribe, visit http://ge

[kudu-CR] docs: release notes for 1.3

2017-03-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: release notes for 1.3 .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6334/1/docs/prior_release_notes.adoc File docs/prior_release_n

[kudu-CR] docs: release notes for 1.3

2017-03-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: release notes for 1.3 .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6334 To unsubscribe, visit http://gerrit.cloude

[kudu-CR](branch-1.3.x) [util] fixed env util-test.cc compilation on OS X

2017-03-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [util] fixed env_util-test.cc compilation on OS X .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6351 To unsubscribe, visi

[kudu-CR] ITBLL: clean up exception handling

2017-03-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ITBLL: clean up exception handling .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6407 To unsubscribe, visit http://gerrit

[kudu-CR] ITBLL: clean up exception handling

2017-03-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: ITBLL: clean up exception handling .. ITBLL: clean up exception handling Change-Id: I8fcd624e709d0f3a931055c1b4b38aab2d5b2e37 Reviewed-on: http://g

[kudu-CR] kudu-mapreduce: tweak poms

2017-03-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: kudu-mapreduce: tweak poms .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6418/1//COMMIT_MSG Commit Message: Line 22: fat jar containing

[kudu-CR] kudu-spark: switch default profile from spark1/2.10 to spark2/2.11

2017-03-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: kudu-spark: switch default profile from spark1/2.10 to spark2/2.11 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6420 To

[kudu-CR] Spark ITBLL

2017-03-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Spark ITBLL .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6419/1/java/kudu-client-tools/src/main/java/org/apache/kudu/mapreduce/tools/Bi

[kudu-CR] [docs] Add another workaround for macOS

2017-03-21 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6456 to review the following change. Change subject: [docs] Add another workaround for macOS .. [docs] Add another workaroun

[kudu-CR] [docs] Add another workaround for macOS

2017-03-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Add another workaround for macOS .. [docs] Add another workaround for macOS I encountered an issue that was preventing me from building thir

[kudu-CR] design docs: Add tablet history GC design doc

2017-03-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: design docs: Add tablet history GC design doc .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/2853/4/docs/design-docs/tablet-history-gc.md

[kudu-CR] design docs: Add tablet history GC design doc

2017-03-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: design docs: Add tablet history GC design doc .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/2853/4/docs/design-docs/tablet-history-gc.md

[kudu-CR] docs: Add documentation for background tasks

2017-03-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: Add documentation for background tasks .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6501/4/docs/background_tasks.adoc File docs/ba

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/6516/1/java/kudu-client/src/main/ja

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY .. Patch Set 4: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6516/4/java/kudu-clien

[kudu-CR](branch-1.3.x) KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6519 To unsubscribe,

[kudu-CR] [docs] Add security guide

2017-04-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Add security guide .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6479/2/docs/security.adoc File docs/security.adoc: Line 213: Kud

[kudu-CR] KUDU-1939. Remove WARNING about large arenas

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1939. Remove WARNING about large arenas .. KUDU-1939. Remove WARNING about large arenas This WARNING turned out to be scary to users and hasn'

[kudu-CR] KUDU-1939. Remove WARNING about large arenas

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1939. Remove WARNING about large arenas .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6542 To unsubscribe, visit htt

[kudu-CR] KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloud

[kudu-CR] KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation .. KUDU-1963. Avoid NPE and SSLException when a TabletClient

[kudu-CR](branch-1.3.x) KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloud

[kudu-CR] docs: Add documentation for background tasks

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: Add documentation for background tasks .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6501 To unsubscribe, visit htt

[kudu-CR] docs: Add documentation for background tasks

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: Add documentation for background tasks .. Patch Set 5: Verified+1 Odd looking test failure but no way a docs changed caused it. -- To view, visit ht

[kudu-CR] docs: Add documentation for background tasks

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: docs: Add documentation for background tasks .. docs: Add documentation for background tasks This patch adds a documentation page for background ta

[kudu-CR](branch-1.3.x) KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation .. Patch Set 1: Verified+1 C++ test failure, this is a Java patch.

[kudu-CR](branch-1.3.x) KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1963. Avoid NPE and SSLException when a TabletClient is closed during negotiation .. KUDU-1963. Avoid NPE and SSLException when a TabletClient

[kudu-CR](branch-1.3.x) docs: Add a section for evolving gflags

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: docs: Add a section for evolving gflags .. docs: Add a section for evolving gflags This patch adds a new section for evolving gflags in the documen

[kudu-CR](branch-1.3.x) docs: Add a section for evolving gflags

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: Add a section for evolving gflags .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6546 To unsubscribe, visit http://g

[kudu-CR](branch-1.3.x) docs: Move documentation style guide from user guide to design-docs

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: Move documentation style guide from user guide to design-docs .. Patch Set 1: Code-Review+2 Verified+1 Flaky tests... -- To view, visit http://gerr

[kudu-CR](branch-1.3.x) docs: Move documentation style guide from user guide to design-docs

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: docs: Move documentation style guide from user guide to design-docs .. docs: Move documentation style guide from user guide to design-docs This is

[kudu-CR](branch-1.3.x) docs: Add documentation for background tasks

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: Add documentation for background tasks .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6548 To unsubscribe, visit htt

[kudu-CR](branch-1.3.x) docs: Add documentation for background tasks

2017-04-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: docs: Add documentation for background tasks .. docs: Add documentation for background tasks This patch adds a documentation page for background ta

[kudu-CR] kudu-spark: switch default profile from spark1/2.10 to spark2/2.11

2017-04-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: kudu-spark: switch default profile from spark1/2.10 to spark2/2.11 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6420 To

[kudu-CR] kudu-mapreduce: tweak poms

2017-04-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: kudu-mapreduce: tweak poms .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6418 To unsubscribe, visit http://gerrit.clouder

[kudu-CR] Spark ITBLL

2017-04-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Spark ITBLL .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6419 To unsubscribe, visit http://gerrit.cloudera.org:8080/sett

[kudu-CR](gh-pages) Add announcement post for 1.3.0

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

[kudu-CR](gh-pages) Add announcement post for 1.3.0

2017-04-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add announcement post for 1.3.0 .. Add announcement post for 1.3.0 Change-Id: I002e24b90ea28cd25168315c8e0e30645c7ff1ae Reviewed-on: http://gerrit.

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

2017-04-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump version to 1.3.1-SNAPSHOT .. Patch Set 1: You missed https://github.com/apache/kudu/blob/branch-1.3.x/python/setup.py#L41 -- To view, visit http://ge

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

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

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

2017-04-06 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 2: (18 comments) http://gerrit.cloudera.org:8080/#/c/6566/2/java/kudu-client/src/main/java/org/

[kudu-CR] KUDU-1424. Add getters to PartialRow

2017-04-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1424. Add getters to PartialRow .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6554/1/java/kudu-client/src/main/java/org/apache/kudu

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

2017-04-06 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 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6545 To unsubscribe, v

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

2017-04-06 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.2.x) KUDU-1607. Unpin tablet flush after failed bootstrap

2017-04-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/6572 Change subject: KUDU-1607. Unpin tablet flush after failed bootstrap .. KUDU-1607. Unpin tablet flush after failed bootstrap We hav

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

2017-04-06 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 1: Code-Review+2 Verified+1 Unrelated flake. -- To view, visit http://g

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

2017-04-06 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-06 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 1: Argh I think this is dependent on https://github.com/apache/kudu/commit/4114ba9bf1cdfb

[kudu-CR] [python] Fix flaky test test connect timeouts

2017-04-06 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/6573 to review the following change. Change subject: [python] Fix flaky test test_connect_timeouts .. [python] Fix flaky te

[kudu-CR] KUDU-1424. Add getters to PartialRow

2017-04-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1424. Add getters to PartialRow .. KUDU-1424. Add getters to PartialRow - Adds getters for each type to PartialRow - Adds public isNull and is

[kudu-CR] KUDU-1424. Add getters to PartialRow

2017-04-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1424. Add getters to PartialRow .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6554 To unsubscribe, visit http://gerr

[kudu-CR] Revert "KUDU-1853. Tablet copy: Don't orphan blocks on failure"

2017-04-12 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/6613 to review the following change. Change subject: Revert "KUDU-1853. Tablet copy: Don't orphan blocks on failure" .. Rev

[kudu-CR] Revert "KUDU-1853. Tablet copy: Don't orphan blocks on failure"

2017-04-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Revert "KUDU-1853. Tablet copy: Don't orphan blocks on failure" .. Patch Set 1: Code-Review-2 Getting a quick unit test run, Todd is filing a jira so we'll

[kudu-CR] KUDU-1968. Aborted tablet copies delete live blocks

2017-04-12 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/6613 to look at the new patch set (#2). Change subject: KUDU-1968. Aborted tablet copies delete live blocks

[kudu-CR] KUDU-1968. Aborted tablet copies delete live blocks

2017-04-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1968. Aborted tablet copies delete live blocks .. KUDU-1968. Aborted tablet copies delete live blocks This patch reverts "KUDU-1853. Tablet co

[kudu-CR](branch-1.3.x) KUDU-1968. Aborted tablet copies delete live blocks

2017-04-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1968. Aborted tablet copies delete live blocks .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6614 To unsubscribe, vi

[kudu-CR] java: add a stress test for short-lived clients

2017-04-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: java: add a stress test for short-lived clients .. java: add a stress test for short-lived clients This simulates an Impala frontend-like workload

[kudu-CR] java: add a stress test for short-lived clients

2017-04-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: add a stress test for short-lived clients .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6615 To unsubscribe, visit

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

2017-04-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump version to 1.3.1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6621 To unsubscribe, visit http://gerrit.cloudera.org

[kudu-CR](branch-1.3.x) Release notes for 1.3.1

2017-04-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for 1.3.1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6622 To unsubscribe, visit http://gerrit.cloudera.o

[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 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6660/2/src/kudu/conse

[kudu-CR](branch-1.2.x) KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart .. Patch Set 1: I was curious about the need for this patch. It's all tes

[kudu-CR](branch-1.3.x) [docs] Add another workaround for macOS

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

[kudu-CR](branch-1.3.x) [docs] Add another workaround for macOS

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Add another workaround for macOS .. [docs] Add another workaround for macOS I encountered an issue that was preventing me from building thir

[kudu-CR] 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/6668 to review the following change. Change subject: Bump version to 1.3.2-SNAPSHOT .. Bump version to 1.3.2-SNAPSHOT Chan

[kudu-CR](branch-1.2.x) KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart

2017-04-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart .. Patch Set 1: > As I understand, it's not necessary to merge this into

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