[kudu-CR] c++ client: use operation timeout as deadline for finding new leader master

2016-07-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: c++ client: use operation timeout as deadline for finding new leader master .. Patch Set 2: Alright, let's try this and see if people complain about the UX. (I'm

[kudu-CR](gh-pages) Update the overview page

2016-07-22 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3726 to review the following change. Change subject: Update the overview page .. Update the overview page Long-overdue

[kudu-CR](gh-pages) Update the docs webpages to reflect the master branch

2016-07-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Update the docs webpages to reflect the master branch .. Patch Set 1: I think updating prior to release is nice, but only if we do the "this is an unreleased

[kudu-CR] ksck: improve output for long-running ksck checksums

2016-07-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: ksck: improve output for long-running ksck checksums .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3715/1//COMMIT_MSG Commit Message: PS1,

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. Patch Set 4: (3 comments)

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-22 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3632 to look at the new patch set (#5). Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial)

[kudu-CR] ksck: improve output for long-running ksck checksums

2016-07-22 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3715 to look at the new patch set (#2). Change subject: ksck: improve output for long-running ksck checksums

[kudu-CR] Fix block manager-test running in some builds

2016-07-24 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3733 to look at the new patch set (#3). Change subject: Fix block_manager-test running in some builds .. Fix

[kudu-CR] KUDU-1538: prevent block ID reuse to avoid potential data loss

2016-07-21 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1538: prevent block ID reuse to avoid potential data loss .. Patch Set 2: (1 comment)

[kudu-CR] WIP: KUDU-1271: allow primary keys not at the start of the row

2016-07-21 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3722 Change subject: WIP: KUDU-1271: allow primary keys not at the start of the row .. WIP: KUDU-1271: allow primary keys not at the start of

[kudu-CR] docs: Add missing DISTRIBUTE to quickstart

2016-07-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: Add missing DISTRIBUTE to quickstart .. Patch Set 1: Did you verify this syntax by trying it? (I can't remember if the DISTRIBUTE BY goes before or after

[kudu-CR] docs: fix links to MR examples in developing

2016-07-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: fix links to MR examples in developing .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3821/1/docs/developing.adoc File

[kudu-CR] KUDU-1548. Fix flaky RaftConsensusITest.TestReplaceChangeConfigOperation

2016-07-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1548. Fix flaky RaftConsensusITest.TestReplaceChangeConfigOperation .. KUDU-1548. Fix flaky RaftConsensusITest.TestReplaceChangeConfigOperation

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-07-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1358 (part 3): new multi-master stress test .. Patch Set 13: Code-Review+1 have you looped this one? Also, Dan, any further comments? -- To view, visit

[kudu-CR] [util] fixed build on MacOS X

2016-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [util] fixed build on MacOS X .. Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3836/1/src/kudu/util/CMakeLists.txt File

[kudu-CR] [KuduScanBatch::const iterator] a minor clean-up

2016-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [KuduScanBatch::const_iterator] a minor clean-up .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/3834/1/src/kudu/client/scan_batch.h File

[kudu-CR] remote bootstrap client: mild API changes

2016-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: remote_bootstrap_client: mild API changes .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3811/3//COMMIT_MSG Commit Message: Line 15:

[kudu-CR] Fix ksck-test reliance on unordered map iteration order

2016-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix ksck-test reliance on unordered map iteration order .. Patch Set 1: Hey Will. I actually put a patch up already for this here

[kudu-CR] Timestamp::FromUint64 should return void

2016-08-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Timestamp::FromUint64 should return void .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3943/1/src/kudu/common/timestamp.h File

[kudu-CR] Clarify that delta compaction is for REDOs

2016-08-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Clarify that delta compaction is for REDOs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3944/1/src/kudu/tablet/tablet.h File

[kudu-CR] Timestamp::FromUint64 should return void

2016-08-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Timestamp::FromUint64 should return void .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3943 To unsubscribe, visit

[kudu-CR] fs tool: improve format for dumping a rowset

2016-08-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3946 Change subject: fs_tool: improve format for dumping a rowset .. fs_tool: improve format for dumping a rowset Change-Id:

[kudu-CR] KUDU-1555. PBC Flush() method should be async

2016-08-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1555. PBC Flush() method should be async .. KUDU-1555. PBC Flush() method should be async WritablePBContainerFile::Flush is meant to flush the data

[kudu-CR] KUDU-564 (part 1): log a 'diff' when tablet config changes

2016-08-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes .. Patch Set 3: oops... accidentally un-pushed rev 2... -- To view, visit

[kudu-CR] KUDU-564 (part 1): log a 'diff' when tablet config changes

2016-08-11 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3939 to look at the new patch set (#3). Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes

[kudu-CR] KUDU-1555. PBC Flush() method should be async

2016-08-11 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3951 to review the following change. Change subject: KUDU-1555. PBC Flush() method should be async ..

[kudu-CR] maintenance manager: fix gflags docs

2016-08-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: maintenance_manager: fix gflags docs .. maintenance_manager: fix gflags docs The gflag doc referenced an "emergency flush thread" which was removed more

[kudu-CR] KUDU-564 (part 1): log a 'diff' when tablet config changes

2016-08-11 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3939 to look at the new patch set (#2). Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes

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

2016-08-14 Thread Todd Lipcon (Code Review)
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

[kudu-CR] monotime: remove granularity argument

2016-08-14 Thread Todd Lipcon (Code Review)
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

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

2016-08-14 Thread Todd Lipcon (Code Review)
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.

[kudu-CR] Reorganize range partition client API

2016-08-14 Thread Todd Lipcon (Code Review)
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

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

2016-08-14 Thread Todd Lipcon (Code Review)
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:

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

2016-08-14 Thread Todd Lipcon (Code Review)
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

[kudu-CR] Various comment / doc improvements

2016-08-14 Thread Todd Lipcon (Code Review)
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

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

2016-08-14 Thread Todd Lipcon (Code Review)
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

2016-08-14 Thread Todd Lipcon (Code Review)
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] Replace gscoped ptr with unique ptr for ConsensusMetadata

2016-08-14 Thread Todd Lipcon (Code Review)
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,

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

2016-08-14 Thread Todd Lipcon (Code Review)
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

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

2016-08-14 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 12: (3 comments)

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

2016-08-15 Thread Todd Lipcon (Code Review)
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,

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

2016-08-15 Thread Todd Lipcon (Code Review)
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

[kudu-CR] Memory tracking for result tracker

2016-08-15 Thread Todd Lipcon (Code Review)
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

[kudu-CR] Memory tracking for result tracker

2016-08-15 Thread Todd Lipcon (Code Review)
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

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

2016-08-15 Thread Todd Lipcon (Code Review)
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

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

2016-08-15 Thread Todd Lipcon (Code Review)
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] [util/env posix.cc] fixed build on MacOS X

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. 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 build on MacOS X. This is a

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

2016-08-15 Thread Todd Lipcon (Code Review)
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] Another pass on 0.10.0 release notes

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Mike Percy, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3979 to review the following change. Change subject: Another pass on 0.10.0 release notes ..

[kudu-CR] KUDU-564 (part 1): log a 'diff' when tablet config changes

2016-08-12 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3939 to look at the new patch set (#4). Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes

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

2016-08-12 Thread Todd Lipcon (Code Review)
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 (#14). Change subject: Add time/watermark based garbage collection to ResultTracker ..

[kudu-CR] Add regex string match assert helper macros

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add regex string match assert helper macros .. Patch Set 1: I think this same functionality is already available using the ContainsRegex or MatchesRegex matcher

[kudu-CR] Memory tracking for result tracker

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 20: Verified+1 known flake -- To view, visit http://gerrit.cloudera.org:8080/3627 To unsubscribe, visit

[kudu-CR] env posix: improve error message on short writes

2016-08-12 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3959 to review the following change. Change subject: env_posix: improve error message on short writes ..

[kudu-CR] Memory tracking for result tracker

2016-08-12 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3627 to look at the new patch set (#20). Change subject: Memory tracking for result tracker ..

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

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3961 Change subject: Start a background thread to run ResultTracker GC .. Start a background thread to run ResultTracker GC Change-Id:

[kudu-CR] [util/stopwatch.h] fixed integer overflow

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [util/stopwatch.h] fixed integer overflow .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3960 To unsubscribe, visit

[kudu-CR] monotime: use a signed integer for the value

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: monotime: use a signed integer for the value .. Patch Set 1: yea, 2^63 microseconds is 292,277.266 years, so I dont think we're in danger of a system's uptime

[kudu-CR] monotime: use a signed integer for the value

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: monotime: use a signed integer for the value .. monotime: use a signed integer for the value MonoTime previously used a uint64_t to store its value. This

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

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 2: I looped exactly_once_writes-itest with this patch 1000 times (100% pass):

[kudu-CR] Add regex string match assert helper macros

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add regex string match assert helper macros .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3962 To unsubscribe, visit

[kudu-CR] test result server: improve indexes, add sorting

2016-08-10 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3889 to review the following change. Change subject: test_result_server: improve indexes, add sorting ..

[kudu-CR] Rename raft-remote-bootstrap design doc

2016-08-10 Thread Todd Lipcon (Code Review)
Hello Misty Stanley-Jones, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3890 to review the following change. Change subject: Rename raft-remote-bootstrap design doc .. Rename

[kudu-CR] test result server: improve indexes, add sorting

2016-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: test_result_server: improve indexes, add sorting .. Patch Set 2: (I deployed this) -- To view, visit http://gerrit.cloudera.org:8080/3889 To unsubscribe, visit

[kudu-CR] test result server: improve indexes, add sorting

2016-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: test_result_server: improve indexes, add sorting .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3889 To unsubscribe, visit

[kudu-CR] test result server: improve indexes, add sorting

2016-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: test_result_server: improve indexes, add sorting .. test_result_server: improve indexes, add sorting * adds a new index to the underlying database table.

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

2016-08-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 6: (14 comments) http://gerrit.cloudera.org:8080/#/c/3076/6/src/kudu/tablet/compaction.cc File

[kudu-CR] Reorganize range partition client API

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Reorganize range partition client API .. Patch Set 4: Code-Review+1 (2 comments) basically just looked at the API, rather than the impl, since it looks like

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

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [java-client] Reorganize range partition client API .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3958 To unsubscribe, visit

[kudu-CR] client/sample.cc: fixed a couple of crashes

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: client/sample.cc: fixed a couple of crashes .. client/sample.cc: fixed a couple of crashes Fixed 'pure virtual method called' error with subsequent

[kudu-CR] KUDU-1553: Crash when specifying multiple unbounded range predicates on a column

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1553: Crash when specifying multiple unbounded range predicates on a column .. KUDU-1553: Crash when specifying multiple unbounded range predicates

[kudu-CR] master: additional leader lock assertions in catalog manager

2016-08-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: master: additional leader lock assertions in catalog manager .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3684 To unsubscribe,

[kudu-CR] [java-client] Re-enable multi-master tests

2016-08-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [java-client] Re-enable multi-master tests .. Patch Set 4: Adar, do you think we can push this now? I'll hit a rebase to make sure it still passes tests -- To

[kudu-CR] [java-client] Re-enable multi-master tests

2016-08-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [java-client] Re-enable multi-master tests .. Patch Set 4: Oh, apparently it needs a manual rebase (I guess due to the package change) -- To view, visit

[kudu-CR] KUDU-1303 Document code style guidelines for C++11 move semantics and rvalue references

2016-08-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1303 Document code style guidelines for C++11 move semantics and rvalue references .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] Predicate evaluation pushdown

2016-08-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Predicate evaluation pushdown .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3841/1/src/kudu/cfile/binary_dict_block.cc File

[kudu-CR] remote bootstrap client: mild API changes

2016-08-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: remote_bootstrap_client: mild API changes .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3811 To unsubscribe, visit

[kudu-CR] Rename Remote Bootstrap to Tablet Copy (part 1)

2016-08-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Rename Remote Bootstrap to Tablet Copy (part 1) .. Rename Remote Bootstrap to Tablet Copy (part 1) This does a bunch of automated substitutions of the

[kudu-CR] Rename Remote Bootstrap: some manual changes

2016-08-05 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3852 to look at the new patch set (#2). Change subject: Rename Remote Bootstrap: some manual changes ..

[kudu-CR](gh-pages) Add weekly update for 8/8

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3864 Change subject: Add weekly update for 8/8 .. Add weekly update for 8/8 Change-Id: Idd1dcb28082b476c18a92a91f6f84af4c69df66b --- A

[kudu-CR](gh-pages) Add weekly update for 8/8

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new patch set (#2). Change subject: Add weekly update for 8/8 .. Add weekly update for 8/8 Change-Id: Idd1dcb28082b476c18a92a91f6f84af4c69df66b --- A _posts/2016-08-08-weekly-update.md 1 file changed,

[kudu-CR] Upgrade to NVML 1.1

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Upgrade to NVML 1.1 .. Upgrade to NVML 1.1 Change-Id: I9b28130407b4ec30bc13a9acfc2390cee7a69de4 Reviewed-on: http://gerrit.cloudera.org:8080/3866

[kudu-CR] [CMakeLists.txt] dot is mandatory for doxygen

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [CMakeLists.txt] dot is mandatory for doxygen .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3855 To unsubscribe, visit

[kudu-CR] catalog manager: cleanup status messages

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: catalog manager: cleanup status messages .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3858 To unsubscribe, visit

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 1: you don't want to use this new field in the client? seems strange to add it with no use case -- To view,

[kudu-CR] Adding some missing 0.10.0 release notes

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Adding some missing 0.10.0 release notes .. Patch Set 1: (8 comments) http://gerrit.cloudera.org:8080/#/c/3802/1/docs/release_notes.adoc File

[kudu-CR] Rename Remote Bootstrap to Tablet Copy (part 1)

2016-08-05 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3848 to look at the new patch set (#2). Change subject: Rename Remote Bootstrap to Tablet Copy (part 1) .. Rename Remote

[kudu-CR] [java-client] Re-enable multi-master tests

2016-08-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [java-client] Re-enable multi-master tests .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3654 To unsubscribe, visit

[kudu-CR] Rename Remote Bootstrap: some manual changes

2016-08-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Rename Remote Bootstrap: some manual changes .. Rename Remote Bootstrap: some manual changes This patch has a bunch more manual fixes to replace "Remote

[kudu-CR] Rename remote bootstrap files to 'tablet copy'

2016-08-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Rename remote bootstrap files to 'tablet copy' .. Rename remote bootstrap files to 'tablet copy' Change-Id: I3f31fb72d11d16ee05bf90be3a7c77a82e5db6f7

[kudu-CR] WIP: Adding some missing 0.10.0 release notes

2016-08-08 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3802 to look at the new patch set (#2). Change subject: WIP: Adding some missing 0.10.0 release notes ..

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: C++ client: deprecating KuduPartialRow::SetString() .. Patch Set 3: Can we try to close this one out for 0.10? Seems like it's basically ready and a good API

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-08-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1358 (part 3): new multi-master stress test .. KUDU-1358 (part 3): new multi-master stress test This commit adds a stress test for multiple masters.

[kudu-CR] master: do not delete unknown tablets

2016-08-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: master: do not delete unknown tablets .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3645 To unsubscribe, visit

[kudu-CR] KUDU-1374: send full tablet report when new leader master is detected

2016-08-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1374: send full tablet report when new leader master is detected .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3643 To

[kudu-CR] master: do not delete unknown tablets

2016-08-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: master: do not delete unknown tablets .. master: do not delete unknown tablets Quoting from docs/design-docs/multi-master-1.0.md: "The master and/or

[kudu-CR] KUDU-1374: send full tablet report when new leader master is detected

2016-08-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1374: send full tablet report when new leader master is detected .. KUDU-1374: send full tablet report when new leader master is detected This

[kudu-CR] make election timeout jitter more aggressive

2016-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: make election timeout jitter more aggressive .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3828/1//COMMIT_MSG Commit Message: Line 7: make

[kudu-CR] c++ client: remove unnecessary code

2016-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3809 To unsubscribe, visit

[kudu-CR] c++ client: remove unnecessary code

2016-08-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3809 To unsubscribe, visit

  1   2   3   4   5   6   7   8   9   10   >