[kudu-CR] catalog manager: adjust maximum tablet count at table creation time

2018-10-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/11716 ) Change subject: catalog_manager: adjust maximum tablet count at table creation time .. Patch Set 3: (2 comments)

[kudu-CR] build-and-test.sh: unbreak Python 2.6 based builds

2018-04-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9936 ) Change subject: build-and-test.sh: unbreak Python 2.6 based builds .. build-and-test.sh: unbreak Python 2.6 based builds

[kudu-CR] build-and-test.sh: unbreak Python 2.6 based builds

2018-04-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9936 ) Change subject: build-and-test.sh: unbreak Python 2.6 based builds .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] build: Adjust SHA-1 file suffix to .sha1

2018-03-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9756 ) Change subject: build: Adjust SHA-1 file suffix to .sha1 .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [consensus] KUDU-2367 fix replica health report

2018-03-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9755 ) Change subject: [consensus] KUDU-2367 fix replica health report .. Patch Set 2: (1 comment)

[kudu-CR](gh-pages) Update website for 1.7.0 release

2018-03-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9733 ) Change subject: Update website for 1.7.0 release .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](gh-pages) Update download URLs

2018-03-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9707 ) Change subject: Update download URLs .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9707 To

[kudu-CR] Minor updates to RELEASING.adoc

2018-03-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9692 ) Change subject: Minor updates to RELEASING.adoc .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1447. Automatically disable THP on process heap

2018-03-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9676 ) Change subject: KUDU-1447. Automatically disable THP on process heap .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1447. Automatically disable THP on process heap

2018-03-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9676 ) Change subject: KUDU-1447. Automatically disable THP on process heap .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2343. java: properly reconnect to new leader master after failover

2018-03-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9612 ) Change subject: KUDU-2343. java: properly reconnect to new leader master after failover .. Patch Set 1: > Example alternate

[kudu-CR] KUDU-2264. java: automatically attempt to re-acquire Kerberos credentials before expiration

2018-03-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9050 ) Change subject: KUDU-2264. java: automatically attempt to re-acquire Kerberos credentials before expiration .. Patch Set 3: (1

[kudu-CR] Fix crash when log dir is not specified

2018-02-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9233 ) Change subject: Fix crash when log_dir is not specified .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](gh-pages) Fix broken doc guide link

2018-02-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9205 ) Change subject: Fix broken doc guide link .. Patch Set 1: Verified+1 Code-Review+2 -- To view, visit

[kudu-CR](gh-pages) Fix broken doc guide link

2018-02-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9205 ) Change subject: Fix broken doc guide link .. Fix broken doc guide link Change-Id:

[kudu-CR](gh-pages) Update the latest committers

2018-02-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9203 ) Change subject: Update the latest committers .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/9203

[kudu-CR](gh-pages) Update the latest committers

2018-02-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9203 ) Change subject: Update the latest committers .. Update the latest committers Change-Id:

[kudu-CR](gh-pages) Update the latest committers

2018-02-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9203 ) Change subject: Update the latest committers .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-01-31 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9172 ) Change subject: maintenance_manager: log the reason for scheduling each operation .. Patch Set 1: (1 comment)

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-01-31 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9172 ) Change subject: maintenance_manager: log the reason for scheduling each operation .. Patch Set 1: (1 comment)

[kudu-CR] [docs] Removed the note about KUDU-1626

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9020 ) Change subject: [docs] Removed the note about KUDU-1626 .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [docs] Removed the note about KUDU-1626

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9020 ) Change subject: [docs] Removed the note about KUDU-1626 .. [docs] Removed the note about KUDU-1626 Change-Id:

[kudu-CR] [docs] Added a section for Kudu clients connecting to secure clusters

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8953 ) Change subject: [docs] Added a section for Kudu clients connecting to secure clusters .. Patch Set 8: Code-Review+2 -- To view,

[kudu-CR] [docs] Added a section for Kudu clients connecting to secure clusters

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8953 ) Change subject: [docs] Added a section for Kudu clients connecting to secure clusters .. [docs] Added a section for Kudu

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has removed a vote on this change. Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .. [docs] Added steps to update

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .. Patch Set 15: Verified+1

[kudu-CR](branch-1.6.x) rpc: allow setting --rpc tls min protocol on older RHEL versions

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9055 ) Change subject: rpc: allow setting --rpc_tls_min_protocol on older RHEL versions .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.5.x) rpc: allow setting --rpc tls min protocol on older RHEL versions

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9056 ) Change subject: rpc: allow setting --rpc_tls_min_protocol on older RHEL versions .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.5.x) rpc: allow setting --rpc tls min protocol on older RHEL versions

