[kudu-CR] Cleanup/refactor tracking of consensus watermarks

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4133 to look at the new patch set (#5). Change subject: Cleanup/refactor tracking of consensus watermarks .. Cleanup/ref

[kudu-CR] Cleanup/refactor tracking of consensus watermarks

Kudu Jenkins has posted comments on this change. Change subject: Cleanup/refactor tracking of consensus watermarks .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/3183/ -- To view, visit http://gerrit.clou

[kudu-CR] Cleanup/refactor tracking of consensus watermarks

Todd Lipcon has posted comments on this change. Change subject: Cleanup/refactor tracking of consensus watermarks .. Patch Set 4: (23 comments) http://gerrit.cloudera.org:8080/#/c/4133/4/src/kudu/consensus/consensus.proto File

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Todd Lipcon has posted comments on this change. Change subject: tool: port kudu-admin to 'kudu cluster' .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4180/3/src/kudu/tools/tool_action_cluster.cc File src/kudu

[kudu-CR] ts tablet manager: set status message after failed tablet copy

Kudu Jenkins has posted comments on this change. Change subject: ts_tablet_manager: set status message after failed tablet copy .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3182/ -- To view, visit http:

[kudu-CR] ts tablet manager: set status message after failed tablet copy

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4194 to look at the new patch set (#3). Change subject: ts_tablet_manager: set status message after failed tablet copy ..

[kudu-CR] tracing: gzip the trace JSON ajax response

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4154 to look at the new patch set (#2). Change subject: tracing: gzip the trace JSON ajax response .. tracing: gzip the

[kudu-CR] tracing: gzip the trace JSON ajax response

Kudu Jenkins has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3181/ -- To view, visit http://gerrit.cloudera.or

[kudu-CR] tracing: gzip the trace JSON ajax response

Todd Lipcon has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4154/1//COMMIT_MSG Commit Message: Line 18: version u

[kudu-CR] Inlined dispatch for predicate evaluation

Todd Lipcon has posted comments on this change. Change subject: Inlined dispatch for predicate evaluation .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4164/2/src/kudu/common/column_predicate.h File src/kudu/

[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

Todd Lipcon has posted comments on this change. Change subject: Fix kudu-ts-cli crash when there is no data in tablet .. Patch Set 7: (9 comments) http://gerrit.cloudera.org:8080/#/c/4134/7/src/kudu/tools/kudu-ts-cli-test.cc F

[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

Kudu Jenkins has posted comments on this change. Change subject: Fix kudu-ts-cli crash when there is no data in tablet .. Patch Set 7: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/3180/ -- To view, visit http

[kudu-CR] ksck: add --checksum scan concurrency to optional flags

Kudu Jenkins has posted comments on this change. Change subject: ksck: add --checksum_scan_concurrency to optional flags .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3179/ -- To view, visit http://gerri

[kudu-CR] ksck: add --checksum scan concurrency to optional flags

Todd Lipcon has posted comments on this change. Change subject: ksck: add --checksum_scan_concurrency to optional flags .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4183/1/src/kudu/tools/tool_action_cluster.

[kudu-CR] ksck: add --checksum scan concurrency to optional flags

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4183 to look at the new patch set (#2). Change subject: ksck: add --checksum_scan_concurrency to optional flags .. ksck:

[kudu-CR] tablet peer: clean up TabletStatusListener interface

Kudu Jenkins has posted comments on this change. Change subject: tablet_peer: clean up TabletStatusListener interface .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3178/ -- To view, visit http://gerrit.c

[kudu-CR] tablet peer: clean up TabletStatusListener interface

Todd Lipcon has posted comments on this change. Change subject: tablet_peer: clean up TabletStatusListener interface .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4193/1/src/kudu/tablet/tablet_peer.h File src

[kudu-CR] tablet peer: clean up TabletStatusListener interface

Hello Mike Percy, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4193 to look at the new patch set (#3). Change subject: tablet_peer: clean up TabletStatusListener interface

[kudu-CR] ts tablet manager: set status message after failed tablet copy

Kudu Jenkins has posted comments on this change. Change subject: ts_tablet_manager: set status message after failed tablet copy .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3177/ -- To view, visit http:

[kudu-CR] tablet peer: clean up TabletStatusListener interface

Hello Mike Percy, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4193 to look at the new patch set (#2). Change subject: tablet_peer: clean up TabletStatusListener interface

[kudu-CR] ts tablet manager: set status message after failed tablet copy

Kudu Jenkins has posted comments on this change. Change subject: ts_tablet_manager: set status message after failed tablet copy .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3176/ -- To view, visit http:

[kudu-CR] tablet peer: clean up TabletStatusListener interface

Kudu Jenkins has posted comments on this change. Change subject: tablet_peer: clean up TabletStatusListener interface .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3175/ -- To view, visit http://gerrit.c

[kudu-CR] ts tablet manager: set status message after failed tablet copy

Hello Mike Percy, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4194 to review the following change. Change subject: ts_tablet_manager: set status message after failed tablet copy .

[kudu-CR] transaction tracker: back-off when logging in-flight transactions

Todd Lipcon has posted comments on this change. Change subject: transaction_tracker: back-off when logging in-flight transactions .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4159/1/src/kudu/tablet/transact

[kudu-CR] tracing: gzip the trace JSON ajax response

Todd Lipcon has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4154/1//COMMIT_MSG Commit Message: Line 16: the serve

[kudu-CR] tool: port log-dump

Dinesh Bhat has posted comments on this change. Change subject: tool: port log-dump .. Patch Set 3: (4 comments) Hi Adar, sorry I missed the train here, but these are more of curious questions than review comments as such so

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Dinesh Bhat has posted comments on this change. Change subject: tool: port kudu-admin to 'kudu cluster' .. Patch Set 3: Code-Review+1 (6 comments) Few nits, otherwise LGTM. http://gerrit.cloudera.org:8080/#/c/4180/3/src/kudu/

[kudu-CR] KUDU-1582. Optimize budgeted compaction policy with an approximation

David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1582. Optimize budgeted compaction policy with an approximation .. Patch Set 2: (9 comments) http://gerrit.cloudera.org:8080/#/c/4153/2/src/kudu/tab

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

Alexey Serbin has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 25: (5 comments) http://gerrit.cloudera.org:8080/#/c/3952/25/src/kudu/client/client-test.cc Fil

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Matthew Jacobs has posted comments on this change. Change subject: [java-client] Add ScanToken.stringifySerializedToken .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4173 To unsubscribe, visit

[kudu-CR] c++ client: expose private GetTablet API

Kudu Jenkins has posted comments on this change. Change subject: c++ client: expose private GetTablet API .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3174/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] c++ client: expose private GetTablet API

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4179 to look at the new patch set (#2). Change subject: c++ client: expose private GetTablet API .. c++ client: expose p

[kudu-CR] c++ client: expose private GetTablet API

Adar Dembo has posted comments on this change. Change subject: c++ client: expose private GetTablet API .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4179/1/src/kudu/client/client.h File src/kudu/client/clie

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

Adar Dembo has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 25: (5 comments) http://gerrit.cloudera.org:8080/#/c/3952/25/src/kudu/client/client-test.cc File s

[kudu-CR] c++ client: expose private GetTablet API

Todd Lipcon has posted comments on this change. Change subject: c++ client: expose private GetTablet API .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4179/1/src/kudu/client/client.h File src/kudu/client/cli

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

Alexey Serbin has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 25: (44 comments) Thank you for review. I'm planning to post updated patch soon. http://gerri

[kudu-CR] tablet peer: clean up TabletStatusListener interface

Adar Dembo has posted comments on this change. Change subject: tablet_peer: clean up TabletStatusListener interface .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4193/1/src/kudu/tablet/tablet_peer.h File src/

[kudu-CR] tablet peer: clean up TabletStatusListener interface

Hello Dinesh Bhat, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4193 to review the following change. Change subject: tablet_peer: clean up TabletStatusListener interface .. ta

[kudu-CR] tablet peer: clean up TabletStatusListener interface

Kudu Jenkins has posted comments on this change. Change subject: tablet_peer: clean up TabletStatusListener interface .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3173/ -- To view, visit http://gerrit.c

[kudu-CR] log block manager: reduce allocation at startup

Adar Dembo has posted comments on this change. Change subject: log_block_manager: reduce allocation at startup .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4192/1/src/kudu/fs/log_block_manager.cc File src/k

[kudu-CR] compaction policy: avoid O(n^2) calls to EstimateOnDiskSize

David Ribeiro Alves has posted comments on this change. Change subject: compaction_policy: avoid O(n^2) calls to EstimateOnDiskSize .. Patch Set 1: LGTM but could you add the numbers you obtained to the commit message -- To v

[kudu-CR] log block manager: reduce allocation at startup

Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4192 to review the following change. Change subject: log_block_manager: reduce allocation at startup .. log_block_manager: r

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Adar Dembo has posted comments on this change. Change subject: [java-client] Add ScanToken.stringifySerializedToken .. Patch Set 2: Code-Review+2 Leaving open in case MJ has more comments. -- To view, visit http://gerrit.clou

[kudu-CR] compaction policy: avoid O(n^2) calls to EstimateOnDiskSize

Todd Lipcon has posted comments on this change. Change subject: compaction_policy: avoid O(n^2) calls to EstimateOnDiskSize .. Patch Set 1: I put this on the test cluster in question and the FindBestOp call went from taking ab

[kudu-CR] compaction policy: avoid O(n^2) calls to EstimateOnDiskSize

Kudu Jenkins has posted comments on this change. Change subject: compaction_policy: avoid O(n^2) calls to EstimateOnDiskSize .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3172/ -- To view, visit http://g

[kudu-CR] compaction policy: avoid O(n^2) calls to EstimateOnDiskSize

Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4191 to review the following change. Change subject: compaction_policy: avoid O(n^2) calls to EstimateOnDiskSize ..

[kudu-CR](gh-pages) jekyll: Use GitHub-flavored markdown and GitHub style highlighting

Mike Percy has submitted this change and it was merged. Change subject: jekyll: Use GitHub-flavored markdown and GitHub style highlighting .. jekyll: Use GitHub-flavored markdown and GitHub style highlighting The default Kramd

[kudu-CR](gh-pages) blog: Minor tweaks to flume article

Mike Percy has posted comments on this change. Change subject: blog: Minor tweaks to flume article .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4190 To unsubscribe, visit http://ge

[kudu-CR](gh-pages) blog: Minor tweaks to flume article

Mike Percy has submitted this change and it was merged. Change subject: blog: Minor tweaks to flume article .. blog: Minor tweaks to flume article This just corrects a couple minor things that were missed in the initial push.

[kudu-CR](gh-pages) jekyll: Use GitHub-flavored markdown and GitHub style highlighting

Mike Percy has posted comments on this change. Change subject: jekyll: Use GitHub-flavored markdown and GitHub style highlighting .. Patch Set 1: Code-Review+2 Verified+1 I am pushing this myself to un-break the site. -- To

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Matthew Jacobs has posted comments on this change. Change subject: [java-client] Add ScanToken.stringifySerializedToken .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4173/1//COMMIT_MSG Commit Message: PS1, L

[kudu-CR](gh-pages) jekyll: Use GitHub-flavored markdown and GitHub style highlighting

Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/4189 Change subject: jekyll: Use GitHub-flavored markdown and GitHub style highlighting .. jekyll: Use GitHub-flavored markdown and GitHub style

[kudu-CR](gh-pages) blog: Minor tweaks to flume article

Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/4190 Change subject: blog: Minor tweaks to flume article .. blog: Minor tweaks to flume article This just corrects a couple minor things that we

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Kudu Jenkins has posted comments on this change. Change subject: [java-client] Add ScanToken.stringifySerializedToken .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3171/ -- To view, visit http://gerrit.c

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Dan Burkert has posted comments on this change. Change subject: [java-client] Add ScanToken.stringifySerializedToken .. Patch Set 2: (9 comments) http://gerrit.cloudera.org:8080/#/c/4173/1//COMMIT_MSG Commit Message: PS1, Lin

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4173 to look at the new patch set (#2). Change subject: [java-client] Add ScanToken.stringifySerializedToken .. [java-cl

[kudu-CR] WIP: tie log retention to consensus watermarks

David Ribeiro Alves has posted comments on this change. Change subject: WIP: tie log retention to consensus watermarks .. Patch Set 1: (3 comments) partial review. back on it later (may wait a little bit to see if it stops be

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Matthew Jacobs has posted comments on this change. Change subject: [java-client] Add ScanToken.stringifySerializedToken .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4173/1//COMMIT_MSG Commit Message: PS1, L

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Adar Dembo has posted comments on this change. Change subject: [java-client] Add ScanToken.stringifySerializedToken .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/4173/1//COMMIT_MSG Commit Message: PS1, Line

[kudu-CR] Cleanup/refactor tracking of consensus watermarks

David Ribeiro Alves has posted comments on this change. Change subject: Cleanup/refactor tracking of consensus watermarks .. Patch Set 4: (17 comments) http://gerrit.cloudera.org:8080/#/c/4133/4/src/kudu/consensus/consensus_qu

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Alexey Serbin has posted comments on this change. Change subject: tool: port kudu-admin to 'kudu cluster' .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4180 To unsubscribe, visit http://gerrit

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4180 to look at the new patch set (#3). Change subject: tool: port kudu-admin to 'kudu cluster' .. tool: port kudu-admin

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Adar Dembo has posted comments on this change. Change subject: tool: port kudu-admin to 'kudu cluster' .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4180/1/src/kudu/tools/kudu-admin-test.cc File src/kudu/too

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Kudu Jenkins has posted comments on this change. Change subject: tool: port kudu-admin to 'kudu cluster' .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3170/ -- To view, visit http://gerrit.cloudera.org:8

[kudu-CR] [java-client] Add ScanToken.stringifySerializedToken

Matthew Jacobs has posted comments on this change. Change subject: [java-client] Add ScanToken.stringifySerializedToken .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4173/1//COMMIT_MSG Commit Message: PS1,

[kudu-CR] tracing: gzip the trace JSON ajax response

David Ribeiro Alves has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4154/1//COMMIT_MSG Commit Message: Line 16: t

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Alexey Serbin has posted comments on this change. Change subject: tool: port kudu-admin to 'kudu cluster' .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4180/1/src/kudu/tools/kudu-admin-test.cc File src/kudu/

[kudu-CR] transaction tracker: back-off when logging in-flight transactions

David Ribeiro Alves has posted comments on this change. Change subject: transaction_tracker: back-off when logging in-flight transactions .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4159/1/src/kudu/tablet/

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

Adar Dembo has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 25: (45 comments) I actually reviewed the changes to client-test this time. http://gerrit.clouder

[kudu-CR] KUDU-668. log block manager should tolerate empty metadata

Adar Dembo has posted comments on this change. Change subject: KUDU-668. log block manager should tolerate empty metadata .. Patch Set 1: (2 comments) Yeah, we can be more forgiving in some cases (i.e. data_size >= but metada

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4180 to look at the new patch set (#2). Change subject: tool: port kudu-admin to 'kudu cluster' .. tool: port kudu-admin

[kudu-CR] tool: port kudu-admin to 'kudu cluster'

Kudu Jenkins has posted comments on this change. Change subject: tool: port kudu-admin to 'kudu cluster' .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3169/ -- To view, visit http://gerrit.cloudera.org:8

[kudu-CR] ksck: add --checksum scan concurrency to optional flags

David Ribeiro Alves has posted comments on this change. Change subject: ksck: add --checksum_scan_concurrency to optional flags .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4183/1/src/kudu/tools/tool_action_

[kudu-CR] ksck: add --checksum scan concurrency to optional flags

Kudu Jenkins has posted comments on this change. Change subject: ksck: add --checksum_scan_concurrency to optional flags .. Patch Set 1: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/3168/ -- To view, visit ht

[kudu-CR](gh-pages) kudu flume sink blog post

Mike Percy has submitted this change and it was merged. Change subject: kudu flume sink blog post .. kudu flume sink blog post Change-Id: I810146ab24c88bc6cc562d81746b9bf5303396ed Reviewed-on: http://gerrit.cloudera.org:8080/35

[kudu-CR](gh-pages) kudu flume sink blog post

Mike Percy has posted comments on this change. Change subject: kudu flume sink blog post .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3510 To unsubscribe, visit http://gerrit.cloudera.org:8080/s