[kudu-CR] Add initial release notes section for 1.0.0

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add initial release notes section for 1.0.0 .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2978/ -- To view, visit http://gerrit.cloudera.o

[kudu-CR] tool: properly handle -version

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: properly handle -version .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2975/ -- To view, visit http://gerrit.cloudera.org:8080/4036

[kudu-CR] Convert pbc-dump over to new tool infrastructure

2016-08-17 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4037 to review the following change. Change subject: Convert pbc-dump over to new tool infrastructure .. Convert pbc-dump ov

[kudu-CR] tool: improve help output

2016-08-17 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4038 to review the following change. Change subject: tool: improve help output .. tool: improve help output - adds blank li

[kudu-CR] tool: improve help output

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: improve help output .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2977/ -- To view, visit http://gerrit.cloudera.org:8080/4038 To un

[kudu-CR] Convert pbc-dump over to new tool infrastructure

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Convert pbc-dump over to new tool infrastructure .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2976/ -- To view, visit http://gerrit.cloud

[kudu-CR] Add initial release notes section for 1.0.0

2016-08-17 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4039 to review the following change. Change subject: Add initial release notes section for 1.0.0 .. Add initial release note

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4013/1/src/kudu/tools/tool_action.h File src/kudu/t

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Adar Dembo (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4013 to look at the new patch set (#2). Change subject: tool: better handling for positional arguments ..

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2974/ -- To view, visit http://gerrit.clouder

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: (11 comments) > Sure wish there was some test coverage on these tools... Alright, you got me. I'll

[kudu-CR] tool: properly handle -version

2016-08-17 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4036 to review the following change. Change subject: tool: properly handle -version .. tool: properly handle -version Chang

[kudu-CR] tool: properly handle -version

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: properly handle -version .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2973/ -- To view, visit http://gerrit.cloudera.org:8080/4036

[kudu-CR] Fix orphaned declarations from /table improvement

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Fix orphaned declarations from /table improvement .. Fix orphaned declarations from /table improvement Thanks to Alexey for pointing out that I forgot to

[kudu-CR] Fix orphaned declarations from /table improvement

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix orphaned declarations from /table improvement .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4023 To unsubscribe, visit http:

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2972/ -- To view, visit http://gerrit.cl

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-17 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/4034 Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Add AvroKuduEventProducer to Kudu-Flume integration This adds a K

[kudu-CR] [benchmarks/tpch] added AUTO FLUSH BACKGROUND mode

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [benchmarks/tpch] added AUTO_FLUSH_BACKGROUND mode .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2971/ -- To view, visit http://gerrit.clo

[kudu-CR] [benchmarks/tpch] added AUTO FLUSH BACKGROUND mode

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/4024 Change subject: [benchmarks/tpch] added AUTO_FLUSH_BACKGROUND mode .. [benchmarks/tpch] added AUTO_FLUSH_BACKGROUND mode Added an abilit

[kudu-CR](gh-pages) new range partitioning features blog post

2016-08-17 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: new range partitioning features blog post .. Patch Set 3: (10 comments) http://gerrit.cloudera.org:8080/#/c/4012/3/_posts/2016-08-18-new-range-partitioning-feat

[kudu-CR](gh-pages) new range partitioning features blog post

2016-08-17 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#3). Change subject: new range partitioning features blog post .. new range partitioning features blog post Change-Id: I53504d849c2aca9ff613b11e67d1533536283931 --- A _posts/2016-08-18-n

[kudu-CR](gh-pages) new range partitioning features blog post

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: new range partitioning features blog post .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4012/2/_posts/2016-08-18-new-range-partitioning-featur

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

2016-08-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 9: (22 comments) http://gerrit.cloudera.org:8080/#/c/3952/9/src/kudu/client/batcher.h File src/ku

[kudu-CR] Fix orphaned declarations from /table improvement

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Fix orphaned declarations from /table improvement .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4023 To unsubscribe, visit htt

[kudu-CR](gh-pages) new range partitioning features blog post

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: new range partitioning features blog post .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/4012/2/_posts/2016-08-18-new-range-partitioning-featu

[kudu-CR] Fix orphaned declarations from /table improvement

2016-08-17 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/4023 Change subject: Fix orphaned declarations from /table improvement .. Fix orphaned declarations from /table improvement Thanks to Alexey

[kudu-CR] Fix orphaned declarations from /table improvement

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix orphaned declarations from /table improvement .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2970/ -- To view, visit http://gerrit.clou

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4013/1/src/kudu/tools/tool_action.h File src/kudu/

[kudu-CR] improvements to /table

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: improvements to /table .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/4021/3/src/kudu/server/webui_util.h File src/kudu/server/webui_util.h:

[kudu-CR] docs: design for handling permanent master failures

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: design for handling permanent master failures .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3393 To unsubscribe, visit htt

[kudu-CR] docs: design for handling permanent master failures

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: docs: design for handling permanent master failures .. docs: design for handling permanent master failures Here's a design doc that describes how we might

[kudu-CR] improvements to /table

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: improvements to /table .. improvements to /table This is a bunch of improvements to the /table endpoint: 0. The order of elements has been made more logi

[kudu-CR](gh-pages) new range partitioning features blog post

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: new range partitioning features blog post .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/4012/2/_posts/2016-08-18-new-range-partitioning-feature

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: (4 comments) Sure wish there was some test coverage on these tools... http://gerrit.cloudera.org:8

[kudu-CR](gh-pages) new range partitioning features blog post

2016-08-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: new range partitioning features blog post .. Patch Set 2: (20 comments) New rendered version: https://github.com/danburkert/kudu/blob/gh-pages/_posts/2016-08-18-

[kudu-CR](gh-pages) new range partitioning features blog post

2016-08-17 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: new range partitioning features blog post .. new range partitioning features blog post Change-Id: I53504d849c2aca9ff613b11e67d1533536283931 --- A _posts/2016-08-18-n

[kudu-CR] improvements to /table

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: improvements to /table .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4021 To unsubscribe, visit http://gerrit.cloudera.org:8080/

[kudu-CR] improvements to /table

2016-08-17 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: improvements to /table .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/4021/1//COMMIT_MSG Commit Message: Line 29: 5. The ALL_CAPS enum names

[kudu-CR] improvements to /table

2016-08-17 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4021 to look at the new patch set (#3). Change subject: improvements to /table .. improvements to /table This is a bunc

[kudu-CR] improvements to /table

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: improvements to /table .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2969/ -- To view, visit http://gerrit.cloudera.org:8080/4021 To unsub

[kudu-CR] improvements to /table

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: improvements to /table .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4021/1//COMMIT_MSG Commit Message: Line 29: 5. The ALL_CAPS enum names fo

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

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/2968/ -- To view, visit http://gerrit.cl

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

2016-08-17 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3952 to look at the new patch set (#9). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] improvements to /table

2016-08-17 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4021 to look at the new patch set (#2). Change subject: improvements to /table .. improvements to /table This is a bunc

[kudu-CR] improvements to /table

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: improvements to /table .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2967/ -- To view, visit http://gerrit.cloudera.org:8080/4021 To unsub

[kudu-CR] KUDU-763 consensus queue metrics on followers are messed up

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-763 consensus queue metrics on followers are messed up .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3501 To unsubscribe, v

[kudu-CR] KUDU-763 consensus queue metrics on followers are messed up

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-763 consensus queue metrics on followers are messed up .. KUDU-763 consensus queue metrics on followers are messed up On follower tablet replicas, th

[kudu-CR] improvements to /table

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: improvements to /table .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2966/ -- To view, visit http://gerrit.cloudera.org:8080/4021 To unsub

[kudu-CR] improvements to /table

2016-08-17 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/4021 Change subject: improvements to /table .. improvements to /table This is a bunch of improvements to the /table endpoint: 0. The order o

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [util/monotime] added handy operators for MonoTime .. [util/monotime] added handy operators for MonoTime Change-Id: Ia8a336bc79461adebfb2d5c136d71f90efcd3

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: > (5 comments) > > There is a Go library which allows to implement rich CLI tools. If > using

[kudu-CR] [spark] Add insert-ignore, update, and delete as write options

2016-08-17 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: [spark] Add insert-ignore, update, and delete as write options .. Patch Set 2: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/4016/2/java/kudu-sp

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util/monotime] added handy operators for MonoTime .. Patch Set 5: (1 comment) Thank you for the review! http://gerrit.cloudera.org:8080/#/c/3999/5/src/kudu/ma

[kudu-CR] [spark] Add insert-ignore, update, and delete as write options

2016-08-17 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: [spark] Add insert-ignore, update, and delete as write options .. [spark] Add insert-ignore, update, and delete as write options Change-Id: I2781104c8a655da0287977b9

[kudu-CR] [spark] Add insert-ignore, update, and delete as write options

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [spark] Add insert-ignore, update, and delete as write options .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2965/ -- To view, visit http:

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

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

[kudu-CR] [spark] Add insert-ignore, update, and delete as write options

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [spark] Add insert-ignore, update, and delete as write options .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2964/ -- To view, visit http:

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [util/monotime] added handy operators for MonoTime .. Patch Set 5: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3999/5/src/kudu/master/catalog_man

[kudu-CR] [spark] Add insert-ignore, update, and delete as write options

2016-08-17 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Will Berkeley, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4016 to review the following change. Change subject: [spark] Add insert-ignore, update, and delete as write options ..

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-17 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3999 to look at the new patch set (#5). Change subject: [util/monotime] added handy operators for MonoTime .. [util/mono

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [util/monotime] added handy operators for MonoTime .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2963/ -- To view, visit http://gerrit.clo

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: Code-Review+1 (5 comments) There is a Go library which allows to implement rich CLI tools. If us