[kudu-CR] KUDU-1704: add READ OWN WRITES scan mode

2017-12-12 Thread Hao Hao (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8804 to look at the new patch set (#2). Change subject: KUDU-1704: add READ_OWN_WRITES scan mode ..

[kudu-CR] KUDU-1704: add READ OWN WRITES scan mode

2017-12-12 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add READ_OWN_WRITES scan mode .. Patch Set 2: (3 comments)

[kudu-CR] KUDU-1704: add java client support for READ OWN WRITES mode

2017-12-14 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8847 Change subject: KUDU-1704: add java client support for READ_OWN_WRITES mode .. KUDU-1704: add java client support for READ_OWN_WRITES mode

[kudu-CR] KUDU-1704: add READ OWN WRITES scan mode

2017-12-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add READ_OWN_WRITES scan mode .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-1704: add READ OWN WRITES scan mode

2017-12-15 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add READ_OWN_WRITES scan mode .. Patch Set 4: (1 comment)

[kudu-CR] [java clien] update propagate TS for AUTO FLUSH SYNC flush mode

2017-12-13 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8837 Change subject: [java clien] update propagate TS for AUTO_FLUSH_SYNC flush mode .. [java clien] update propagate TS for AUTO_FLUSH_SYNC flush

[kudu-CR] KUDU-1704: add READ OWN WRITES scan mode

2017-12-13 Thread Hao Hao (Code Review)
Hello Alexey Serbin, David Ribeiro Alves, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8804 to look at the new patch set (#3). Change subject: KUDU-1704: add READ_OWN_WRITES scan mode

[kudu-CR] KUDU-1704: add READ OWN WRITES scan mode

2017-12-13 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add READ_OWN_WRITES scan mode .. Patch Set 2: (20 comments)

[kudu-CR] design-docs: improve cfile.md

2017-12-19 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8860 ) Change subject: design-docs: improve cfile.md .. Patch Set 1: (6 comments)

[kudu-CR] [spark] Remove syncCache in KuduConnection

2017-11-21 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8551 ) Change subject: [spark] Remove syncCache in KuduConnection .. Patch Set 4: (1 comment)

[kudu-CR] [spark] Remove syncCache in KuduConnection

2017-11-21 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8551 to look at the new patch set (#4). Change subject: [spark] Remove syncCache in KuduConnection

[kudu-CR] [spark] Remove syncCache in KuduConnection

2017-11-21 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8551 to look at the new patch set (#5). Change subject: [spark] Remove syncCache in KuduConnection

[kudu-CR] [spark] Remove syncCache in KuduConnection

2017-11-21 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8551 ) Change subject: [spark] Remove syncCache in KuduConnection .. Patch Set 5: (1 comment)

[kudu-CR] [spark] add 'local-cluster' mode for unit test

2017-11-21 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8585 ) Change subject: [spark] add 'local-cluster' mode for unit test .. Patch Set 1: > (5 comments) > > I wasn't able to get the tests to run on

[kudu-CR] KUDU-1454 [part 2]: enable scan locality for Spark

2017-11-15 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8560 Change subject: KUDU-1454 [part 2]: enable scan locality for Spark .. KUDU-1454 [part 2]: enable scan locality for Spark This patch provides

[kudu-CR] [java] Add ReplicaSelection in KuduScanToken

2017-11-15 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8559 Change subject: [java] Add ReplicaSelection in KuduScanToken .. [java] Add ReplicaSelection in KuduScanToken This patch adds ReplicaSelection

[kudu-CR] KUDU-1454 [part 1]: update propagated timestamp on the driver

2017-11-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8552 ) Change subject: KUDU-1454 [part 1]: update propagated timestamp on the driver .. Patch Set 3: (8 comments)

[kudu-CR] KUDU-1454 [part 1]: update propagated timestamp on the driver

2017-11-16 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8552 to look at the new patch set (#3). Change subject: KUDU-1454 [part 1]: update propagated timestamp on the driver

[kudu-CR] [java] Add ReplicaSelection in KuduScanToken

2017-11-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8559 ) Change subject: [java] Add ReplicaSelection in KuduScanToken .. Patch Set 2: (3 comments)

[kudu-CR] [java] Add ReplicaSelection in KuduScanToken

