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

2016-07-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add weekly update for 07/25 .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/3763/1/_posts/2016-07-26-weekly-update.md File

[kudu-CR] Fix mailing list archive links to not reference incubator

2016-07-22 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Fix mailing list archive links to not reference incubator .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3725 To unsubscribe,

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

2016-07-22 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Update the overview page .. Update the overview page Long-overdue updates to the overview page, including our new ASF status, official Python support,

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

2016-07-22 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Update the overview page .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3726 To unsubscribe, visit

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

2016-07-22 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Update the overview page .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3726 To unsubscribe, visit

[kudu-CR](gh-pages) Blog post for the graduation

2016-07-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Blog post for the graduation .. Patch Set 1: Thanks for pushing. Looks like we're missing line breaks on the section headers like "Availability and Oversight",

[kudu-CR](gh-pages) Blog post for the graduation

2016-07-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Blog post for the graduation .. Patch Set 1: Mind pushing a rendered version for review? -- To view, visit http://gerrit.cloudera.org:8080/3741 To unsubscribe,

[kudu-CR](gh-pages) Blog post for the graduation

2016-07-25 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Blog post for the graduation .. Blog post for the graduation This is just a reblog with the appropriate markdown syntax. Change-Id:

[kudu-CR](gh-pages) www: Redirect kudu.i.a.o to kudu.a.o

2016-07-25 Thread Mike Percy (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3756 to review the following change. Change subject: www: Redirect kudu.i.a.o to kudu.a.o .. www: Redirect

[kudu-CR](gh-pages) www: Redirect kudu.i.a.o to kudu.a.o

2016-07-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: www: Redirect kudu.i.a.o to kudu.a.o .. Patch Set 1: Code-Review+2 Verified+1 Thanks for your vote of confidence, JD ;) -- To view, visit

[kudu-CR] [make site.h] added C++ client API documentaion

2016-07-22 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [make_site.h] added C++ client API documentaion .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3729/1/docs/support/scripts/make_site.sh File

[kudu-CR] [make site.h] added C++ client API documentaion

2016-07-22 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [make_site.h] added C++ client API documentaion .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3729 To unsubscribe, visit

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-07-28 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 1: (11 comments) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File docs/index.adoc: PS1,

[kudu-CR] Fix flakiness in RaftConsensusITest.TestReplaceChangeConfigOperation

