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

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9050 ) Change subject: KUDU-2264. java: automatically attempt to re-acquire Kerberos credentials .. Patch Set 5: Code-Review+2 -- To view,

[kudu-CR] KUDU-2191 (6/n): Hive Metastore catalog manager integration

2018-03-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8312 ) Change subject: KUDU-2191 (6/n): Hive Metastore catalog manager integration .. Patch Set 7: (7 comments)

[kudu-CR] iwyu: workaround missing IWYU results when using Ninja

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9554 ) Change subject: iwyu: workaround missing IWYU results when using Ninja .. Patch Set 1: (1 comment)

[kudu-CR] Remove namespace hack in hms client.h

2018-03-07 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9553 ) Change subject: Remove namespace hack in hms_client.h .. Remove namespace hack in hms_client.h To avoid having to use the very

[kudu-CR] iwyu: workaround missing IWYU results when using Ninja

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9554 ) Change subject: iwyu: workaround missing IWYU results when using Ninja .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.6.x) [tablet] fix nullptr dereference while capturing iterators

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9549 ) Change subject: [tablet] fix nullptr dereference while capturing iterators .. Patch Set 2: Verified+1 Flakes in

[kudu-CR](branch-1.6.x) [tablet] fix nullptr dereference while capturing iterators

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/9549 ) Change subject: [tablet] fix nullptr dereference while capturing iterators .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] iwyu: workaround missing IWYU results when using Ninja

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9554 Change subject: iwyu: workaround missing IWYU results when using Ninja .. iwyu: workaround missing IWYU results when using Ninja This

[kudu-CR] KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow

2018-03-07 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Adar Dembo, Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9375 to look at the new patch set (#6). Change subject: KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow

[kudu-CR] KUDU-2297 (part 6): enable periodic stack sampling by default, avoid bias

2018-03-07 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9523 to look at the new patch set (#3). Change subject: KUDU-2297 (part 6): enable periodic stack sampling by default, avoid bias

[kudu-CR] KUDU-2297 (part 6): enable periodic stack sampling by default, avoid bias

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9523 ) Change subject: KUDU-2297 (part 6): enable periodic stack sampling by default, avoid bias .. Patch Set 2: (8 comments)

[kudu-CR] [WIP] KUDU-2289: Tablet deletion should be throttled

2018-03-07 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9551 to look at the new patch set (#2). Change subject: [WIP] KUDU-2289: Tablet deletion should be throttled .. [WIP]

[kudu-CR] Remove namespace hack in hms client.h

2018-03-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9553 ) Change subject: Remove namespace hack in hms_client.h .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9375 ) Change subject: KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow .. Patch Set 5: (2 comments)

[kudu-CR] Remove namespace hack in hms client.h

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9553 ) Change subject: Remove namespace hack in hms_client.h .. Patch Set 1: Code-Review+2 (1 comment)

[kudu-CR] Remove namespace hack in hms client.h

2018-03-07 Thread Dan Burkert (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9553 to review the following change. Change subject: Remove namespace hack in hms_client.h .. Remove namespace hack in

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

2018-03-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9050 ) Change subject: KUDU-2264. java: automatically attempt to re-acquire Kerberos credentials .. Patch Set 5: Code-Review+1 (1 comment)

[kudu-CR] KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9375 ) Change subject: KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow .. Patch Set 5: (7 comments)

[kudu-CR] docs: update scaling guide with new thread information

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9552 ) Change subject: docs: update scaling guide with new thread information .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2191 (6/n): Hive Metastore catalog manager integration

2018-03-07 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8312 to look at the new patch set (#6). Change subject: KUDU-2191 (6/n): Hive Metastore catalog manager integration

[kudu-CR] docs: update scaling guide with new thread information

2018-03-07 Thread Adar Dembo (Code Review)
Hello Alex Rodoni, Jean-Daniel Cryans, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9552 to review the following change. Change subject: docs: update scaling guide with new thread information

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

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9050 ) Change subject: KUDU-2264. java: automatically attempt to re-acquire Kerberos credentials .. Patch Set 4: (11 comments)

[kudu-CR] env: generalize resource limits and add RLIMIT NPROC support

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9521 ) Change subject: env: generalize resource limits and add RLIMIT_NPROC support .. env: generalize resource limits and add

[kudu-CR] KUDU-1913: cap number of threads on server-wide pools

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9522 ) Change subject: KUDU-1913: cap number of threads on server-wide pools .. KUDU-1913: cap number of threads on server-wide pools

[kudu-CR] KUDU-1913: cap number of threads on server-wide pools

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9522 ) Change subject: KUDU-1913: cap number of threads on server-wide pools .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR](branch-1.6.x) KUDU-2274. RaftConsensus should not access cmeta when shutdown