2017-11-16 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8559 to look at the new patch set (#2). Change subject: [java] Add ReplicaSelection in KuduScanToken ..

[kudu-CR] KUDU-1454 [part 2]: enable scan locality for Spark

2017-11-16 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8560 to look at the new patch set (#2). Change subject: KUDU-1454 [part 2]: enable scan locality for Spark

[kudu-CR] KUDU-1454 [part 2]: enable scan locality for Spark

2017-11-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8560 ) Change subject: KUDU-1454 [part 2]: enable scan locality for Spark .. Patch Set 2: (1 comment)

[kudu-CR] [spark] add 'local-cluster' mode for unit test

2017-11-16 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8585 Change subject: [spark] add 'local-cluster' mode for unit test .. [spark] add 'local-cluster' mode for unit test This patch adds

[kudu-CR] KUDU-1454 [part 1]: update propagated timestamp on the driver

2017-11-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8552 ) Change subject: KUDU-1454 [part 1]: update propagated timestamp on the driver .. Patch Set 4: > Build Failed > >

[kudu-CR] [spark] Remove AsyncClient in KuduContext

2017-11-16 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8551 to look at the new patch set (#3). Change subject: [spark] Remove AsyncClient in KuduContext

[kudu-CR] [spark] Remove AsyncClient in KuduContext

2017-11-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8551 ) Change subject: [spark] Remove AsyncClient in KuduContext .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-1737 : Submit column characteristics via KuduContext

2017-11-17 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/6010 ) Change subject: KUDU-1737 : Submit column characteristics via KuduContext .. Patch Set 3: >

[kudu-CR] [spark] Remove AsyncClient in KuduContext

2017-11-17 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8551 ) Change subject: [spark] Remove AsyncClient in KuduContext .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-1809: Add batch size configuration for ScanToken API

2017-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8435 ) Change subject: KUDU-1809: Add batch size configuration for ScanToken API .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1809: Add batch size configuration for ScanToken API

2017-11-01 Thread Hao Hao (Code Review)
Hello Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8435 to look at the new patch set (#2). Change subject: KUDU-1809: Add batch size configuration for ScanToken API

[kudu-CR] KUDU-1809: Add batch size configuration for ScanToken API

2017-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8435 ) Change subject: KUDU-1809: Add batch size configuration for ScanToken API .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-1411: Implement HT timestamp propagation in KuduScanToken

2017-11-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8452 ) Change subject: KUDU-1411: Implement HT timestamp propagation in KuduScanToken .. Patch Set 1: Another existing flaky test

[kudu-CR] KUDU-1809: Add batch size configuration for ScanToken API

2017-11-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8435 ) Change subject: KUDU-1809: Add batch size configuration for ScanToken API .. Patch Set 2: The failed test

[kudu-CR] KUDU-1411: Implement HT timestamp propagation in KuduScanToken

2017-11-01 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8452 Change subject: KUDU-1411: Implement HT timestamp propagation in KuduScanToken .. KUDU-1411: Implement HT timestamp propagation in

[kudu-CR] KUDU-1809: Add batch size configuration for ScanToken API

2017-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8435 ) Change subject: KUDU-1809: Add batch size configuration for ScanToken API .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1411: Implement HT timestamp propagation in KuduScanToken

2017-11-06 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8452 to look at the new patch set (#2). Change subject: KUDU-1411: Implement HT timestamp propagation in KuduScanToken

[kudu-CR] KUDU-1411: Implement HT timestamp propagation in KuduScanToken

2017-11-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8452 ) Change subject: KUDU-1411: Implement HT timestamp propagation in KuduScanToken .. Patch Set 2: (1 comment)

[kudu-CR] tablet copy client-test: have servers use same number of dirs

2017-12-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8767 ) Change subject: tablet_copy_client-test: have servers use same number of dirs .. Patch Set 4: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-1704: add BOUNDED READ scan mode

2017-12-08 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8804 Change subject: KUDU-1704: add BOUNDED_READ scan mode .. KUDU-1704: add BOUNDED_READ scan mode This patch adds a new read mode BOUNDED_READ

[kudu-CR] catalog-manager: fix some duplicated error messages

2017-12-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8805 ) Change subject: catalog-manager: fix some duplicated error messages .. Patch Set 1: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-1704: add BOUNDED READ scan mode

2017-12-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add BOUNDED_READ scan mode .. Patch Set 1: The failed test ObjectIdGeneratorTest.TestCanoicalizeUuid seems to be

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

