[kudu-CR] external mini cluster: bump start process timeout a bit more

2019-05-30 Thread Adar Dembo (Code Review)
Hello Will Berkeley, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13484 to review the following change. Change subject: external_mini_cluster: bump start process timeout a bit more .. ext

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12468 ) Change subject: KUDU-2514 Part 1: Support extra config for table. .. Patch Set 11: (18 comments) http://gerrit.cloudera.org:8080/#/c/124

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 1)

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13266 ) Change subject: KUDU-2791: TTL cache in DNS resolver (part 1) .. KUDU-2791: TTL cache in DNS resolver (part 1) Added TTL cache

[kudu-CR] [util] extract libnvm cache from libkudu util

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13468 ) Change subject: [util] extract libnvm_cache from libkudu_util .. [util] extract libnvm_cache from libkudu_util This patch extr

[kudu-CR] [util] extract libnvm cache from libkudu util

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13468 ) Change subject: [util] extract libnvm_cache from libkudu_util .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 1)

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13266 ) Change subject: KUDU-2791: TTL cache in DNS resolver (part 1) .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org

[kudu-CR] WIP: [KUDU-442] Add Kudu support for Hive

2019-05-30 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13483 Change subject: WIP: [KUDU-442] Add Kudu support for Hive .. WIP: [KUDU-442] Add Kudu support for Hive Adds initial support for a Kudu Hi

[kudu-CR] [tablet] Support accurate count of rows

2019-05-30 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13456 to look at the new patch set (#4). Change subject: [tablet] Support accurate count of rows .. [tablet]

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 1)

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13266 ) Change subject: KUDU-2791: TTL cache in DNS resolver (part 1) .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/13266/6

[kudu-CR] [util] extract libnvm cache from libkudu util

2019-05-30 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13468 to look at the new patch set (#3). Change subject: [util] extract libnvm_cache from libkudu_util .. [ut

[kudu-CR] [util] extract libnvm cache from libkudu util

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13468 ) Change subject: [util] extract libnvm_cache from libkudu_util .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/13468/2

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 1)

2019-05-30 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13266 to look at the new patch set (#7). Change subject: KUDU-2791: TTL cache in DNS resolver (part 1) ..

[kudu-CR] docs: update docs for heap sampling

2019-05-30 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13459 ) Change subject: docs: update docs for heap sampling .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13459/1/docs/troubl

[kudu-CR] KUDU-2398: Kudu kubernetes statefulset manifest

2019-05-30 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13407 ) Change subject: KUDU-2398: Kudu kubernetes statefulset manifest .. Patch Set 8: Code-Review+2 I think this is a good start. It will sure

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 2)

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13469 ) Change subject: KUDU-2791: TTL cache in DNS resolver (part 2) .. Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/13469/2/s

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 1)

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13266 ) Change subject: KUDU-2791: TTL cache in DNS resolver (part 1) .. Patch Set 6: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/

[kudu-CR] [util] extract libkudu nvm cache from libkudu util

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13468 ) Change subject: [util] extract libkudu_nvm_cache from libkudu_util .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/13468

[kudu-CR] metrics-test: fix wrong metric name and improve metric filtering test

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13471 ) Change subject: metrics-test: fix wrong metric name and improve metric filtering test .. Patch Set 1: (4 comments) http://gerrit.cloude

[kudu-CR] [tserver] Move cell size checking to prepare phase

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13470 ) Change subject: [tserver] Move cell size checking to prepare phase .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/1347