2018-01-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9056 ) Change subject: rpc: allow setting --rpc_tls_min_protocol on older RHEL versions .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.5.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8941 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 8: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.6.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9008 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.6.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9008 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. KUDU-2251: rowset size can overflow int in

[kudu-CR](branch-1.5.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8941 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. KUDU-2251: rowset size can overflow int in

[kudu-CR] [doc] Added a section for Kudu clients connecting to secure clusters.

2018-01-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8953 ) Change subject: [doc] Added a section for Kudu clients connecting to secure clusters. .. Patch Set 5: (1 comment)

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters.

2018-01-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters. .. Patch Set 4: (1 comment)

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters.

2018-01-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters. .. Patch Set 4: (2 comments)

[kudu-CR] docs: fix tablet moving in 1.5.0 release notes

2018-01-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8939 ) Change subject: docs: fix tablet moving in 1.5.0 release notes .. docs: fix tablet moving in 1.5.0 release notes Release

[kudu-CR] docs: fix tablet moving in 1.5.0 release notes

2018-01-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8939 ) Change subject: docs: fix tablet moving in 1.5.0 release notes .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.5.x) KUDU-2231: sparse column predicate can cause excessive data-block reads

2017-12-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8903 ) Change subject: KUDU-2231: sparse column predicate can cause excessive data-block reads .. Patch Set 1: Code-Review+2 -- To

[kudu-CR](branch-1.5.x) KUDU-2231: sparse column predicate can cause excessive data-block reads

2017-12-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8903 ) Change subject: KUDU-2231: sparse column predicate can cause excessive data-block reads .. KUDU-2231: sparse column

[kudu-CR](branch-1.6.x) KUDU-2231: sparse column predicate can cause excessive data-block reads

2017-12-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8902 ) Change subject: KUDU-2231: sparse column predicate can cause excessive data-block reads .. Patch Set 1: Code-Review+2 -- To

[kudu-CR](branch-1.6.x) KUDU-2231: sparse column predicate can cause excessive data-block reads

2017-12-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8902 ) Change subject: KUDU-2231: sparse column predicate can cause excessive data-block reads .. KUDU-2231: sparse column

[kudu-CR] KUDU-2216. Post process gtest generated xml to include the output from the *.txt files

2017-12-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8757 ) Change subject: KUDU-2216. Post process gtest generated xml to include the output from the *.txt files .. Patch Set 7: (2

[kudu-CR](gh-pages) Add backdated announcement blog post for 1.6.0

2017-12-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8812 ) Change subject: Add backdated announcement blog post for 1.6.0 .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] docs: clarify disk failure recovery administration

2017-12-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8811 ) Change subject: docs: clarify disk failure recovery administration .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] docs: clarify disk failure recovery administration

2017-12-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8811 ) Change subject: docs: clarify disk failure recovery administration .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] docs: clarify disk failure recovery administration

2017-12-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8811 ) Change subject: docs: clarify disk failure recovery administration .. Patch Set 1: (3 comments)

[kudu-CR](branch-1.6.x) kudu-hive: skip publishing artifact

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8794 ) Change subject: kudu-hive: skip publishing artifact .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.6.x) kudu-hive: skip publishing artifact

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8794 ) Change subject: kudu-hive: skip publishing artifact .. kudu-hive: skip publishing artifact kudu-hive is experimental, so

[kudu-CR](branch-1.6.x) java: re-revisit maven shade configuration

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8793 ) Change subject: java: re-revisit maven shade configuration .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.6.x) java: re-revisit maven shade configuration

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8793 ) Change subject: java: re-revisit maven shade configuration .. java: re-revisit maven shade configuration I introduced a

[kudu-CR] java: re-revisit maven shade configuration

2017-12-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8792 ) Change subject: java: re-revisit maven shade configuration .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.5.x) KUDU-2209. HybridClock doesn't handle changes in STA NANO flag

2017-12-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8699 ) Change subject: KUDU-2209. HybridClock doesn't handle changes in STA_NANO flag .. Patch Set 1: Verified+1 Code-Review+2 -- To

[kudu-CR](branch-1.5.x) KUDU-2209. HybridClock doesn't handle changes in STA NANO flag

