[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] [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 (#3). 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 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: -Code-Review -Verified -- 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 Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new patch set (#2). Change subject: Add blog post for 1.0.0 .. Add blog post for 1.0.0 Change-Id: I052ee5dea2d7a4ef3942ca3f7f93085b86d9d82e --- A _posts/2016-09-20-apache-kudu-1-0-0-released.md 1 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: > Patch Set 1: -Code-Review -Verified Dan found a typo -- To view, visit http://gerrit.cloudera.org:8080/4478 To

[kudu-CR] [twitter-demo] use 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/4477 Change subject: [twitter-demo] use AUTO_FLUSH_BACKGROUND .. [twitter-demo] use AUTO_FLUSH_BACKGROUND Changed the twitter demo

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

2016-09-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add blog post for 1.0.0 .. Patch Set 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 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] WIP: consensus: refactor tracking of received OpIds out of ReplicaState

2016-09-20 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4476 to review the following change. Change subject: WIP: consensus: refactor tracking of received OpIds out of ReplicaState

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

2016-09-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/4475 Change subject: Update kudu-lint to latest LLVM APIs .. Update kudu-lint to latest LLVM APIs Also removed the 'unused Status' matcher,

[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: We probably want to integrate tooling like this into clang-tidy as a "tidy check" in the longer run, but I

[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] [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 (#5). Change subject: [web-ui] KUDU-1619 Separate tables for live and dead tservers on /tablet-servers

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

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves 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 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 Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 1: Code-Review+2 -- 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: 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] Add support for clang from XCode 8 in macOS

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 2: Code-Review+2 I agree the malloc patch should be separate. -- To view, visit

[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] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Add support for clang from XCode 8 in macOS .. Add support for clang from XCode 8 in macOS Since upgrading XCode to 8 (even in El Capitan), cmake

[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] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/4482 Change subject: Add support for clang from XCode 8 in macOS .. Add support for clang from XCode 8 in macOS Since upgrading XCode

[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] Implement an upgrade test

2016-09-20 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Implement an upgrade test .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/4424/2//COMMIT_MSG Commit Message: PS2, Line 7: upgrade Wouldn't this

[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] [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: Yep sounds good to me. I haven't fully investigated why the change fixes the test for me. The

[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] [tests] MANUAL FLUSH --> AUTO FLUSH BACKGROUND

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

[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] 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: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4440 To

[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] [WIP] KUDU-1563. Add support for INSERT IGNORE

2016-09-20 Thread Brock Noland (Code Review)
Brock Noland has uploaded a new change for review. http://gerrit.cloudera.org:8080/4491 Change subject: [WIP] KUDU-1563. Add support for INSERT IGNORE .. [WIP] KUDU-1563. Add support for INSERT IGNORE Add's `INSERT IGNORE'

[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