[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 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/3076/7/src/kudu/tablet/compaction.cc File src/kudu/tablet/co

[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 6: (14 comments) http://gerrit.cloudera.org:8080/#/c/3076/6/src/kudu/tablet/compaction.cc File src/kudu/tablet/

[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 7: Build Started http://104.196.14.100/job/kudu-gerrit/2833/ -- To view, visit http://gerrit.cloudera.org:808

[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 (#7). Change subject: KUDU-236. Implement tablet history GC .. KUDU-236. Implement tab

[kudu-CR] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

Kudu Jenkins has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2831/ -- To view, visit http://gerrit.cloudera.org

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

Todd Lipcon has posted comments on this change. Change subject: KUDU-1553: Crash when specifying multiple unbounded range predicates on a column .. Patch Set 3: not sure why dist-test keeps having issues... I do see some SYN c

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

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 a

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

Mike Percy has posted comments on this change. Change subject: KUDU-1555. PBC Flush() method should be async .. Patch Set 2: Wow, wish we had noticed this earlier -- To view, visit http://gerrit.cloudera.org:8080/3951 To unsu

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

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 http://ger

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

Mike Percy has posted comments on this change. Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3939 To unsubscribe, vis

[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 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/3880/4/src/kudu/integration-tests/externa

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

Kudu Jenkins has posted comments on this change. Change subject: KUDU-1553: Crash when specifying multiple unbounded range predicates on a column .. Patch Set 3: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/28

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

Kudu Jenkins has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2827/ -- To view, visit http://gerrit.cl

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

Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3952 Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode Implemented A

[kudu-CR] maintenance manager: fix gflags docs

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 t

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

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

Kudu Jenkins has posted comments on this change. Change subject: KUDU-1555. PBC Flush() method should be async .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2825/ -- To view, visit http://gerrit.cloudera

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

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 http://gerrit.cloudera.org:8

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

Kudu Jenkins has posted comments on this change. Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2823/ -- To view, visit http://ge

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

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-1555.

[kudu-CR] maintenance manager: fix gflags docs

Adar Dembo has posted comments on this change. Change subject: maintenance_manager: fix gflags docs .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3950 To unsubscribe, visit http://gerrit.cloud

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

Kudu Jenkins has posted comments on this change. Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2822/ -- To view, visit http://ge

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

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] maintenance manager: fix gflags docs

Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3950 to review the following change. Change subject: maintenance_manager: fix gflags docs .. maintenance_manager: fix gflags

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

Mike Percy has posted comments on this change. Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes .. Patch Set 1: > A tiny nit: is it possible to re-format the commit message to > preserve the useful in

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

Alexey Serbin has posted comments on this change. Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes .. Patch Set 1: Code-Review+1 A tiny nit: is it possible to re-format the commit message to preserve t

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

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: http://gerrit.cloudera.org

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

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, v

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

Kudu Jenkins has posted comments on this change. Change subject: KUDU-1553: Crash when specifying multiple unbounded range predicates on a column .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2820/ -- T

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

Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3945 to look at the new patch set (#3). Change subject: KUDU-1553: Crash when specifying multiple unbounded range predicates on a column ..

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

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: I0f1d08e08d2a3d20a

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

Adar Dembo has posted comments on this change. Change subject: fs_tool: improve format for dumping a rowset .. Patch Set 1: Given that the tool will now take a different code path entirely, could you amend the commit message t

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

Kudu Jenkins has posted comments on this change. Change subject: fs_tool: improve format for dumping a rowset .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2819/ -- To view, visit http://gerrit.cloudera.

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

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 o

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

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] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

Chris George has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 3: Specifically this documentation: http://kudu.apache.org/docs/developing.html#_kudu_integration_with_spa

[kudu-CR] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

Chris George has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 3: Code-Review-1 I would prefer that we not drop CreatableRelationProvider. I'm using cassandra spark conn

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

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: https://gerrit.cloudera.o

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

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 http://gerrit.

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

Todd Lipcon has posted comments on this change. Change subject: Clarify that delta compaction is for REDOs .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3944 To unsubscribe, visit http://gerri

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

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 src/kudu/tablet/ta

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

Kudu Jenkins has posted comments on this change. Change subject: Clarify that delta compaction is for REDOs .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2818/ -- To view, visit http://gerrit.cloudera.or

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

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 http://gerrit.cloudera.org:8080/

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

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 delta

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

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 src/kudu/common/ti

[kudu-CR] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

Will Berkeley has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3871/2/java/kudu-spark/src/main/scala/org/apache/kudu/

[kudu-CR] KUDU-1553: kudu-client crash

Todd Lipcon has posted comments on this change. Change subject: KUDU-1553: kudu-client crash .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3945/2//COMMIT_MSG Commit Message: Line 12: where pred

[kudu-CR] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

Dan Burkert has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/3871/2//COMMIT_MSG Commit Message: Line 19: The change

[kudu-CR] KUDU-1553: kudu-client crash

Dan Burkert has posted comments on this change. Change subject: KUDU-1553: kudu-client crash .. Patch Set 2: Thanks for the patch, Bruce! I suspected there may be even more issues lurking here, so I wrote up a bunch of test c

[kudu-CR] KUDU-1553: kudu-client crash

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3945 to look at the new patch set (#2). Change subject: KUDU-1553: kudu-client crash .. KUDU-1553: kudu-client crash Wh

[kudu-CR] KUDU-1553: kudu-client crash

Kudu Jenkins has posted comments on this change. Change subject: KUDU-1553: kudu-client crash .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2817/ -- To view, visit http://gerrit.cloudera.org:8080/3945 To

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

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 src/kudu/common/t

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

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

[kudu-CR] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

Kudu Jenkins has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2816/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3871 to look at the new patch set (#3). Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. KUDU-1533 Spark Kud

[kudu-CR] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

Will Berkeley has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 2: (8 comments) http://gerrit.cloudera.org:8080/#/c/3871/2//COMMIT_MSG Commit Message: Line 13: for Kudu

[kudu-CR] KUDU-1553: kudu-client crash

song bruce zhang has uploaded a new change for review. http://gerrit.cloudera.org:8080/3945 Change subject: KUDU-1553: kudu-client crash .. KUDU-1553: kudu-client crash When issue sql query through impala, kudu-client will cr

[kudu-CR] KUDU-1553: kudu-client crash

Kudu Jenkins has posted comments on this change. Change subject: KUDU-1553: kudu-client crash .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2815/ -- To view, visit http://gerrit.cloudera.org:8080/3945 To

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

Kudu Jenkins has posted comments on this change. Change subject: Clarify that minor delta compaction is for REDOs .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2814/ -- To view, visit http://gerrit.cloud

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

Kudu Jenkins has posted comments on this change. Change subject: Timestamp::FromUint64 should return void .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2813/ -- To view, visit http://gerrit.cloudera.org:

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

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 minor

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

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

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

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. Impl

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

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 shou