2017-12-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has removed a vote on this change. Change subject: KUDU-2209. HybridClock doesn't handle changes in STA_NANO flag .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR](branch-1.5.x) KUDU-2209. HybridClock doesn't handle changes in STA NANO flag

2017-12-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8699 ) Change subject: KUDU-2209. HybridClock doesn't handle changes in STA_NANO flag .. KUDU-2209. HybridClock doesn't handle

[kudu-CR] Update version to 1.7.0-SNAPSHOT

2017-11-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8694 ) Change subject: Update version to 1.7.0-SNAPSHOT .. Update version to 1.7.0-SNAPSHOT Change-Id:

[kudu-CR] Update version to 1.7.0-SNAPSHOT

2017-11-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8694 ) Change subject: Update version to 1.7.0-SNAPSHOT .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] docs: clarify at-rest encryption known limitation

2017-11-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8556 ) Change subject: docs: clarify at-rest encryption known limitation .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] Revert "KUDU-2191 (3/n): Add mini HMS option to external mini cluster"

2017-11-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has abandoned this change. ( http://gerrit.cloudera.org:8080/8457 ) Change subject: Revert "KUDU-2191 (3/n): Add mini HMS option to external mini cluster" .. Abandoned https://gerrit.cloudera.org/#/c/8460/

[kudu-CR] Revert "KUDU-2191 (2/n): Hive Metastore client"

2017-11-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has abandoned this change. ( http://gerrit.cloudera.org:8080/8458 ) Change subject: Revert "KUDU-2191 (2/n): Hive Metastore client" .. Abandoned https://gerrit.cloudera.org/#/c/8460/ supersedes -- To view,

[kudu-CR] Revert "KUDU-2191 (2/n): Hive Metastore client"

2017-11-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8458 ) Change subject: Revert "KUDU-2191 (2/n): Hive Metastore client" .. Patch Set 1: (1 comment)

[kudu-CR] Revert "KUDU-2191 (2/n): Hive Metastore client"

2017-11-03 Thread Jean-Daniel Cryans (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8458 to look at the new patch set (#2). Change subject: Revert "KUDU-2191 (2/n): Hive Metastore client"

[kudu-CR] Revert "KUDU-2191 (2/n): Hive Metastore client"

2017-11-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has reverted this change. ( http://gerrit.cloudera.org:8080/8458 ) Change subject: Revert "KUDU-2191 (2/n): Hive Metastore client" .. Uploaded patch set 1. -- To view, visit

[kudu-CR] Revert "KUDU-2191 (3/n): Add mini HMS option to external mini cluster"

2017-11-03 Thread Jean-Daniel Cryans (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8457 to review the following change. Change subject: Revert "KUDU-2191 (3/n): Add mini HMS option to external mini cluster"

[kudu-CR] Revert "KUDU-2191 (2/n): Hive Metastore client"

2017-11-03 Thread Jean-Daniel Cryans (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8458 to review the following change. Change subject: Revert "KUDU-2191 (2/n): Hive Metastore client"

[kudu-CR] Revert "KUDU-2191 (3/n): Add mini HMS option to external mini cluster"

2017-11-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has reverted this change. ( http://gerrit.cloudera.org:8080/8457 ) Change subject: Revert "KUDU-2191 (3/n): Add mini HMS option to external mini cluster" .. Uploaded patch set 1. -- To view, visit

[kudu-CR](branch-1.2.x) [client-test] Reduce flakyness of TestWriteWithDeadTabletServer

2017-11-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8429 ) Change subject: [client-test] Reduce flakyness of TestWriteWithDeadTabletServer .. [client-test] Reduce flakyness of

[kudu-CR](branch-1.2.x) [client-test] Reduce flakyness of TestWriteWithDeadTabletServer

2017-11-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8429 ) Change subject: [client-test] Reduce flakyness of TestWriteWithDeadTabletServer .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] python: upgrade pip when building client and restore old workarounds

2017-10-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8406 ) Change subject: python: upgrade pip when building client and restore old workarounds .. python: upgrade pip when building

[kudu-CR] python: upgrade pip when building client and restore old workarounds

2017-10-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8406 ) Change subject: python: upgrade pip when building client and restore old workarounds .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR] [java client] improve AsyncKuduScanner logging

2017-10-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8382 ) Change subject: [java client] improve AsyncKuduScanner logging .. Patch Set 2: (3 comments)

[kudu-CR] [java client] imporve AsyncKuduScanner logging

2017-10-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8382 ) Change subject: [java client] imporve AsyncKuduScanner logging .. Patch Set 1: (1 comment)