2018-03-07 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9548 to look at the new patch set (#2). Change subject: KUDU-2274. RaftConsensus should not access cmeta when shutdown

[kudu-CR] WIP: exported kudu client static archives

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has abandoned this change. ( http://gerrit.cloudera.org:8080/9479 ) Change subject: WIP: exported kudu client static archives .. Abandoned Still not something we actually want to do. -- To view, visit

[kudu-CR] [WIP] KUDU-2289: Tablet deletion should be throttled

2018-03-07 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9551 Change subject: [WIP] KUDU-2289: Tablet deletion should be throttled .. [WIP] KUDU-2289: Tablet deletion should be throttled When a

[kudu-CR](branch-1.6.x) KUDU-2274. RaftConsensus should not access cmeta when shutdown

2018-03-07 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9548 to review the following change. Change subject: KUDU-2274. RaftConsensus should not access cmeta when shutdown ..

[kudu-CR](branch-1.6.x) [tablet service] remove useless call

2018-03-07 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9550 to review the following change. Change subject: [tablet_service] remove useless call .. [tablet_service]

[kudu-CR](branch-1.6.x) [tablet] fix nullptr dereference while capturing iterators

2018-03-07 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9549 to review the following change. Change subject: [tablet] fix nullptr dereference while capturing iterators

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

2018-03-07 Thread Hao Hao (Code Review)
Hello Dan Burkert, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8847 to look at the new patch set (#9). Change subject: KUDU-1704: add java client support for READ_YOUR_WRITES mode

[kudu-CR] KUDU-2191 (6/n): Hive Metastore catalog manager integration

2018-03-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8312 ) Change subject: KUDU-2191 (6/n): Hive Metastore catalog manager integration .. Patch Set 4: (7 comments)

[kudu-CR] KUDU-2191 (6/n): Hive Metastore catalog manager integration

2018-03-07 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8312 to look at the new patch set (#5). Change subject: KUDU-2191 (6/n): Hive Metastore catalog manager integration

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9541 ) Change subject: Add CatalogManager::master_consensus() accessor .. Add CatalogManager::master_consensus() accessor A patch I'm

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9541 ) Change subject: Add CatalogManager::master_consensus() accessor .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9541 ) Change subject: Add CatalogManager::master_consensus() accessor .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9541 ) Change subject: Add CatalogManager::master_consensus() accessor .. Patch Set 2: (1 comment)

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Dan Burkert (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9541 to look at the new patch set (#3). Change subject: Add CatalogManager::master_consensus() accessor

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9541 ) Change subject: Add CatalogManager::master_consensus() accessor .. Patch Set 2: (1 comment)

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Dan Burkert (Code Review)
Hello Will Berkeley, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9541 to look at the new patch set (#2). Change subject: Add CatalogManager::master_consensus() accessor

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9541 ) Change subject: Add CatalogManager::master_consensus() accessor .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [doc] Document the new decimal column type

2018-03-07 Thread Grant Henke (Code Review)
Hello Alex Rodoni, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9432 to look at the new patch set (#3). Change subject: [doc] Document the new decimal column type

[kudu-CR] [doc] Document the new decimal column type

2018-03-07 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9432 ) Change subject: [doc] Document the new decimal column type .. Patch Set 2: (11 comments)

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9541 ) Change subject: Add CatalogManager::master_consensus() accessor .. Patch Set 1: Code-Review+2 Maybe Will wants to take a look too. --

[kudu-CR] Add CatalogManager::master consensus() accessor

2018-03-07 Thread Dan Burkert (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9541 to review the following change. Change subject: Add CatalogManager::master_consensus() accessor .. Add

[kudu-CR] KUDU-2129 make ksck less scary when copying

2018-03-07 Thread Andrew Wong (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9528 to look at the new patch set (#3). Change subject: KUDU-2129 make ksck less scary when copying ..

[kudu-CR] KUDU-2129 make ksck less scary when copying

2018-03-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9528 ) Change subject: KUDU-2129 make ksck less scary when copying .. Patch Set 2: (6 comments)

[kudu-CR] KUDU-2129 make ksck less scary when copying

2018-03-07 Thread Andrew Wong (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9528 to look at the new patch set (#2). Change subject: KUDU-2129 make ksck less scary when copying ..

[kudu-CR] env: generalize resource limits and add RLIMIT NPROC support

2018-03-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9521 ) Change subject: env: generalize resource limits and add RLIMIT_NPROC support .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-1913: cap number of threads on server-wide pools

2018-03-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9522 ) Change subject: KUDU-1913: cap number of threads on server-wide pools .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow

2018-03-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/9375 ) Change subject: KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-1913: cap number of threads on server-wide pools

2018-03-07 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9522 to look at the new patch set (#2). Change subject: KUDU-1913: cap number of threads on server-wide pools

[kudu-CR] env: generalize resource limits and add RLIMIT NPROC support

2018-03-07 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9521 to look at the new patch set (#2). Change subject: env: generalize resource limits and add RLIMIT_NPROC support

[kudu-CR] KUDU-1913: cap number of threads on server-wide pools

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9522 ) Change subject: KUDU-1913: cap number of threads on server-wide pools .. Patch Set 1: (1 comment)

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

2018-03-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9050 ) Change subject: KUDU-2264. java: automatically attempt to re-acquire Kerberos credentials .. Patch Set 4: (11 comments)

[kudu-CR] env: generalize resource limits and add RLIMIT NPROC support

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9521 ) Change subject: env: generalize resource limits and add RLIMIT_NPROC support .. Patch Set 1: (2 comments)

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

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9050 ) Change subject: KUDU-2264. java: automatically attempt to re-acquire Kerberos credentials .. Patch Set 3: (8 comments)

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