2016-07-29 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3819 to look at the new patch set (#2). Change subject: Fix flakiness in RaftConsensusITest.TestReplaceChangeConfigOperation

[kudu-CR] Fix flakiness in RaftConsensusITest.TestReplaceChangeConfigOperation

2016-07-29 Thread Mike Percy (Code Review)
Hello Dinesh Bhat, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3819 to review the following change. Change subject: Fix flakiness in RaftConsensusITest.TestReplaceChangeConfigOperation

[kudu-CR] Fix flakiness in RaftConsensusITest.TestReplaceChangeConfigOperation

2016-07-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Fix flakiness in RaftConsensusITest.TestReplaceChangeConfigOperation .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3819/2//COMMIT_MSG Commit

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

2016-07-30 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3819 to look at the new patch set (#3). Change subject: KUDU-1548. Fix flaky RaftConsensusITest.TestReplaceChangeConfigOperation

[kudu-CR] Fix flakiness in RaftConsensusITest.TestReplaceChangeConfigOperation

2016-07-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Fix flakiness in RaftConsensusITest.TestReplaceChangeConfigOperation .. Patch Set 2: (1 comment) > RaftConsensusITest(line 594) That's fine without waiting

[kudu-CR] KUDU-1416 Upsert support for Flume sink

2016-08-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1416 Upsert support for Flume sink .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3157 To unsubscribe, visit

[kudu-CR] KUDU-1416 Upsert support for Flume sink

2016-08-01 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: KUDU-1416 Upsert support for Flume sink .. KUDU-1416 Upsert support for Flume sink This patch adds a new SimpleKeyedKuduEventProducer class that can

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

2016-08-11 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3944 to look at the new patch set (#2). Change subject: Clarify that delta compaction is for REDOs .. Clarify that

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

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Clarify that minor delta compaction is for REDOs .. Patch Set 1: (1 comment) actually i have a couple more changes i want to make

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

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy 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] C++ client: deprecating KuduPartialRow::SetString()

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: C++ client: deprecating KuduPartialRow::SetString() .. Patch Set 12: Alexey: Feel free to abandon this. The alternative patch was merged:

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

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Clarify that delta compaction is for REDOs .. Clarify that delta compaction is for REDOs This just clarifies docs and comments around delta compaction, as

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

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Timestamp::FromUint64 should return void .. Timestamp::FromUint64 should return void It does no validation, can't fail, and has the same signature as one

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

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1555. PBC Flush() method should be async .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3951 To unsubscribe, visit

[kudu-CR](gh-pages) www: Add Kudu talks to Community page

2016-08-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: www: Add Kudu talks to Community page .. Patch Set 1: Code-Review+2 Verified+1 Verified this looks good, pushing -- To view, visit

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/3868/3/src/kudu/common/partial_row-test.cc

[kudu-CR](gh-pages) www: Add Kudu talks to Community page

2016-08-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: www: Add Kudu talks to Community page .. www: Add Kudu talks to Community page Change-Id: I26fde3dabd5f08ac656f286118704bec82f27877 Reviewed-on:

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/3868/9/src/kudu/common/partial_row-test.cc

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data .. Patch Set 9: (3 comments) http://gerrit.cloudera.org:8080/#/c/3868/9/src/kudu/common/partial_row-test.cc

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data .. Patch Set 10: Code-Review+1 (1 comment) Looks good, just found one more doc nit

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

2016-08-12 Thread Mike Percy (Code Review)
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 (#6). Change subject: KUDU-236. Implement tablet history GC .. KUDU-236. Implement

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

2016-08-13 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Add regex string match assert helper macros .. Add regex string match assert helper macros These delegate to gmock but are easier to remember than:

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

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

[kudu-CR] Create tablet history GC randomized itest

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

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

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

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

2016-08-14 Thread Mike Percy (Code Review)
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-CR] Pull RETURN NOT LOG BLOCK MANAGER macro into its own file

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

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

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

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

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

[kudu-CR] Create tablet history GC randomized itest

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

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

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

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

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

[kudu-CR] Create randomized tablet history GC itest

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

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

2016-08-14 Thread Mike Percy (Code Review)
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-CR] Pull RETURN NOT LOG BLOCK MANAGER macro into its own file

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

[kudu-CR] WIP: Fix ordered iterators

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

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

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

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

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

2016-08-12 Thread Mike Percy (Code Review)
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 (#8). Change subject: KUDU-236. Implement tablet history GC .. KUDU-236. Implement

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

2016-08-12 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/3962 Change subject: Add regex string match assert helper macros .. Add regex string match assert helper macros Change-Id:

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

2016-08-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 6: (11 comments) Posting partially addressed feedback except for 2 comments in compaction.cc and 2 comments in

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

2016-08-12 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3962 to look at the new patch set (#2). Change subject: Add regex string match assert helper macros .. Add regex string

[kudu-CR](gh-pages) www: Add Spark Summit EU talk

2016-08-12 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/3949 Change subject: www: Add Spark Summit EU talk .. www: Add Spark Summit EU talk Change-Id: I575c1ee7219a96a3c019cb2fe0899dfeef66 --- M

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

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Rename raft-remote-bootstrap design doc .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3890 To unsubscribe, visit

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

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Rename raft-remote-bootstrap design doc .. Rename raft-remote-bootstrap design doc Change-Id: I1bf1de7edfd1412eb744f0c72c0e8c6db5491044 Reviewed-on:

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

2016-08-11 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3076 to look at the new patch set (#5). Change subject: WIP: KUDU-236. Implement tablet history GC .. WIP: KUDU-236.

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

2016-08-11 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3944 to review the following change. Change subject: Clarify that minor delta compaction is for REDOs .. Clarify that

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

2016-08-11 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3943 to review the following change. Change subject: Timestamp::FromUint64 should return void .. Timestamp::FromUint64

[kudu-CR](gh-pages) www: Add Spark Summit EU talk

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: www: Add Spark Summit EU talk .. www: Add Spark Summit EU talk Change-Id: I575c1ee7219a96a3c019cb2fe0899dfeef66 Reviewed-on:

[kudu-CR](gh-pages) www: Add Spark Summit EU talk

2016-08-11 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: www: Add Spark Summit EU talk .. Patch Set 1: Code-Review+2 Verified+1 Tested, looks good -- To view, visit http://gerrit.cloudera.org:8080/3949 To unsubscribe,

[kudu-CR] Add Apache Kudu logo to the source tree

2016-08-12 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: Add Apache Kudu logo to the source tree .. Abandoned We'll try to find another place for this. -- To view, visit http://gerrit.cloudera.org:8080/3701 To unsubscribe,

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

2016-08-05 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Rename Remote Bootstrap: some manual changes .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3852 To unsubscribe, visit

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

2016-08-05 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Rename Remote Bootstrap: some manual changes .. Patch Set 1: Code-Review+2 (1 comment)

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

2016-08-05 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Rename Remote Bootstrap to Tablet Copy (part 1) .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3848 To unsubscribe, visit

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-08 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3868/1//COMMIT_MSG Commit Message: Line

[kudu-CR] KUDU-1416 Upsert support for Flume sink

2016-08-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1416 Upsert support for Flume sink .. Patch Set 2: Code-Review+2 (1 comment) Looks good to me. I'm going to just rebase this myself and push it unless I run

[kudu-CR] KUDU-1416 Upsert support for Flume sink

2016-08-01 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3157 to look at the new patch set (#3). Change subject: KUDU-1416 Upsert support for Flume sink .. KUDU-1416 Upsert

[kudu-CR] make election timeout jitter more aggressive

2016-08-02 Thread Mike Percy (Code Review)
Mike Percy 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] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data .. Patch Set 2: (3 comments) Can we add basic test coverage for SetString() and SetBinary()?

[kudu-CR] [client][gutil] introduced ATTRIBUTE DEPRECATED

2016-08-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: [client][gutil] introduced ATTRIBUTE_DEPRECATED .. [client][gutil] introduced ATTRIBUTE_DEPRECATED The ATTRIBUTE_DEPRECATED macro is to mark deprecated

[kudu-CR] [client][gutil] introduced ATTRIBUTE DEPRECATED

2016-08-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [client][gutil] introduced ATTRIBUTE_DEPRECATED .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3874 To unsubscribe, visit

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/3868/3/src/kudu/common/partial_row-test.cc

[kudu-CR] KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata

2016-08-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata .. Patch Set 6: (1 comment)

[kudu-CR] Update some comments in Tablet impl and rename a method

2016-08-15 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3981 to review the following change. Change subject: Update some comments in Tablet impl and rename a method .. Update

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Mike Percy (Code Review)
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 (#13). Change subject: KUDU-236 (part 1). Implement tablet history GC ..

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

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-236 (part 2). Create randomized tablet history GC itest

2016-08-15 Thread Mike Percy (Code Review)
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 (#6). Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest

[kudu-CR] Various comment / doc improvements

2016-08-15 Thread Mike Percy (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3977 to look at the new patch set (#2). Change subject: Various comment / doc improvements .. Various

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-236 (part 1). Implement tablet history GC .. Patch Set 13: > Could you also add a GCing thread to mt-tablet-test? It's a good > way to test interactions

[kudu-CR] Bump version to 1.0.0-SNAPSHOT

2016-08-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Bump version to 1.0.0-SNAPSHOT .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4000 To unsubscribe, visit

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-236 (part 1). Implement tablet history GC .. Patch Set 16: (5 comments) http://gerrit.cloudera.org:8080/#/c/3076/16/src/kudu/tablet/compaction-test.cc File

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-16 Thread Mike Percy (Code Review)
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 (#17). Change subject: KUDU-236 (part 1). Implement tablet history GC .. KUDU-236

[kudu-CR] KUDU-236 (part 2). Create randomized tablet history GC itest

2016-08-15 Thread Mike Percy (Code Review)
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 (#7). Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest

[kudu-CR] KUDU-236 (part 2). Create randomized tablet history GC itest

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest .. Patch Set 7: I ran 3000 loops of this with 3000 rounds each and they all passed (in RELEASE mode).

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Mike Percy (Code Review)
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 (#14). Change subject: KUDU-236 (part 1). Implement tablet history GC ..

[kudu-CR] KUDU-236 (part 3). Enable tablet history GC by default

2016-08-15 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3983 to review the following change. Change subject: KUDU-236 (part 3). Enable tablet history GC by default .. KUDU-236

[kudu-CR] Add release notes for tablet history GC

2016-08-15 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3988 to review the following change. Change subject: Add release notes for tablet history GC .. Add release notes for

[kudu-CR] Another pass on 0.10.0 release notes

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Another pass on 0.10.0 release notes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3979/1/docs/release_notes.adoc File docs/release_notes.adoc:

[kudu-CR] Support downgrade to version that has LocalConsensus

2016-08-15 Thread Mike Percy (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3985 to review the following change. Change subject: Support downgrade to version that has LocalConsensus ..

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

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3976 To unsubscribe, visit

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

2016-08-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add weekly update for 8/16 .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4008 To unsubscribe, visit

[kudu-CR](branch-0.10.x) Change to non-SNAPSHOT version on 0.10.x branch

2016-08-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Change to non-SNAPSHOT version on 0.10.x branch .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4001 To unsubscribe, visit

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 3: Code-Review+2 We can address my previous feedback in a follow-up commit if we want to get the rest of

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File docs/index.adoc: PS1,

[kudu-CR] thirdparty: Revert build dir cleanup changes

2017-02-02 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5872 to review the following change. Change subject: thirdparty: Revert build dir cleanup changes .. thirdparty: Revert

[kudu-CR] tablet copy: Include local peer UUID in TabletCopyService log messages

2017-02-02 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/5873 Change subject: tablet copy: Include local peer UUID in TabletCopyService log messages .. tablet copy: Include local peer UUID in

[kudu-CR] [tools] Add missing help text from few tools

2017-02-03 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [tools] Add missing help text from few tools .. Patch Set 2: LGTM -- To view, visit http://gerrit.cloudera.org:8080/5885 To unsubscribe, visit

  1   2   3   4   5   6   7   8   9   10   >