[kudu-CR] [java client] imporve AsyncKuduScanner logging

2017-10-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8382 ) Change subject: [java client] imporve AsyncKuduScanner logging .. Patch Set 1: (5 comments)

[kudu-CR](gh-pages) Add slides from my Strata 2017 talk

2017-10-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8362 ) Change subject: Add slides from my Strata 2017 talk .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] kudu-hive: add provided scope dependency on hadoop-mapreduce-client-common

2017-10-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8316 ) Change subject: kudu-hive: add provided scope dependency on hadoop-mapreduce-client-common .. Patch Set 1: Code-Review+2 -- To

[kudu-CR] kudu-hive: add provided scope dependency on hadoop-mapreduce-client-common

2017-10-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8316 ) Change subject: kudu-hive: add provided scope dependency on hadoop-mapreduce-client-common .. kudu-hive: add provided

[kudu-CR] java client: remove JDK8 library calls from MiniKuduCluster

2017-10-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8248 ) Change subject: java client: remove JDK8 library calls from MiniKuduCluster .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.5.x) KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8223 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. Patch Set 1: Code-Review+2 -- To

[kudu-CR](branch-1.5.x) KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8223 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. KUDU-2167: fix C++ client

[kudu-CR](branch-1.4.x) KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8224 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. Patch Set 1: Code-Review+2 -- To

[kudu-CR](branch-1.4.x) KUDU-2167: fix C++ client crash due to bad assumption regarding scan data

2017-10-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8224 ) Change subject: KUDU-2167: fix C++ client crash due to bad assumption regarding scan data .. KUDU-2167: fix C++ client

[kudu-CR](branch-1.4.x) KUDU-2032 (part 1): pass pre-resolution hostname into RPC proxies

2017-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8185 ) Change subject: KUDU-2032 (part 1): pass pre-resolution hostname into RPC proxies .. Patch Set 2: > The latest Jenkins failure

[kudu-CR] python: pin to cython 0.26.1

2017-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8155 ) Change subject: python: pin to cython 0.26.1 .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients

2017-09-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] [docs] fixup releasing git instructions

2017-09-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] fixup releasing git instructions .. [docs] fixup releasing git instructions Change-Id: I83f98350ddcaf062477db1ee0b90f1811f87524b

[kudu-CR] [docs] fixup releasing git instructions

2017-09-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] fixup releasing git instructions .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7876 To unsubscribe, visit

[kudu-CR] KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients

2017-09-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients .. Patch Set 3: (3 comments)

[kudu-CR] KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients

2017-09-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients .. Patch Set 3: (3 comments)

[kudu-CR] KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients

2017-09-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients .. Patch Set 2: (1 comment)

[kudu-CR](gh-pages) Add backdated announcement blog post for 1.5

2017-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add backdated announcement blog post for 1.5 .. Add backdated announcement blog post for 1.5 Change-Id: I301db21a8c5e75b2cbe47b3992094d7b1eca7731

[kudu-CR](gh-pages) Add backdated announcement blog post for 1.5

2017-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add backdated announcement blog post for 1.5 .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8058 To unsubscribe, visit

[kudu-CR](gh-pages) Add backdated announcement blog post for 1.5

2017-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add backdated announcement blog post for 1.5 .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8058 To unsubscribe, visit

[kudu-CR] KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients

2017-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries.

2017-09-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries. .. Patch Set 1: (1 comment)

[kudu-CR] docs: add 1.5.0 release note for thread count reduction

2017-08-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: add 1.5.0 release note for thread count reduction .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7905 To

[kudu-CR] docs: add 1.5.0 release note for thread count reduction

2017-08-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: add 1.5.0 release note for thread count reduction .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7905/3/docs/release_notes.adoc

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1894 fixed deadlock in client.Connection .. KUDU-1894 fixed deadlock in client.Connection Due to the reverse order of acquiring of the

[kudu-CR] KUDU-1544: Race in Java client's AsyncKuduSession.apply()

2017-08-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply() .. Patch Set 1: Code-Review+2 Did you loop TestAsyncKuduSession? -- To view, visit

[kudu-CR](branch-1.3.x) [build-support] IWYU build configuration for Jenkins

2017-08-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [build-support] IWYU build configuration for Jenkins .. [build-support] IWYU build configuration for Jenkins In this branch, the IWYU target is

  1   2   3   4   5   6   7   8   >