2018-03-07 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Hao Hao, Anonymous Coward #380, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9050 to look at the new patch set (#4). Change subject: KUDU-2264. java: automatically attempt

[kudu-CR] KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9375 ) Change subject: KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow

2018-03-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/9375 ) Change subject: KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow .. Patch Set 5: (2 comments)

[kudu-CR] KUDU-2297 (part 6): enable periodic stack sampling by default, avoid bias

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9523 ) Change subject: KUDU-2297 (part 6): enable periodic stack sampling by default, avoid bias .. Patch Set 2: (8 comments)

[kudu-CR] Fix RUN FLAKY ONLY build

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9533 ) Change subject: Fix RUN_FLAKY_ONLY build .. Fix RUN_FLAKY_ONLY build Now that our tests have shard numbers appended (foo-test.)

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

2018-03-07 Thread Dan Burkert (Code Review)
Dan Burkert 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 comment)

[kudu-CR] Fix RUN FLAKY ONLY build

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9533 ) Change subject: Fix RUN_FLAKY_ONLY build .. Patch Set 1: Code-Review+2 (1 comment) It's perl, so automatic +2.

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

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon 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: (2

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

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin 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: (3

[kudu-CR] Fix RUN FLAKY ONLY build

2018-03-07 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9533 to review the following change. Change subject: Fix RUN_FLAKY_ONLY build .. Fix RUN_FLAKY_ONLY build Now that our

[kudu-CR] KUDU-721: [Python] Add DECIMAL column type support

2018-03-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9496 ) Change subject: KUDU-721: [Python] Add DECIMAL column type support .. KUDU-721: [Python] Add DECIMAL column type support

[kudu-CR] KUDU-721: [Python] Add DECIMAL column type support

2018-03-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9496 ) Change subject: KUDU-721: [Python] Add DECIMAL column type support .. Patch Set 4: merging this. we can follow up if we missed

[kudu-CR] KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9375 ) Change subject: KUDU-2297 (part 5): dump stacks into diagnostics log on service queue overflow .. Patch Set 5: (6 comments)

[kudu-CR] build: don't name unsharded tests with a shard suffix

2018-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9524 ) Change subject: build: don't name unsharded tests with a shard suffix .. Patch Set 1: (1 comment) No need to purge the flaky test DB

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

2018-03-07 Thread Dan Burkert (Code Review)
Dan Burkert 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: (8

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

2018-03-07 Thread Anonymous Coward (Code Review)
Anonymous Coward #380 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:

[kudu-CR] [WIP] updated to use READ YOUR WRITES scan mode for Jepsen tests

2018-03-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: [WIP] updated to use READ_YOUR_WRITES scan mode for Jepsen tests .. Patch Set 1: (4 comments) > (5 comments) > > we'll

[kudu-CR] build: don't name unsharded tests with a shard suffix

2018-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9524 ) Change subject: build: don't name unsharded tests with a shard suffix .. Patch Set 1: (1 comment) Thanks. Does it make sense to also

[kudu-CR] WIP KUDU-2129 make ksck less scary when copying

2018-03-07 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9528 ) Change subject: WIP KUDU-2129 make ksck less scary when copying .. Patch Set 1: (6 comments)

[kudu-CR] [tools] KUDU-2331: Allow move tool to work when uninvolved tserver is down

2018-03-07 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9510 ) Change subject: [tools] KUDU-2331: Allow move tool to work when uninvolved tserver is down .. Patch Set 3: (3 comments)

[kudu-CR] [tools] KUDU-2331: Allow move tool to work when uninvolved tserver is down

2018-03-07 Thread Will Berkeley (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9510 to look at the new patch set (#4). Change subject: [tools] KUDU-2331: Allow move tool to work when uninvolved tserver is down

[kudu-CR] [tools] KUDU-2331: Allow move tool to work when uninvolved tserver is down

2018-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9510 ) Change subject: [tools] KUDU-2331: Allow move tool to work when uninvolved tserver is down .. Patch Set 3: (3 comments)