2017-10-25 Thread Hao Hao (Code Review)
Hao Hao 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] improve AsyncKuduScanner logging

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

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

2017-10-25 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8382 to look at the new patch set (#2). Change subject: [java client] improve AsyncKuduScanner logging

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

2017-10-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8382 ) Change subject: [java client] imporve AsyncKuduScanner logging .. Patch Set 1: The failed test should be unrelated. -- To view, visit

[kudu-CR] docs: update security doc on --redact flag usage

2017-10-25 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8394 Change subject: docs: update security doc on --redact flag usage .. docs: update security doc on --redact flag usage Change-Id:

[kudu-CR] docs: update security doc on --redact flag usage

2017-10-25 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8394 to look at the new patch set (#2). Change subject: docs: update security doc on --redact flag usage

[kudu-CR] docs: update security doc on --redact flag usage

2017-10-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8394 ) Change subject: docs: update security doc on --redact flag usage .. Patch Set 2: (1 comment)

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

2017-10-25 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8382 Change subject: [java client] imporve AsyncKuduScanner logging .. [java client] imporve AsyncKuduScanner logging As filed in KUDU-2199, in a

[kudu-CR] KUDU-2055 [part 5]: Coalesce hole punch for LBM

2017-10-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8162 ) Change subject: KUDU-2055 [part 5]: Coalesce hole punch for LBM .. Patch Set 11: (5 comments)

[kudu-CR] KUDU-2055 [part 5]: Coalesce hole punch for LBM

2017-10-24 Thread Hao Hao (Code Review)
Hello Tidy Bot, Dan Burkert, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8162 to look at the new patch set (#11). Change subject: KUDU-2055 [part 5]: Coalesce hole punch for LBM

[kudu-CR] log block manager: return early if read-only

2017-10-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: log block manager: return early if read-only .. Patch Set 5: Code-Review+1 -- To view, visit

[kudu-CR] log block manager: return early if read-only

2017-10-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: log block manager: return early if read-only .. Patch Set 4: (3 comments)

[kudu-CR] log block manager: return early if read-only

2017-10-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: log block manager: return early if read-only .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-1809: Add batch size configuration for ScanToken API

2017-10-31 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8435 Change subject: KUDU-1809: Add batch size configuration for ScanToken API .. KUDU-1809: Add batch size configuration for ScanToken API Both

[kudu-CR] Add a benchmark test for tablet deletion

2017-10-30 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8420 Change subject: Add a benchmark test for tablet deletion .. Add a benchmark test for tablet deletion Commit 15f3f9b2f optimizes the

[kudu-CR] Add a benchmark test for tablet deletion

2017-10-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8420 ) Change subject: Add a benchmark test for tablet deletion .. Patch Set 2: (10 comments)

[kudu-CR] Add a benchmark test for tablet deletion

2017-10-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8420 ) Change subject: Add a benchmark test for tablet deletion .. Patch Set 2: (2 comments)

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

2017-10-30 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8382 to look at the new patch set (#4). Change subject: [java client] improve AsyncKuduScanner logging

[kudu-CR] Add a benchmark test for tablet deletion

2017-10-30 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8420 to look at the new patch set (#2). Change subject: Add a benchmark test for tablet deletion .. Add a

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

2017-10-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8382 ) Change subject: [java client] improve AsyncKuduScanner logging .. Patch Set 3: (1 comment)

[kudu-CR] Add a benchmark test for tablet deletion

2017-10-30 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8420 to look at the new patch set (#3). Change subject: Add a benchmark test for tablet deletion .. Add a

[kudu-CR] Add a table renaming tool

2018-05-08 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10337 to look at the new patch set (#3). Change subject: Add a table renaming tool .. Add a

[kudu-CR] Add a table renaming tool

2018-05-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10337 ) Change subject: Add a table renaming tool .. Patch Set 2: (3 comments)

[kudu-CR] Add a table renaming tool

2018-05-08 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10337 to look at the new patch set (#2). Change subject: Add a table renaming tool .. Add a

[kudu-CR] KUDU-2191 (11/n): remove fallback logic from HmsCatalog

2018-05-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10380 ) Change subject: KUDU-2191 (11/n): remove fallback logic from HmsCatalog .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2191 (11/n): remove fallback logic from HmsCatalog

2018-05-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10380 ) Change subject: KUDU-2191 (11/n): remove fallback logic from HmsCatalog .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-2191 (11/n): remove fallback logic from HmsCatalog

2018-05-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10380 ) Change subject: KUDU-2191 (11/n): remove fallback logic from HmsCatalog .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2191 (11/n): remove fallback logic from HmsCatalog

