[kudu-CR] Improve tpch usability

2016-06-01 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/3280 Change subject: Improve tpch usability .. Improve tpch usability * tpch.sh should use $ROOT to find the CPU scaling script (I had a

[kudu-CR] Fix reorder warning in TabletMetadata

2016-06-01 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/3281 Change subject: Fix reorder warning in TabletMetadata .. Fix reorder warning in TabletMetadata Fix the following warning: In file

[kudu-CR] Improve tpch usability

2016-06-01 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Improve tpch usability .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1712/ -- To view, visit http://gerrit.cloudera.org:8080/3280 To

[kudu-CR](gh-pages) No default partitioning blog post

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: No default partitioning blog post .. Patch Set 2: (1 comment) Thanks for adding the error message! Might save someone some time at some point.

[kudu-CR](gh-pages) No default partitioning blog post

2016-06-01 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: No default partitioning blog post .. No default partitioning blog post Change-Id: I14ace416b3eb5b61bb522a388e564be2e7dce3a9 --- A

[kudu-CR](gh-pages) No default partitioning blog post

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: No default partitioning blog post .. Patch Set 1: (1 comment) Is there any error message that could potentially appear? I don't remember if an empty

[kudu-CR](gh-pages) No default partitioning blog post

2016-06-01 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Misty Stanley-Jones, Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3277 to review the following change. Change subject: No default partitioning blog post

[kudu-CR](branch-0.9.x) Add a few more release notes for 0.9.0

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3276 Change subject: Add a few more release notes for 0.9.0 .. Add a few more release notes for 0.9.0 Change-Id:

[kudu-CR] Add a few more release notes for 0.9.0

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add a few more release notes for 0.9.0 .. Add a few more release notes for 0.9.0 Change-Id: Ibbd66838b51e467d4c4808fa09972d0cba717143 Reviewed-on:

[kudu-CR] Add a few more release notes for 0.9.0

2016-06-01 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Add a few more release notes for 0.9.0 .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3273 To unsubscribe, visit

[kudu-CR] Add TableCreator::add_range_split and deprecate TableCreator::split_rows

2016-06-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add TableCreator::add_range_split and deprecate TableCreator::split_rows .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/3275/2//COMMIT_MSG

[kudu-CR] Add TableCreator::add_range_split and deprecate TableCreator::split_rows

2016-06-01 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3275 to look at the new patch set (#2). Change subject: Add TableCreator::add_range_split and deprecate TableCreator::split_rows

[kudu-CR] KUDU-1308 [c++-client]: support tables with non-covering range partitions

2016-06-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1308 [c++-client]: support tables with non-covering range partitions .. Patch Set 2: (4 comments)

[kudu-CR] Replace kudu::{lock_guard, unique_lock} with std lib equivalents

2016-06-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents .. Patch Set 7: > Although I'm not overriding Adar's concern, he should chime back > in. I don't

[kudu-CR] KUDU-1308 [c++-client]: support tables with non-covering range partitions

2016-06-01 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1308 [c++-client]: support tables with non-covering range partitions .. Patch Set 2: (1 comment)

[kudu-CR] Add TableCreator::add_range_split and deprecate TableCreator::split_rows

2016-06-01 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3275 to review the following change. Change subject: Add TableCreator::add_range_split and deprecate TableCreator::split_rows

[kudu-CR] Add a few more release notes for 0.9.0

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add a few more release notes for 0.9.0 .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/3273/1/docs/release_notes.adoc File

[kudu-CR] Add a few more release notes for 0.9.0

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3273 to look at the new patch set (#2). Change subject: Add a few more release notes for 0.9.0 .. Add a few more

[kudu-CR](branch-0.9.x) KUDU-1471 NPE in AsyncKuduClient from Spark

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1471 NPE in AsyncKuduClient from Spark .. KUDU-1471 NPE in AsyncKuduClient from Spark Change-Id: Iecb0f0f50f62db3a3d5fb10f80da70fedeb2629d

[kudu-CR](branch-0.9.x) KUDU-1471 NPE in AsyncKuduClient from Spark

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1471 NPE in AsyncKuduClient from Spark .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3274 To unsubscribe, visit

[kudu-CR] Replace kudu::{lock_guard, unique_lock} with std lib equivalents

2016-06-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents .. Patch Set 7: Although I'm not overriding Adar's concern, he should chime back in. -- To view,

[kudu-CR] Replace kudu::{lock_guard, unique_lock} with std lib equivalents

2016-06-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents .. Patch Set 7: Code-Review+2 On second thought, I agree with Adar that it looks cleaner without

[kudu-CR](branch-0.9.x) KUDU-1471 NPE in AsyncKuduClient from Spark

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3274 Change subject: KUDU-1471 NPE in AsyncKuduClient from Spark .. KUDU-1471 NPE in AsyncKuduClient from Spark Change-Id:

[kudu-CR] KUDU-1471 NPE in AsyncKuduClient from Spark

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1471 NPE in AsyncKuduClient from Spark .. KUDU-1471 NPE in AsyncKuduClient from Spark Change-Id: Iecb0f0f50f62db3a3d5fb10f80da70fedeb2629d

[kudu-CR] Replace boost::{lock, unique_lock, mutex} with std lib equivalents

2016-06-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Replace boost::{lock, unique_lock, mutex} with std lib equivalents .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3262 To

[kudu-CR] Replace boost::{lock, unique_lock, mutex} with std lib equivalents

2016-06-01 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3262 to look at the new patch set (#4). Change subject: Replace boost::{lock, unique_lock, mutex} with std lib equivalents

[kudu-CR] Replace kudu::{lock_guard, unique_lock} with std lib equivalents

2016-06-01 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3271 to look at the new patch set (#6). Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents

[kudu-CR] KUDU-1308 [c++-client]: support tables with non-covering range partitions

2016-06-01 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3255 to look at the new patch set (#3). Change subject: KUDU-1308 [c++-client]: support tables with non-covering range partitions

[kudu-CR] KUDU-1471 NPE in AsyncKuduClient from Spark

2016-06-01 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1471 NPE in AsyncKuduClient from Spark .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3243 To unsubscribe, visit

[kudu-CR] KUDU-1471 NPE in AsyncKuduClient from Spark

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3243 to look at the new patch set (#4). Change subject: KUDU-1471 NPE in AsyncKuduClient from Spark .. KUDU-1471 NPE in

[kudu-CR] Replace boost::{lock, unique_lock, mutex} with std lib equivalents

2016-06-01 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Replace boost::{lock, unique_lock, mutex} with std lib equivalents .. Patch Set 3: (2 comments)

[kudu-CR] Replace kudu::{lock_guard, unique_lock} with std lib equivalents

2016-06-01 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents .. Patch Set 5: We already have quite a few std::lock_guard instances in the codebase (mostly

[kudu-CR] Replace kudu::{lock_guard, unique_lock} with std lib equivalents

2016-06-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents .. Patch Set 5: I started reviewing, then thought maybe this is the right patch to add "using

[kudu-CR] Replace boost::{lock, unique_lock, mutex} with std lib equivalents

2016-06-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Replace boost::{lock, unique_lock, mutex} with std lib equivalents .. Patch Set 3: Code-Review+2 (2 comments) Would be nice to add some "using std::lock_guard"

[kudu-CR] Replace kudu::{lock_guard, unique_lock} with std lib equivalents

2016-06-01 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3271 to look at the new patch set (#2). Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents

[kudu-CR] Replace boost::{lock, unique_lock, mutex} with std lib equivalents

2016-06-01 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Replace boost::{lock, unique_lock, mutex} with std lib equivalents .. Patch Set 2: (3 comments) It wasn't a straight find/replace because I did a lot of import

[kudu-CR] Replace boost::{lock, unique_lock, mutex} with std lib equivalents

2016-06-01 Thread Dan Burkert (Code Review)
Hello Mike Percy, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3262 to look at the new patch set (#3). Change subject: Replace boost::{lock, unique_lock, mutex} with std lib equivalents

[kudu-CR] Replace kudu::{lock_guard, unique_lock} with std lib equivalents

2016-06-01 Thread Dan Burkert (Code Review)
Hello Mike Percy, Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3271 to review the following change. Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents

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

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add weekly update for 06/01 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3269 To unsubscribe, visit

[kudu-CR](branch-0.9.x) Change flush defaults to encourage parallel IO and larger flushes

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Change flush defaults to encourage parallel IO and larger flushes .. Patch Set 1: > Oops, thought I had cherry-picked this. Good catch. Are you running >

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

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3269 Change subject: Add weekly update for 06/01 .. Add weekly update for 06/01 Change-Id: I6a930b0006fc9cbbd1da81b3286d2300fe9eb2af

[kudu-CR](branch-0.9.x) Change flush defaults to encourage parallel IO and larger flushes

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3268 Change subject: Change flush defaults to encourage parallel IO and larger flushes .. Change flush defaults to encourage parallel

[kudu-CR](branch-0.9.x) Disable tracing for tablet bootstrap during remote bootstrap

2016-06-01 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Disable tracing for tablet bootstrap during remote bootstrap .. Disable tracing for tablet bootstrap during remote bootstrap The

[kudu-CR](branch-0.9.x) Disable tracing for tablet bootstrap during remote bootstrap

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3267 Change subject: Disable tracing for tablet bootstrap during remote bootstrap .. Disable tracing for tablet bootstrap during remote

[kudu-CR](branch-0.9.x) Make 0.9.0 not SNAPSHOT for release

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Make 0.9.0 not SNAPSHOT for release .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3261 To unsubscribe, visit

[kudu-CR](branch-0.9.x) Make 0.9.0 not SNAPSHOT for release

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Make 0.9.0 not SNAPSHOT for release .. Make 0.9.0 not SNAPSHOT for release Change-Id: I912336be4cce91f8708673b32e6f18ff0022f0cf Reviewed-on:

[kudu-CR](branch-0.9.x) Add python/pytest.ini in RAT exclude

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add python/pytest.ini in RAT exclude .. Add python/pytest.ini in RAT exclude Needed just for branch-0.9.x. Change-Id: