[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

Kudu Jenkins has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2895/ -- To view, visit http:

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

Todd Lipcon has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3786 To unsubscribe

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

Hello Mike Percy, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3786 to look at the new patch set (#7). Change subject: KUDU-526: use on-disk cmeta when loading existing master state ..

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

Kudu Jenkins has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/2894/ -- To view, visit http:

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

Hello Mike Percy, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3786 to look at the new patch set (#6). Change subject: KUDU-526: use on-disk cmeta when loading existing master state ..

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

Kudu Jenkins has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 15: Build Started http://104.196.14.100/job/kudu-gerrit/2892/ -- To view, visit http:/

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3628 to look at the new patch set (#15). Change subject: Add time/watermark based garbage collection to ResultTracker ..

[kudu-CR] Start a background thread to run ResultTracker GC

Kudu Jenkins has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2891/ -- To view, visit http://gerrit.clou

[kudu-CR] Memory tracking for result tracker

Hello David Ribeiro Alves, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3627 to look at the new patch set (#21). Change subject: Memory tracking for result tracker .. Memory tracking fo

[kudu-CR] Start a background thread to run ResultTracker GC

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3961 to look at the new patch set (#3). Change subject: Start a background thread to run ResultTracker GC .. Start a bac

[kudu-CR] Memory tracking for result tracker

Kudu Jenkins has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 21: Build Started http://104.196.14.100/job/kudu-gerrit/2893/ -- To view, visit http://gerrit.cloudera.org:8080/

[kudu-CR] [util/env posix.cc] fixed build on MacOS X

Todd Lipcon has posted comments on this change. Change subject: [util/env_posix.cc] fixed build on MacOS X .. Patch Set 1: Code-Review+2 oops, my bad! -- To view, visit http://gerrit.cloudera.org:8080/3978 To unsubscribe, vis

[kudu-CR] [util/env posix.cc] fixed build on MacOS X

Kudu Jenkins has posted comments on this change. Change subject: [util/env_posix.cc] fixed build on MacOS X .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2890/ -- To view, visit http://gerrit.cloudera.or

[kudu-CR] [util/env posix.cc] fixed build on MacOS X

Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3978 Change subject: [util/env_posix.cc] fixed build on MacOS X .. [util/env_posix.cc] fixed build on MacOS X Fixed a tiny typo which broke b

[kudu-CR] Memory tracking for result tracker

Todd Lipcon has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 20: (3 comments) http://gerrit.cloudera.org:8080/#/c/3627/19/src/kudu/rpc/result_tracker.cc File src/kudu/rpc/resu

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

Kudu Jenkins has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2889/ -- To view, visit http:

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3786 to look at the new patch set (#5). Change subject: KUDU-526: use on-disk cmeta when loading existing master state ..

[kudu-CR] Replace gscoped ptr with unique ptr for ConsensusMetadata

Adar Dembo has submitted this change and it was merged. Change subject: Replace gscoped_ptr with unique_ptr for ConsensusMetadata .. Replace gscoped_ptr with unique_ptr for ConsensusMetadata Change-Id: Iffc81148d497f1da95ccd25c

[kudu-CR] integration tests: end-to-end master permanent failure test

Todd Lipcon has posted comments on this change. Change subject: integration_tests: end-to-end master permanent failure test .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3969/2/src/kudu/tools/tool_action.cc

[kudu-CR] KUDU-1474: single to multi-master deployment migration

Todd Lipcon has posted comments on this change. Change subject: KUDU-1474: single to multi-master deployment migration .. Patch Set 6: (6 comments) http://gerrit.cloudera.org:8080/#/c/3880/6/src/kudu/gutil/strings/join.h File

[kudu-CR] fs: allow format with arbitrary uuid

Todd Lipcon has posted comments on this change. Change subject: fs: allow format with arbitrary uuid .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3968/2/src/kudu/fs/fs_manager.cc File src/kudu/fs/fs_manager.

[kudu-CR] Replace gscoped ptr with unique ptr for ConsensusMetadata

Todd Lipcon has posted comments on this change. Change subject: Replace gscoped_ptr with unique_ptr for ConsensusMetadata .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3967 To unsubscribe, vis

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

Adar Dembo has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3786/3/src/kudu/master/sys_catalog.c

[kudu-CR] Improve some comments in the tablet compaction code

Mike Percy has abandoned this change. Change subject: Improve some comments in the tablet compaction code .. Abandoned Superceded by https://gerrit.cloudera.org/3977 -- To view, visit http://gerrit.cloudera.org:8080/3973 To u

[kudu-CR] KUDU-236. Implement tablet history GC

Todd Lipcon has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 12: (3 comments) http://gerrit.cloudera.org:8080/#/c/3076/12/src/kudu/integration-tests/tablet_history_gc-itest

[kudu-CR] Various comment / doc improvements

Todd Lipcon has posted comments on this change. Change subject: Various comment / doc improvements .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3977/1/src/kudu/tablet/delta_compaction.cc File src/kudu/tablet

[kudu-CR] Start a background thread to run ResultTracker GC

Todd Lipcon has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 2: (1 comment) will try to write a test. fwiw I've been testing this on a cluster ~2 days now and

[kudu-CR] MergeIterator: don't stop iterating on an empty block

Kudu Jenkins has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2888/ -- To view, visit http://gerrit.

[kudu-CR] MergeIterator: don't stop iterating on an empty block

Hello Anonymous Coward #80, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3976 to look at the new patch set (#2). Change subject: MergeIterator: don't stop iterating on an empty block .

[kudu-CR] Pull RETURN NOT LOG BLOCK MANAGER macro into its own file

Mike Percy has abandoned this change. Change subject: Pull RETURN_NOT_LOG_BLOCK_MANAGER macro into its own file .. Abandoned This might not be needed -- To view, visit http://gerrit.cloudera.org:8080/3972 To unsubscribe, visi

[kudu-CR] WIP: Fix ordered iterators

Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3976 to review the following change. Change subject: WIP: Fix ordered iterators .. WIP: Fix ordered iterators Needs test.

[kudu-CR] Various comment / doc improvements

Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3977 to review the following change. Change subject: Various comment / doc improvements .. Various comment / doc improvemen

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2884/ -- To view, visit http://ger

[kudu-CR] Create randomized tablet history GC itest

Kudu Jenkins has posted comments on this change. Change subject: Create randomized tablet history GC itest .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2883/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] WIP: Fix ordered iterators

Kudu Jenkins has posted comments on this change. Change subject: WIP: Fix ordered iterators .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2886/ -- To view, visit http://gerrit.cloudera.org:8080/3976 To u

[kudu-CR] Create randomized tablet history GC itest

Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3975 to look at the new patch set (#5). Change subject: Create randomized tablet history GC itest .. Create

[kudu-CR] KUDU-236. Implement tablet history GC

Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3076 to look at the new patch set (#12). Change subject: KUDU-236. Implement tablet history GC .. KUDU-236.

[kudu-CR] Various comment / doc improvements

Kudu Jenkins has posted comments on this change. Change subject: Various comment / doc improvements .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2887/ -- To view, visit http://gerrit.cloudera.org:8080/3

[kudu-CR] KUDU-236. Implement tablet history GC

Kudu Jenkins has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 12: Build Started http://104.196.14.100/job/kudu-gerrit/2885/ -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] Use nullptr instead of NULL in compaction

Mike Percy has submitted this change and it was merged. Change subject: Use nullptr instead of NULL in compaction .. Use nullptr instead of NULL in compaction No functional changes here. Change-Id: I2e9cba25065dbb4e577e3b4034d

[kudu-CR] KUDU-236. Implement tablet history GC

Mike Percy has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/3076/11/src/kudu/tablet/tablet_history_gc-test.cc File src/k

[kudu-CR] integration tests: end-to-end master permanent failure test

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3969 to look at the new patch set (#2). Change subject: integration_tests: end-to-end master permanent failure test .. i

[kudu-CR] integration tests: end-to-end master permanent failure test

Kudu Jenkins has posted comments on this change. Change subject: integration_tests: end-to-end master permanent failure test .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2881/ -- To view, visit http://g

[kudu-CR] KUDU-1474: single to multi-master deployment migration

Kudu Jenkins has posted comments on this change. Change subject: KUDU-1474: single to multi-master deployment migration .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/2880/ -- To view, visit http://gerrit

[kudu-CR] KUDU-1474: single to multi-master deployment migration

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3880 to look at the new patch set (#6). Change subject: KUDU-1474: single to multi-master deployment migration .. KUDU-1

[kudu-CR] fs: allow format with arbitrary uuid

Kudu Jenkins has posted comments on this change. Change subject: fs: allow format with arbitrary uuid .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2882/ -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] KUDU-236. Implement tablet history GC

Adar Dembo has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 11: Could you also add a GCing thread to mt-tablet-test? It's a good way to test interactions between concurrent

[kudu-CR] KUDU-236. Implement tablet history GC

Adar Dembo has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 11: (1 comment) Just passing through with a question. http://gerrit.cloudera.org:8080/#/c/3076/11/src/kudu/tabl

[kudu-CR] Pull RETURN NOT LOG BLOCK MANAGER macro into its own file

Todd Lipcon has posted comments on this change. Change subject: Pull RETURN_NOT_LOG_BLOCK_MANAGER macro into its own file .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3972 To unsubscribe, vis

[kudu-CR] Use nullptr instead of NULL in compaction

Todd Lipcon has posted comments on this change. Change subject: Use nullptr instead of NULL in compaction .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3971 To unsubscribe, visit http://gerrit

[kudu-CR] Create tablet history GC randomized itest

Kudu Jenkins has posted comments on this change. Change subject: Create tablet history GC randomized itest .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2879/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Create tablet history GC randomized itest

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3975 to look at the new patch set (#4). Change subject: Create tablet history GC randomized itest .. Create tablet histo

[kudu-CR] Create tablet history GC randomized itest

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3975 to look at the new patch set (#3). Change subject: Create tablet history GC randomized itest .. Create tablet histo

[kudu-CR] Create tablet history GC randomized itest

Kudu Jenkins has posted comments on this change. Change subject: Create tablet history GC randomized itest .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2878/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] KUDU-1522 Fix AlterTable Timestamp col with default value

Todd Lipcon has posted comments on this change. Change subject: KUDU-1522 Fix AlterTable Timestamp col with default value .. Patch Set 1: (2 comments) Thanks for fixing this. Just a small style nit. http://gerrit.cloudera.org

[kudu-CR] Reorganize range partition client API

Todd Lipcon has submitted this change and it was merged. Change subject: Reorganize range partition client API .. Reorganize range partition client API This commit redesigns the client APIs dealing with adding and dropping rang

[kudu-CR] log block manager: replace custom RAII class with ScopedCleanup

Todd Lipcon has submitted this change and it was merged. Change subject: log_block_manager: replace custom RAII class with ScopedCleanup .. log_block_manager: replace custom RAII class with ScopedCleanup Change-Id: I810626472ad

[kudu-CR] [java-client] Reorganize range partition client API

Todd Lipcon has submitted this change and it was merged. Change subject: [java-client] Reorganize range partition client API .. [java-client] Reorganize range partition client API Equivalient to the corresponding C++ commit whi

[kudu-CR] monotime: remove granularity argument

Todd Lipcon has submitted this change and it was merged. Change subject: monotime: remove granularity argument .. monotime: remove granularity argument In practice, we almost never used the 'COARSE' granularity. Instead, it jus

[kudu-CR] KUDU-1559. Fix block manager metric for aborted blocks

Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1559. Fix block manager metric for aborted blocks .. KUDU-1559. Fix block manager metric for aborted blocks We weren't properly decrementing the "ope

[kudu-CR] monotime: remove granularity argument

Todd Lipcon has posted comments on this change. Change subject: monotime: remove granularity argument .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3966/1//COMMIT_MSG Commit Message: Line 13: Let's remove it

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2875/ -- To view, visit http://ger

[kudu-CR] Create tablet history GC randomized itest

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3975 to look at the new patch set (#2). Change subject: Create tablet history GC randomized itest .. Create tablet histo

[kudu-CR] Improve some comments in the tablet compaction code

Kudu Jenkins has posted comments on this change. Change subject: Improve some comments in the tablet compaction code .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2876/ -- To view, visit http://gerrit.cl

[kudu-CR] KUDU-236. Implement tablet history GC

Kudu Jenkins has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/2877/ -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] Create tablet history GC randomized itest

Kudu Jenkins has posted comments on this change. Change subject: Create tablet history GC randomized itest .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2874/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] KUDU-236. Implement tablet history GC

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3076 to look at the new patch set (#11). Change subject: KUDU-236. Implement tablet history GC .. KUDU-236. Implement ta

[kudu-CR] KUDU-236. Implement tablet history GC

Mike Percy has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/3076/6/src/kudu/tablet/compaction.cc File src/kudu/tablet/com

[kudu-CR] Pull RETURN NOT LOG BLOCK MANAGER macro into its own file

Kudu Jenkins has posted comments on this change. Change subject: Pull RETURN_NOT_LOG_BLOCK_MANAGER macro into its own file .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2870/ -- To view, visit http://ger

[kudu-CR] Use nullptr instead of NULL in compaction

Kudu Jenkins has posted comments on this change. Change subject: Use nullptr instead of NULL in compaction .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2869/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Improve some comments in the tablet compaction code

Kudu Jenkins has posted comments on this change. Change subject: Improve some comments in the tablet compaction code .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2871/ -- To view, visit http://gerrit.cl

[kudu-CR] Use nullptr instead of NULL in compaction

Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3971 to review the following change. Change subject: Use nullptr instead of NULL in compaction .. Use nullptr instead of NU

[kudu-CR] KUDU-236. Implement tablet history GC

Kudu Jenkins has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/2868/ -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] Pull RETURN NOT LOG BLOCK MANAGER macro into its own file

Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3972 to review the following change. Change subject: Pull RETURN_NOT_LOG_BLOCK_MANAGER macro into its own file .. Pull RETU

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3974 to review the following change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Create ba

[kudu-CR] Create tablet history GC randomized itest

Kudu Jenkins has posted comments on this change. Change subject: Create tablet history GC randomized itest .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2873/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2872/ -- To view, visit http://ger

[kudu-CR] Improve some comments in the tablet compaction code

Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3973 to review the following change. Change subject: Improve some comments in the tablet compaction code .. Improve some co

[kudu-CR] Create tablet history GC randomized itest

Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3975 to review the following change. Change subject: Create tablet history GC randomized itest .. Create tablet history GC

[kudu-CR] KUDU-236. Implement tablet history GC

Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3076 to look at the new patch set (#10). Change subject: KUDU-236. Implement tablet history GC .. KUDU-236.