[kudu-CR] tool: basic integration test

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: basic integration test .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2998/ -- To view, visit http://gerrit.cloudera.org:8080/4058

[kudu-CR] Actually support downgrade to version that has LocalConsensus

2016-08-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Actually support downgrade to version that has LocalConsensus .. Patch Set 1: (1 comment)

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

2016-08-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: tool: better handling for positional arguments .. tool: better handling for positional arguments This patch modifies Action to describe all of its

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

2016-08-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4013 To unsubscribe, visit

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

2016-08-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/3952/6/src/kudu/client/session-internal.cc

[kudu-CR] Revert "KUDU-1490: Show software version of tablet servers"

2016-08-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Revert "KUDU-1490: Show software version of tablet servers" .. Patch Set 1: This is necessary for _rolling downgrade_ but shouldn't be necessary for a "turn

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

2016-08-19 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 (#12). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode ..

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

2016-08-19 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 14: Build Started http://104.196.14.100/job/kudu-gerrit/2990/ -- To view, visit

[kudu-CR] Actually support downgrade to version that has LocalConsensus

2016-08-19 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/4059 Change subject: Actually support downgrade to version that has LocalConsensus .. Actually support downgrade to version that has

[kudu-CR] Actually support downgrade to version that has LocalConsensus

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Actually support downgrade to version that has LocalConsensus .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2991/ -- To view, visit

[kudu-CR] Revert "KUDU-1490: Show software version of tablet servers"

2016-08-19 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/4060 Change subject: Revert "KUDU-1490: Show software version of tablet servers" .. Revert "KUDU-1490: Show software version of tablet servers"

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

2016-08-19 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 13: Build Started http://104.196.14.100/job/kudu-gerrit/2989/ -- To view, visit

[kudu-CR] Revert "KUDU-1490: Show software version of tablet servers"

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Revert "KUDU-1490: Show software version of tablet servers" .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2992/ -- To view, visit

[kudu-CR] Forward compat for changing TSRegistrationPB

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Forward compat for changing TSRegistrationPB .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2994/ -- To view, visit

[kudu-CR] Forward compat for changing TSRegistrationPB

2016-08-19 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4062 to review the following change. Change subject: Forward compat for changing TSRegistrationPB .. Forward compat for

[kudu-CR] Remove CHECK that undos always exist

2016-08-19 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4061 to review the following change. Change subject: Remove CHECK that undos always exist .. Remove CHECK that undos

[kudu-CR] Remove CHECK that undos always exist

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Remove CHECK that undos always exist .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2993/ -- To view, visit

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

2016-08-19 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 15: Build Started http://104.196.14.100/job/kudu-gerrit/2997/ -- To view, visit

[kudu-CR] Forward compat for changing TSRegistrationPB

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Forward compat for changing TSRegistrationPB .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2995/ -- To view, visit

[kudu-CR] Forward compat for changing TSRegistrationPB

2016-08-19 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4062 to look at the new patch set (#3). Change subject: Forward compat for changing TSRegistrationPB .. Forward compat

[kudu-CR] tool: basic integration test

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: basic integration test .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2999/ -- To view, visit http://gerrit.cloudera.org:8080/4058

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

2016-08-19 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4024 to look at the new patch set (#2). Change subject: [benchmarks/tpch] introduced AUTO_FLUSH_BACKGROUND mode ..

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

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

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

2016-08-19 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: new range partitioning features blog post .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4012 To unsubscribe, visit

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

2016-08-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/3952/6/src/kudu/client/session-internal.cc

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

2016-08-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: new range partitioning features blog post .. Patch Set 5: Code-Review+1 This is not related directly to the blog post, but I found that at least in

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

2016-08-19 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: new range partitioning features blog post .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4012 To unsubscribe, visit

[kudu-CR] Forward compat for changing TSRegistrationPB

2016-08-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Forward compat for changing TSRegistrationPB .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4062/3/src/kudu/master/ts_descriptor.cc File

[kudu-CR] docs: Add instructions for upgrading to 0.10.0

2016-08-19 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4069 to review the following change. Change subject: docs: Add instructions for upgrading to 0.10.0 .. docs: Add

[kudu-CR] docs: Add instructions for upgrading to 0.10.0

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: docs: Add instructions for upgrading to 0.10.0 .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3003/ -- To view, visit

[kudu-CR] Fix release notes link for cpp-client-api

2016-08-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Fix release notes link for cpp-client-api .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4068 To unsubscribe, visit

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

2016-08-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 15: (41 comments) Thank you for the review! Will post updated version soon.

[kudu-CR] docs: Add instructions for upgrading to 0.10.0

2016-08-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: Add instructions for upgrading to 0.10.0 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4069/1/docs/installation.adoc File

[kudu-CR] KUDU-1571. Generate "deprecated" metrics in MDL

2016-08-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1571. Generate "deprecated" metrics in MDL .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3001/ -- To view, visit

[kudu-CR] KUDU-1571. Generate "deprecated" metrics in MDL

2016-08-19 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/4067 to review the following change. Change subject: KUDU-1571. Generate "deprecated" metrics in MDL ..

[kudu-CR] KUDU-1571. Generate "deprecated" metrics in MDL

2016-08-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1571. Generate "deprecated" metrics in MDL .. Patch Set 1: Code-Review-1 I would prefer we don't go down this route. It's adding complexity to our release