2018-05-14 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10380 ) Change subject: KUDU-2191 (11/n): remove fallback logic from HmsCatalog .. KUDU-2191 (11/n): remove fallback logic from HmsCatalog

[kudu-CR] KUDU-2191:bump Hive version

2018-05-14 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10403 Change subject: KUDU-2191:bump Hive version .. KUDU-2191:bump Hive version Bumps the Hive version to 2.3.3 to include the important fix for

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 10: > > > Uploaded patch set 10. > > > > I think the test

[kudu-CR] Add a table renaming tool

2018-05-07 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10337 Change subject: Add a table renaming tool .. Add a table renaming tool This commit introduces a tool to rename a table. When HMS integration

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-07 Thread Hao Hao (Code Review)
Hello Alexey Serbin, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9526 to look at the new patch set (#10). Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 10: (16 comments)

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 10: (8 comments)

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-08 Thread Hao Hao (Code Review)
Hello Alexey Serbin, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9526 to look at the new patch set (#11). Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-08 Thread Hao Hao (Code Review)
Hello Alexey Serbin, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9526 to look at the new patch set (#12). Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode

[kudu-CR] Add a table renaming tool

2018-05-09 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10337 ) Change subject: Add a table renaming tool .. Add a table renaming tool This commit introduces a tool to rename a table. When HMS

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 13: (8 comments)

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-09 Thread Hao Hao (Code Review)
Hello Alexey Serbin, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9526 to look at the new patch set (#13). Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 13: Verified+1 Unrelated flaky test. -- To view, visit

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-09 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/9526 To unsubscribe,

[kudu-CR] Add a column renaming tool

2018-05-10 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10368 Change subject: Add a column renaming tool .. Add a column renaming tool This commit introduces a tool to rename a table's column. Similar

[kudu-CR] Add a column renaming tool

2018-05-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10368 ) Change subject: Add a column renaming tool .. Patch Set 1: Verified+1 Unrelated flaky test. -- To view, visit

[kudu-CR] Add a column renaming tool

2018-05-10 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: Add a column renaming tool .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/10368 To unsubscribe, visit

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 13: > > Unrelated flaky test. > > Did you try to run it

[kudu-CR] Add a column renaming tool

2018-05-10 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10368 ) Change subject: Add a column renaming tool .. Add a column renaming tool This commit introduces a tool to rename a table's column.

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 13: > > Patch Set 13: Code-Review+2 > > Thank you very

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 14: (1 comment)

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-11 Thread Hao Hao (Code Review)
Hello Alexey Serbin, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9526 to look at the new patch set (#14). Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode

[kudu-CR] KUDU-2191: Metadata Upgrade Tool

2018-05-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10075 ) Change subject: KUDU-2191: Metadata Upgrade Tool .. Patch Set 13: (10 comments)

[kudu-CR] KUDU-2191: Metadata Upgrade Tool

2018-05-10 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10075 to look at the new patch set (#13). Change subject: KUDU-2191: Metadata Upgrade Tool

[kudu-CR] Rename Hive MetaStore to Metastore

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10379 ) Change subject: Rename Hive MetaStore to Metastore .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/10379

[kudu-CR] KUDU-2191 (11/n): remove fallback logic from HmsCatalog

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10380 ) Change subject: KUDU-2191 (11/n): remove fallback logic from HmsCatalog .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2191 (11/n): remove fallback logic from HmsCatalog

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10380 ) Change subject: KUDU-2191 (11/n): remove fallback logic from HmsCatalog .. Patch Set 1: Also, the failed test cases seem to be related.

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 14: Verified+1 Unrelated flaky test. -- To view, visit

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/9526 To unsubscribe,

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. A new Jepsen checker for READ_YOUR_WRITES scan mode This

[kudu-CR] catalog-manager: refactor error handling

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10376 ) Change subject: catalog-manager: refactor error handling .. Patch Set 1: Code-Review+1 (2 comments) LGTM, just some minor nits.

[kudu-CR] catalog-manager: split AlterTable and DeleteTable

2018-05-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10378 ) Change subject: catalog-manager: split AlterTable and DeleteTable .. Patch Set 1: Code-Review+1 -- To view, visit

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