[kudu-CR] [java] Upgrade to the latest gradle version

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8331 ) Change subject: [java] Upgrade to the latest gradle version .. Patch Set 2: Verified+1 Overriding Jenkins, the failure

[kudu-CR] [java] Convert Gradle to new buildSrc style layout

Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8332 ) Change subject: [java] Convert Gradle to new buildSrc style layout .. [java] Convert Gradle to new buildSrc style layout

[kudu-CR] [java] Upgrade to the latest gradle version

Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/8331 ) Change subject: [java] Upgrade to the latest gradle version .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] [java] Upgrade to the latest gradle version

Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8331 ) Change subject: [java] Upgrade to the latest gradle version .. [java] Upgrade to the latest gradle version Change-Id:

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

Hello Tidy Bot, Dan Burkert, Kudu Jenkins, 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 (#10). Change subject: KUDU-2055 [part 5]: Coalesce hole punch for LBM

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8325 to look at the new patch set (#2). Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Patch Set 2: (2 comments)

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

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: WIP log block manager: return early if read-only .. Patch Set 1: (10 comments)

[kudu-CR] tidy: fix some misc tidy warnings in various files

Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8293 ) Change subject: tidy: fix some misc tidy warnings in various files .. tidy: fix some misc tidy warnings in various files These

[kudu-CR] tidy: fix some misc tidy warnings in various files

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8293 ) Change subject: tidy: fix some misc tidy warnings in various files .. Patch Set 3: Code-Review+2 just a rebase -- To view, visit

[kudu-CR] tool: speed up test runtime a bit

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8337 ) Change subject: tool: speed up test runtime a bit .. Patch Set 2: (1 comment)

[kudu-CR] tool: speed up test runtime a bit

Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8337 to look at the new patch set (#2). Change subject: tool: speed up test runtime a bit .. tool: speed

[kudu-CR] fs: adjust directory sync behavior

Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8290 ) Change subject: fs: adjust directory sync behavior .. fs: adjust directory sync behavior This patch centralizes the

[kudu-CR] fs: adjust directory sync behavior

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8290 ) Change subject: fs: adjust directory sync behavior .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8290

[kudu-CR] tool: speed up test runtime a bit

Hello Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8337 to review the following change. Change subject: tool: speed up test runtime a bit .. tool: speed up test runtime a

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

Adar Dembo 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 10: Code-Review+2 I wonder if Dan wants to take another

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

Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8338 Change subject: WIP log block manager: return early if read-only .. WIP log block manager: return early if read-only WIP: may add

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Patch Set 2: (1 comment)

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Patch Set 2: Code-Review+2 (1 comment)

[kudu-CR] tool: speed up test runtime a bit

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8337 ) Change subject: tool: speed up test runtime a bit .. Patch Set 1: Code-Review+2 (1 comment)

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

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 10: (4 comments)

[kudu-CR] [java] Upgrade to the latest gradle version

Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8331 Change subject: [java] Upgrade to the latest gradle version .. [java] Upgrade to the latest gradle version Change-Id:

[kudu-CR] KUDU-2170 Masters can start with duplicates specified

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8341 ) Change subject: KUDU-2170 Masters can start with duplicates specified .. Patch Set 1: (5 comments)

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

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: WIP log block manager: return early if read-only .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2193 (part 1): switch to a waiting mutex in TSTabletManager

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8345 ) Change subject: KUDU-2193 (part 1): switch to a waiting mutex in TSTabletManager .. Patch Set 1: If we can guarantee that no other locks

[kudu-CR] KUDU-2170 Masters can start with duplicates specified

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8341 ) Change subject: KUDU-2170 Masters can start with duplicates specified .. Patch Set 2: Code-Review+2 -- To view, visit

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

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: WIP log block manager: return early if read-only .. Patch Set 1: (5 comments) Overall, the change makes sense to me. Thanks

[kudu-CR] KUDU-2193 (part 2): avoid holding TSTabletManager::lock for a long time

Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8346 to review the following change. Change subject: KUDU-2193 (part 2): avoid holding TSTabletManager::lock_ for a long time

[kudu-CR] KUDU-2193 (part 1): switch to a waiting mutex in TSTabletManager

Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8345 to review the following change. Change subject: KUDU-2193 (part 1): switch to a waiting mutex in TSTabletManager

[kudu-CR] KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable

Hello Tidy Bot, Mike Percy, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8245 to look at the new patch set (#4). Change subject: KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable

[kudu-CR] KUDU-2170 Masters can start with duplicates specified

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8341 ) Change subject: KUDU-2170 Masters can start with duplicates specified .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Patch Set 3: Verified+1 Agreed. -- To view, visit

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] tool: speed up test runtime a bit

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8337 ) Change subject: tool: speed up test runtime a bit .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] tool: speed up test runtime a bit

Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8337 ) Change subject: tool: speed up test runtime a bit .. tool: speed up test runtime a bit We're good about setting --never_fsync

[kudu-CR] KUDU-2194: use a common trace metric prefix to avoid overflowing intern map

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8340 ) Change subject: KUDU-2194: use a common trace metric prefix to avoid overflowing intern map .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Patch Set 3: (2 comments)

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

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: WIP log block manager: return early if read-only .. Patch Set 1: (3 comments)