[kudu-CR] [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData

2019-05-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13477 ) Change subject: [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData .. Patch Set 2: Code-Review+2 (1 co

[kudu-CR] [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData

2019-05-30 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13477 ) Change subject: [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData .. Patch Set 2: Code-Review+1 --

[kudu-CR] [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData

2019-05-30 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13477 to look at the new patch set (#2). Change subject: [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData ..

[kudu-CR] [tserver] Move cell size checking to prepare phase

2019-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13470 ) Change subject: [tserver] Move cell size checking to prepare phase .. Patch Set 2: Does this change user-facing behavior? Previously, if

[kudu-CR] hms: have tools ignore other clusters

2019-05-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13411 ) Change subject: hms: have tools ignore other clusters .. Patch Set 5: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/13411/4

[kudu-CR] hms: have tools ignore other clusters

2019-05-30 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13411 ) Change subject: hms: have tools ignore other clusters .. hms: have tools ignore other clusters Currently, the HMS tooling will check

[kudu-CR] [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData

2019-05-30 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13477 ) Change subject: [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData .. Patch Set 1: Code-Review+1 --

[kudu-CR] [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData

2019-05-30 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13477 Change subject: [spark tools] KUDU-2831: Deflake DistributedDataGeneratorTest.testGenerateRandomData .. [spark tools] KUDU-2831: Deflak

[kudu-CR] [backup] KUDU-2786 Parallelize tables for backup and restore

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13430 ) Change subject: [backup] KUDU-2786 Parallelize tables for backup and restore .. [backup] KUDU-2786 Parallelize tables for backup a

[kudu-CR] [backup] KUDU-2786 Parallelize tables for backup and restore

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13430 ) Change subject: [backup] KUDU-2786 Parallelize tables for backup and restore .. Patch Set 2: Verified+1 Overriding flaky test -- To vie

[kudu-CR] [backup] KUDU-2786 Parallelize tables for backup and restore

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has removed a vote on this change. Change subject: [backup] KUDU-2786 Parallelize tables for backup and restore .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13430 To un

[kudu-CR] [backup] KUDU-2786 Parallelize tables for backup and restore

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13430 ) Change subject: [backup] KUDU-2786 Parallelize tables for backup and restore .. Patch Set 2: Code-Review+2 -- To view, visit http://gerr

[kudu-CR] [backup] KUDU-2786 Parallelize tables for backup and restore

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13430 ) Change subject: [backup] KUDU-2786 Parallelize tables for backup and restore .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080

[kudu-CR] [backup] KUDU-2787 Pt 2 Allow single table failures for restore

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13416 ) Change subject: [backup] KUDU-2787 Pt 2 Allow single table failures for restore .. Patch Set 3: Verified+1 Code-Review+2 -- To view, vis

[kudu-CR] [backup] KUDU-2787 Pt 2 Allow single table failures for restore

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13416 ) Change subject: [backup] KUDU-2787 Pt 2 Allow single table failures for restore .. [backup] KUDU-2787 Pt 2 Allow single table fail

[kudu-CR] [backup] KUDU-2787 Pt 2 Allow single table failures for restore

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has removed a vote on this change. Change subject: [backup] KUDU-2787 Pt 2 Allow single table failures for restore .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13416 To

[kudu-CR] [backup] KUDU-2787 Allow single table failures for backup

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13415 ) Change subject: [backup] KUDU-2787 Allow single table failures for backup .. [backup] KUDU-2787 Allow single table failures for ba

[kudu-CR] [backup] KUDU-2787 Allow single table failures for backup

2019-05-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13415 ) Change subject: [backup] KUDU-2787 Allow single table failures for backup .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.

[kudu-CR] [backup] KUDU-2787 Allow single table failures for backup

2019-05-30 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13415 ) Change subject: [backup] KUDU-2787 Allow single table failures for backup .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit

[kudu-CR] [java] Deflake TestAuthTokenReacquire.testBasicMasterOperations

2019-05-30 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13461 ) Change subject: [java] Deflake TestAuthTokenReacquire.testBasicMasterOperations .. [java] Deflake TestAuthTokenReacquire.testBa

[kudu-CR] [tablet] Support accurate count of rows

