[kudu-CR] WIP: clock: add a built-in NTP client implementation

Todd Lipcon has posted comments on this change. Change subject: WIP: clock: add a built-in NTP client implementation .. Patch Set 6: did a rebase and another rev with a few more docs but still not a high quality patch. needs

[kudu-CR] WIP: clock: add a built-in NTP client implementation

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7477 to look at the new patch set (#6). Change subject: WIP: clock: add a built-in NTP client implementation .. WIP:

[kudu-CR] KUDU-1942. Kerberos fails to log in on hostnames with capital letters

Sailesh Mukil has posted comments on this change. Change subject: KUDU-1942. Kerberos fails to log in on hostnames with capital letters .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7693/2//COMMIT_MSG

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

Sailesh Mukil has posted comments on this change. Change subject: KUDU-2032 (part 1): pass pre-resolution hostname into RPC proxies .. Patch Set 4: (2 comments)

[kudu-CR] disk failure: add persistent disk states

Todd Lipcon has posted comments on this change. Change subject: disk failure: add persistent disk states .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/7270/15//COMMIT_MSG Commit Message: Line 11: failed

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

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7687 to look at the new patch set (#4). Change subject: KUDU-2032 (part 1): pass pre-resolution hostname into RPC proxies

[kudu-CR] KUDU-2032 (part 2): propagate master hostnames into client

Todd Lipcon has posted comments on this change. Change subject: KUDU-2032 (part 2): propagate master hostnames into client .. Patch Set 2: (3 comments)

[kudu-CR] disk failure: add persistent disk states

Adar Dembo has posted comments on this change. Change subject: disk failure: add persistent disk states .. Patch Set 15: (30 comments) Made it further in. Todd should review this too, since he was in that meeting we had to

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

Adar Dembo has posted comments on this change. Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. Patch Set 1: (8 comments) Mike should probably review this too.

[kudu-CR] KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout

Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout .. KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout Previously, when

[kudu-CR] KUDU-2032 (part 2): propagate master hostnames into client

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7692 to look at the new patch set (#3). Change subject: KUDU-2032 (part 2): propagate master hostnames into client ..

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

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7687 to look at the new patch set (#3). Change subject: KUDU-2032 (part 1): pass pre-resolution hostname into RPC proxies

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

Todd Lipcon has posted comments on this change. Change subject: KUDU-2032 (part 1): pass pre-resolution hostname into RPC proxies .. Patch Set 2: (5 comments)

[kudu-CR] KUDU-1942. Kerberos fails to log in on hostnames with capital letters

Todd Lipcon has posted comments on this change. Change subject: KUDU-1942. Kerberos fails to log in on hostnames with capital letters .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7693/2//COMMIT_MSG Commit

[kudu-CR] KUDU-871. Support tombstoned voting

David Ribeiro Alves has posted comments on this change. Change subject: KUDU-871. Support tombstoned voting .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-2101 Include a table summary at the bottom

Todd Lipcon has posted comments on this change. Change subject: KUDU-2101 Include a table summary at the bottom .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/7707/1/src/kudu/tools/ksck.cc File

[kudu-CR] KUDU-1489: move tablet metadata

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7617 to look at the new patch set (#2). Change subject: KUDU-1489: move tablet metadata .. KUDU-1489: move tablet

[kudu-CR] consensus: Improve contract for API to fetch last-logged op id

Todd Lipcon has posted comments on this change. Change subject: consensus: Improve contract for API to fetch last-logged op id .. Patch Set 1: (11 comments)

[kudu-CR] [iwyu] first pass

Dan Burkert has posted comments on this change. Change subject: [iwyu] first pass .. Patch Set 18: (41 comments) Good job on trimming the pragmas, looking much better now.

[kudu-CR] KUDU-1407: reassign failed tablets

Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 11: (3 comments)

[kudu-CR] KUDU-1407: reassign failed tablets

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#12). Change subject: KUDU-1407: reassign failed tablets .. KUDU-1407: reassign

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

Adar Dembo has posted comments on this change. Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. Patch Set 19: (14 comments) I think I reviewed everything, but I'm curious to see how this will evolve

[kudu-CR] rpc: some small cleanup in ConnectionId

Todd Lipcon has submitted this change and it was merged. Change subject: rpc: some small cleanup in ConnectionId .. rpc: some small cleanup in ConnectionId Change-Id: I0788052f8c943ef102f3f551a85a8b219c65c361 Reviewed-on:

[kudu-CR] KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout

Todd Lipcon has posted comments on this change. Change subject: KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7599 To

[kudu-CR] KUDU-1407: reassign failed tablets

Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 7: (7 comments) http://gerrit.cloudera.org:8080/#/c/7440/7/src/kudu/client/scanner-internal.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#11). Change subject: KUDU-1407: reassign failed tablets .. KUDU-1407: reassign

[kudu-CR] KUDU-871. Support tombstoned voting

Hello David Ribeiro Alves, Todd Lipcon, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6960 to look at the new patch set (#7). Change subject: KUDU-871. Support tombstoned voting

[kudu-CR] consensus: Tablet copy should clear last-logged opid from superblock

Hello David Ribeiro Alves, Todd Lipcon, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7718 to review the following change. Change subject: consensus: Tablet copy should clear last-logged opid from superblock

[kudu-CR] consensus: Improve contract for API to fetch last-logged op id

Hello David Ribeiro Alves, Todd Lipcon, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7717 to review the following change. Change subject: consensus: Improve contract for API to fetch last-logged op id

[kudu-CR] [docs] Deprecate Java 7 and Spark 1

Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Deprecate Java 7 and Spark 1 .. Patch Set 1: Oh and https://github.com/apache/kudu/tree/master/java#system-requirements -- To view, visit

[kudu-CR] [docs] Deprecate Java 7 and Spark 1

Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Deprecate Java 7 and Spark 1 .. Patch Set 1: Can you add a note to http://kudu.apache.org/docs/installation.html#build_java_client ? -- To view,

[kudu-CR] KUDU-1407: reassign failed tablets

David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 10: (6 comments) http://gerrit.cloudera.org:8080/#/c/7440/7/src/kudu/client/scanner-internal.cc File

[kudu-CR] [docs] Deprecate Java 7 and Spark 1

Dan Burkert has posted comments on this change. Change subject: [docs] Deprecate Java 7 and Spark 1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7699 To unsubscribe, visit

[kudu-CR] KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout

Michael Ho has posted comments on this change. Change subject: KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout .. Patch Set 5: The verification failure may be infra related. Not sure if I have the

[kudu-CR] KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7599 to look at the new patch set (#5). Change subject: KUDU-2065, KUDU-2011: Release sidecars on cancellation or timeout

[kudu-CR] KUDU-2101 Include a table summary at the bottom

Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/7707 Change subject: KUDU-2101 Include a table summary at the bottom .. KUDU-2101 Include a table summary at the bottom This add a table

[kudu-CR] disk failure: add persistent disk states

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7270 to look at the new patch set (#15). Change subject: disk failure: add persistent disk states .. disk failure: add

[kudu-CR] Add tablet state summary metrics and fix KUDU-2044

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7618 to look at the new patch set (#4). Change subject: Add tablet state summary metrics and fix KUDU-2044 .. Add

[kudu-CR] KUDU-1942. Kerberos fails to log in on hostnames with capital letters

Sailesh Mukil has posted comments on this change. Change subject: KUDU-1942. Kerberos fails to log in on hostnames with capital letters .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7693/2//COMMIT_MSG

[kudu-CR] KUDU-1942. Kerberos fails to log in on hostnames with capital letters

Todd Lipcon has posted comments on this change. Change subject: KUDU-1942. Kerberos fails to log in on hostnames with capital letters .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7693/2//COMMIT_MSG Commit

[kudu-CR] KUDU-1942. Kerberos fails to log in on hostnames with capital letters

Sailesh Mukil has posted comments on this change. Change subject: KUDU-1942. Kerberos fails to log in on hostnames with capital letters .. Patch Set 2: Code-Review+1 LGTM after Alexey's comments have been addressed. -- To

[kudu-CR] Add tablet state summary metrics and fix KUDU-2044

Will Berkeley has posted comments on this change. Change subject: Add tablet state summary metrics and fix KUDU-2044 .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7618/2//COMMIT_MSG Commit Message: PS2,

[kudu-CR] security: only lookup hostname if HOST substitution is required

Sailesh Mukil has posted comments on this change. Change subject: security: only lookup hostname if _HOST substitution is required .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7694 To

[kudu-CR] Add tablet state summary metrics and fix KUDU-2044

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7618 to look at the new patch set (#3). Change subject: Add tablet state summary metrics and fix KUDU-2044 .. Add

[kudu-CR] [gitignore] added *.autosave

Todd Lipcon has posted comments on this change. Change subject: [gitignore] added *.autosave .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7702 To unsubscribe, visit

[kudu-CR] [gitignore] added *.autosave

Todd Lipcon has submitted this change and it was merged. Change subject: [gitignore] added *.autosave .. [gitignore] added *.autosave Change-Id: I99457244f007c757eb1a5a0aba235d8e33e4460e Reviewed-on:

[kudu-CR] [java] Update outdated dependencies

Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java] Update outdated dependencies .. [java] Update outdated dependencies Includes the following version updates: - Avro 1.8.1 -> 1.8.2 - Flume

[kudu-CR] [iwyu] first pass

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4738 to look at the new patch set (#18). Change subject: [iwyu] first pass .. [iwyu] first pass Updated C++ source

[kudu-CR] KUDU-2032 (part 2): propagate master hostnames into client

Alexey Serbin has posted comments on this change. Change subject: KUDU-2032 (part 2): propagate master hostnames into client .. Patch Set 2: (1 comment) looks good to me, just a nit

[kudu-CR] disk failure: add persistent disk states

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7270 to look at the new patch set (#14). Change subject: disk failure: add persistent disk states .. disk failure: add

[kudu-CR] [gitignore] added *.autosave

Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7702 Change subject: [gitignore] added *.autosave .. [gitignore] added *.autosave Change-Id: I99457244f007c757eb1a5a0aba235d8e33e4460e --- M

[kudu-CR] [iwyu] first pass

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4738 to look at the new patch set (#17). Change subject: [iwyu] first pass .. [iwyu] first pass Updated C++ source

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7701 Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. KUDU-1726: Avoid fsync-per-block in tablet copy This patch incorporates

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

Hao Hao has abandoned this change. Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7700 To unsubscribe, visit

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#19). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

Hao Hao has uploaded a new patch set (#2). Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. KUDU-1726: Avoid fsync-per-block in tablet copy This patch incorporates BlockTransaction API with tablet copy, to

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7700 Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. KUDU-1726: Avoid fsync-per-block in tablet copy This patch incorporates