[kudu-CR] KUDU-2170 Masters can start with duplicates specified

Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8341 to look at the new patch set (#2). Change subject: KUDU-2170 Masters can start with duplicates specified

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

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (1 comment)

[kudu-CR] KUDU-2193 (part 2): avoid holding TSTabletManager::lock for a long time

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8346 ) Change subject: KUDU-2193 (part 2): avoid holding TSTabletManager::lock_ for a long time .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR] [java] Convert Gradle to new buildSrc style layout

Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8332 Change subject: [java] Convert Gradle to new buildSrc style layout .. [java] Convert Gradle to new buildSrc style layout Currently we use

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

Hello Dan Burkert, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8338 to look at the new patch set (#2). Change subject: WIP log block manager: return early if read-only

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

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: WIP log block manager: return early if read-only .. Patch Set 1: (9 comments)

[kudu-CR] [webui] Add templates for tserver webui

Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8307 to look at the new patch set (#6). Change subject: [webui] Add templates for tserver webui .. [webui] Add

[kudu-CR] [java] Upgrade to the latest gradle version

Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/8331 ) Change subject: [java] Upgrade to the latest gradle version .. Patch Set 1: (1 comment)

[kudu-CR] [java] Upgrade to the latest gradle version

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8331 ) Change subject: [java] Upgrade to the latest gradle version .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [java] Upgrade to the latest gradle version

Hello Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8331 to look at the new patch set (#2). Change subject: [java] Upgrade to the latest gradle version

[kudu-CR] [java] Ensure a licence is added to the wrapper

Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8333 Change subject: [java] Ensure a licence is added to the wrapper .. [java] Ensure a licence is added to the wrapper When generating a

[kudu-CR] WIP: KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8245 ) Change subject: WIP: KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable .. Patch Set 1: (1 comment)

[kudu-CR] [java] Upgrade to the latest gradle version

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8331 ) Change subject: [java] Upgrade to the latest gradle version .. Patch Set 1: (1 comment)

[kudu-CR] [java] Convert Gradle to new buildSrc style layout

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8332 ) Change subject: [java] Convert Gradle to new buildSrc style layout .. Patch Set 1: Code-Review+2 -- To view, visit

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

Adar Dembo 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 10: Verified+1 -- To view, visit

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

Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/8162 ) Change subject: KUDU-2055 [part 5]: Coalesce hole punch for LBM .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] KUDU-2194: use a common trace metric prefix to avoid overflowing intern map

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8340 ) Change subject: KUDU-2194: use a common trace metric prefix to avoid overflowing intern map .. Patch Set 1: Code-Review+1 -- To view,

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

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: WIP log block manager: return early if read-only .. Patch Set 1: (1 comment)

[kudu-CR] [webui] Add templates for tserver webui

Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8307 to look at the new patch set (#5). Change subject: [webui] Add templates for tserver webui .. [webui] Add

[kudu-CR] KUDU-2170 Masters can start with duplicates specified

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/8341 ) Change subject: KUDU-2170 Masters can start with duplicates specified .. Patch Set 1: (5 comments)

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Patch Set 3: > Build Failed > >

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

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 10: The failed test hash_util-test.cc seems to be a setup issue

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

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 10: > I wonder if Dan wants to take another look, since this

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] KUDU-2193 (part 1): switch to a waiting mutex in TSTabletManager

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8345 ) Change subject: KUDU-2193 (part 1): switch to a waiting mutex in TSTabletManager .. Patch Set 1: > Patch Set 1: > > If we can guarantee

[kudu-CR] [java] Ensure a licence is added to the wrapper

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8333 ) Change subject: [java] Ensure a licence is added to the wrapper .. Patch Set 1: (1 comment)

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

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (3 comments)

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

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (4 comments)

[kudu-CR] KUDU-2194: use a common trace metric prefix to avoid overflowing intern map

Hello Andrew Wong, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8340 to review the following change. Change subject: KUDU-2194: use a common trace metric prefix to avoid overflowing intern map

[kudu-CR] KUDU-2170 Masters can start with duplicates specified

Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8341 Change subject: KUDU-2170 Masters can start with duplicates specified .. KUDU-2170 Masters can start with duplicates specified Users

[kudu-CR] KUDU-2194: use a common trace metric prefix to avoid overflowing intern map

Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8340 ) Change subject: KUDU-2194: use a common trace metric prefix to avoid overflowing intern map .. KUDU-2194: use a common trace

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

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: Code-Review+1 IWYU is confused. -- To view, visit

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

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (1 comment)

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

Dan Burkert has removed a vote on this change. Change subject: KUDU-2191 (2/n): Hive Metastore client .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/7053 To unsubscribe, visit

[kudu-CR] Integrate BlockCreationTransaction into MajorDeltaCompaction

Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8325 ) Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction .. Integrate BlockCreationTransaction into

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

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8338 ) Change subject: WIP log block manager: return early if read-only .. Patch Set 1: (5 comments)