[kudu-CR] [twitter-demo] use AUTO FLUSH BACKGROUND

2016-09-20 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4477 to look at the new patch set (#2). Change subject: [twitter-demo] use AUTO_FLUSH_BACKGROUND .. [twitter-demo] use

[kudu-CR] [web-ui] KUDU-1619 Separate tables for live and dead tservers on /tablet-servers

2016-09-20 Thread Ninad Shringarpure (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4450 to look at the new patch set (#4). Change subject: [web-ui] KUDU-1619 Separate tables for live and dead tservers on /tablet-servers

[kudu-CR](gh-pages) Add blog post for 1.0.0

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add blog post for 1.0.0 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4478/1/_posts/2016-09-20-apache-kudu-1-0-0-released.md File

[kudu-CR](gh-pages) Add blog post for 1.0.0

2016-09-20 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add blog post for 1.0.0 .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4478 To unsubscribe, visit

[kudu-CR](gh-pages) Add blog post for 1.0.0

2016-09-20 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add blog post for 1.0.0 .. Patch Set 1: > Patch Set 1: -Code-Review -Verified Dan found a typo -- To view, visit http://gerrit.cloudera.org:8080/4478 To

[kudu-CR](gh-pages) Add blog post for 1.0.0

2016-09-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add blog post for 1.0.0 .. Add blog post for 1.0.0 Change-Id: I052ee5dea2d7a4ef3942ca3f7f93085b86d9d82e Reviewed-on: http://gerrit.cloudera.org:8080/4478

[kudu-CR](gh-pages) Add blog post for 1.0.0

2016-09-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/4478 Change subject: Add blog post for 1.0.0 .. Add blog post for 1.0.0 Change-Id: I052ee5dea2d7a4ef3942ca3f7f93085b86d9d82e --- A

[kudu-CR](gh-pages) Add blog post for 1.0.0

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add blog post for 1.0.0 .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4478 To unsubscribe, visit

[kudu-CR] Update kudu-lint to latest LLVM APIs

2016-09-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Update kudu-lint to latest LLVM APIs .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4475/1/build-support/tools/kudu-lint/CMakeLists.txt File

[kudu-CR] [doc] KUDU-1627 Potentially Missing words in the quick start doc

2016-09-20 Thread Ninad Shringarpure (Code Review)
Ninad Shringarpure has uploaded a new change for review. http://gerrit.cloudera.org:8080/4480 Change subject: [doc] KUDU-1627 Potentially Missing words in the quick start doc .. [doc] KUDU-1627 Potentially Missing words in

[kudu-CR] Document Impala and Spark integration known issues & limitations

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Document Impala and Spark integration known issues & limitations .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/4443/2/docs/developing.adoc

[kudu-CR] Update kudu-lint to latest LLVM APIs

2016-09-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Update kudu-lint to latest LLVM APIs .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/4475/1/build-support/tools/kudu-lint/CMakeLists.txt File

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 1: Code-Review-1 Hum, turns out there's a malloc issue with the new compiler. Sorting that out

[kudu-CR] [doc] KUDU-1630 impala tables.html link is showing as text and not actual link

2016-09-20 Thread Ninad Shringarpure (Code Review)
Ninad Shringarpure has uploaded a new change for review. http://gerrit.cloudera.org:8080/4483 Change subject: [doc] KUDU-1630 impala_tables.html link is showing as text and not actual link .. [doc] KUDU-1630

[kudu-CR] [twitter-demo] use AUTO FLUSH BACKGROUND session

2016-09-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [twitter-demo] use AUTO_FLUSH_BACKGROUND session .. Patch Set 3: > did you test this manually? I'm not sure whether jenkins even > builds this (needs oauth)

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 1: -Code-Review -- To view, visit http://gerrit.cloudera.org:8080/4482 To unsubscribe, visit

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 1: It seems this is missing a gperftools side update to work with tcmalloc:

[kudu-CR] [tools]: Keep the verbosity of CLI at FATAL and above

2016-09-20 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools]: Keep the verbosity of CLI at FATAL and above .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/4447/1/src/kudu/tools/tool_main.cc File

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
Hello Dan Burkert, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4482 to look at the new patch set (#2). Change subject: Add support for clang from XCode 8 in macOS .. Add support for

[kudu-CR] c++ client: use entire deadline when retrying single-master DDL operations

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: c++ client: use entire deadline when retrying single-master DDL operations .. c++ client: use entire deadline when retrying single-master DDL

[kudu-CR] [tests] MANUAL FLUSH --> AUTO FLUSH BACKGROUND

2016-09-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tests] MANUAL_FLUSH --> AUTO_FLUSH_BACKGROUND .. Patch Set 1: (1 comment)

[kudu-CR] cli tool: List all tablets/replica uuids with 'kudu table list'

2016-09-20 Thread Dinesh Bhat (Code Review)
Hello Dan Burkert, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4440 to look at the new patch set (#6). Change subject: cli tool: List all tablets/replica_uuids with 'kudu table list'

[kudu-CR] cli tool: List all tablets/replica uuids with 'kudu table list'

2016-09-20 Thread Dinesh Bhat (Code Review)
Hello Dan Burkert, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4481 to review the following change. Change subject: cli tool: List all tablets/replica_uuids with 'kudu table list'

[kudu-CR] cli tool: List all tablets/replica uuids with 'kudu table list'

2016-09-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: cli tool: List all tablets/replica_uuids with 'kudu table list' .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4440 To

[kudu-CR] cli tool: List all tablets/replica uuids with 'kudu table list'

2016-09-20 Thread Dinesh Bhat (Code Review)
Hello Dan Burkert, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4440 to look at the new patch set (#7). Change subject: cli tool: List all tablets/replica_uuids with 'kudu table list'

[kudu-CR] cli tool: List all tablets/replica uuids with 'kudu table list'

2016-09-20 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has abandoned this change. Change subject: cli tool: List all tablets/replica_uuids with 'kudu table list' .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4481 To unsubscribe, visit

[kudu-CR] [doc] KUDU-1627 Potentially Missing words in the quick start doc

2016-09-20 Thread Ninad Shringarpure (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4480 to look at the new patch set (#2). Change subject: [doc] KUDU-1627 Potentially Missing words in the quick start doc

[kudu-CR] [twitter-demo] use AUTO FLUSH BACKGROUND session

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [twitter-demo] use AUTO_FLUSH_BACKGROUND session .. Patch Set 3: did you test this manually? I'm not sure whether jenkins even builds this (needs oauth)

[kudu-CR] [tests] MANUAL FLUSH --> AUTO FLUSH BACKGROUND

2016-09-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tests] MANUAL_FLUSH --> AUTO_FLUSH_BACKGROUND .. Patch Set 1: (2 comments)

[kudu-CR] [doc] KUDU-1627 Potentially Missing words in the quick start doc

2016-09-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [doc] KUDU-1627 Potentially Missing words in the quick start doc .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4480/1//COMMIT_MSG Commit

[kudu-CR] [java client] Improve and hide OperationResponse#getWriteTimestamp

2016-09-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Improve and hide OperationResponse#getWriteTimestamp .. Patch Set 1: (1 comment)

[kudu-CR] [java client] Few ITClient improvements

2016-09-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Few ITClient improvements .. Patch Set 1: (6 comments)

[kudu-CR] refactor schema-design guide

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: refactor schema-design guide .. Patch Set 1: rendered: https://github.com/danburkert/kudu/blob/schema-design/docs/schema_design.adoc -- To view, visit

[kudu-CR] [tests] MANUAL FLUSH --> AUTO FLUSH BACKGROUND

2016-09-20 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4471 to look at the new patch set (#3). Change subject: [tests] MANUAL_FLUSH --> AUTO_FLUSH_BACKGROUND .. [tests]

[kudu-CR] [tests] MANUAL FLUSH --> AUTO FLUSH BACKGROUND

2016-09-20 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4471 to look at the new patch set (#4). Change subject: [tests] MANUAL_FLUSH --> AUTO_FLUSH_BACKGROUND .. [tests]

[kudu-CR] [tests] MANUAL FLUSH --> AUTO FLUSH BACKGROUND

2016-09-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tests] MANUAL_FLUSH --> AUTO_FLUSH_BACKGROUND .. Patch Set 3: (4 comments)

[kudu-CR] [tests] MANUAL FLUSH --> AUTO FLUSH BACKGROUND

2016-09-20 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4471 to look at the new patch set (#5). Change subject: [tests] MANUAL_FLUSH --> AUTO_FLUSH_BACKGROUND .. [tests]

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 6: Still going through the code changes, but wanted to note that you should add the new

[kudu-CR] [java client] Improve and hide OperationResponse#getWriteTimestamp

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4487 Change subject: [java client] Improve and hide OperationResponse#getWriteTimestamp .. [java client] Improve and hide

[kudu-CR] [java client] Few ITClient improvements

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4489 Change subject: [java client] Few ITClient improvements .. [java client] Few ITClient improvements ITClient has been flaky for a

[kudu-CR] [java client] Fix an NPE in KuduException

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4488 Change subject: [java client] Fix an NPE in KuduException .. [java client] Fix an NPE in KuduException Saw this in a Jenkins run

[kudu-CR] [java client] Fix an NPE in KuduException

2016-09-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Fix an NPE in KuduException .. Patch Set 1: (1 comment)

[kudu-CR] [java client] Fix an NPE in KuduException

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Fix an NPE in KuduException .. Patch Set 1: (1 comment)

[kudu-CR] [tests] fix tests - two cpus and fresh install

2016-09-20 Thread Brock Noland (Code Review)
Brock Noland has posted comments on this change. Change subject: [tests] fix tests - two cpus and fresh install .. Patch Set 2: That does look similar. I can look into this more. It's actually bothering me that I cannot

[kudu-CR] cli tool: List all tablets/replica uuids with 'kudu table list'

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: cli tool: List all tablets/replica_uuids with 'kudu table list' .. Patch Set 7: Just looked at the output / formatting, but LGTM -- To view, visit

[kudu-CR] [tests] fix tests - two cpus and fresh install

2016-09-20 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: [tests] fix tests - two cpus and fresh install .. Patch Set 2: I don't know much of anything about the codegen portion of Kudu, so I don't feel OK'ing even

[kudu-CR] [tests] fix tests - two cpus and fresh install

2016-09-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tests] fix tests - two cpus and fresh install .. Patch Set 2: Is this related to KUDU-1590? -- To view, visit http://gerrit.cloudera.org:8080/4446 To

[kudu-CR] [client-test] one more test for AUTO FLUSH BACKGROUND

2016-09-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/4492 Change subject: [client-test] one more test for AUTO_FLUSH_BACKGROUND .. [client-test] one more test for AUTO_FLUSH_BACKGROUND An