2019-05-30 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13456 to look at the new patch set (#3). Change subject: [tablet] Support accurate count of rows .. [tablet]

[kudu-CR] [tserver] Move cell size checking to prepare phase

2019-05-30 Thread Yingchun Lai (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13470 to look at the new patch set (#2). Change subject: [tserver] Move cell size checking to prepare phase .

[kudu-CR] metrics-test: fix wrong metric name and improve metric filtering test

2019-05-30 Thread Yifan Zhang (Code Review)
Yifan Zhang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13471 Change subject: metrics-test: fix wrong metric name and improve metric filtering test .. metrics-test: fix wrong metric name and improve

[kudu-CR] [java] Deflake TestAuthTokenReacquire.testBasicMasterOperations

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13461 ) Change subject: [java] Deflake TestAuthTokenReacquire.testBasicMasterOperations .. Patch Set 3: Code-Review+2 -- To view, visit http:

[kudu-CR] KUDU-2348: Pick a random replica in RemoteTablet.java

2019-05-30 Thread Yifan Zhang (Code Review)
Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12158 ) Change subject: KUDU-2348: Pick a random replica in RemoteTablet.java .. Patch Set 9: > Thinking about this a bit, I wonder what behavio

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 1)

2019-05-30 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13266 to look at the new patch set (#6). Change subject: KUDU-2791: TTL cache in DNS resolver (part 1) ..

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 2)

2019-05-30 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13469 to look at the new patch set (#2). Change subject: KUDU-2791: TTL cache in DNS resolver (part 2) .. KUD

[kudu-CR] [util] extract libkudu nvm cache from libkudu util

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13468 ) Change subject: [util] extract libkudu_nvm_cache from libkudu_util .. Patch Set 1: Verified+1 unrelated flakes in: * mvcc-test.0 *

[kudu-CR] [util] extract libkudu nvm cache from libkudu util

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13468 ) Change subject: [util] extract libkudu_nvm_cache from libkudu_util .. Removed reviewer Kudu Jenkins with the following votes:

[kudu-CR] [tserver] Move cell size checking to prepare phase

2019-05-30 Thread Yingchun Lai (Code Review)
Yingchun Lai has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13470 Change subject: [tserver] Move cell size checking to prepare phase .. [tserver] Move cell size checking to prepare phase It's inefficien

[kudu-CR] [java] Deflake TestSecurity.testImportExportAuthenticationCredentials

2019-05-30 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13466 ) Change subject: [java] Deflake TestSecurity.testImportExportAuthenticationCredentials .. [java] Deflake TestSecurity.testImpor

[kudu-CR] [java] Deflake TestSecurity.testImportExportAuthenticationCredentials

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13466 ) Change subject: [java] Deflake TestSecurity.testImportExportAuthenticationCredentials .. Patch Set 1: Code-Review+2 -- To view, visi

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 1)

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13266 ) Change subject: KUDU-2791: TTL cache in DNS resolver (part 1) .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/13266/

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 1)

2019-05-30 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13266 to look at the new patch set (#5). Change subject: KUDU-2791: TTL cache in DNS resolver (part 1) ..

[kudu-CR] [java] Deflake TestSecurity.testImportExportAuthenticationCredentials

2019-05-30 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/13466 ) Change subject: [java] Deflake TestSecurity.testImportExportAuthenticationCredentials .. Patch Set 1: Verified+1 Flakes -.- -- To v

[kudu-CR] [java] Deflake TestSecurity.testImportExportAuthenticationCredentials

2019-05-30 Thread Will Berkeley (Code Review)
Will Berkeley has removed a vote on this change. Change subject: [java] Deflake TestSecurity.testImportExportAuthenticationCredentials .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:808

[kudu-CR] [util] extract libkudu nvm cache from libkudu util

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13468 Change subject: [util] extract libkudu_nvm_cache from libkudu_util .. [util] extract libkudu_nvm_cache from libkudu_util This patch ext

[kudu-CR] KUDU-2791: TTL cache in DNS resolver (part 2)

2019-05-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13469 Change subject: KUDU-2791: TTL cache in DNS resolver (part 2) .. KUDU-2791: TTL cache in DNS resolver (part 2) This changelist adds nec