[kudu-CR] thirdparty: reorganize tree

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: reorganize tree .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4550/1/thirdparty/build-definitions.sh File

[kudu-CR] KUDU-1657: read-only FsManager::Open on active tablet can crash

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-1657: read-only FsManager::Open on active tablet can crash

2016-09-28 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4551 to look at the new patch set (#2). Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash

[kudu-CR] KUDU-1363: Add IN-list predicate type

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1363: Add IN-list predicate type .. Patch Set 18: (3 comments) http://gerrit.cloudera.org:8080/#/c/2986/18/src/kudu/common/column_predicate-test.cc File

[kudu-CR] [java-client] IN-list predicate

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] IN-list predicate .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4530 To unsubscribe, visit

[kudu-CR] KUDU-1657: read-only FsManager::Open on active tablet can crash

2016-09-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash .. Patch Set 2: why not include the test and only run it in slow mode? jenkins runs tests

[kudu-CR] thirdparty: fix up libtool scripts if needed

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: fix up libtool scripts if needed .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4512/2/thirdparty/build-thirdparty.sh File

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#16). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] thirdparty: build C dependencies with TSAN instrumentation too

2016-09-28 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4559 to review the following change. Change subject: thirdparty: build C dependencies with TSAN instrumentation too

[kudu-CR] KUDU-1657: read-only FsManager::Open on active tablet can crash

2016-09-28 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-1657: read-only FsManager::Open on active tablet can crash

2016-09-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash .. Patch Set 2: (1 comment) that test is going to bit rot super quickly and this seems

[kudu-CR] thirdparty: reorganize tree

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: reorganize tree .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4550/1/thirdparty/build-definitions.sh File

[kudu-CR] WIP: consensus: refactor tracking of received OpIds out of ReplicaState

2016-09-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: consensus: refactor tracking of received OpIds out of ReplicaState .. Patch Set 1: I like the consolidation and, as I had said on the mailing list,

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#14). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 2: (7 comments) http://gerrit.cloudera.org:8080/#/c/4533/1/src/kudu/tools/kudu-admin-test.cc

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 15: (5 comments) http://gerrit.cloudera.org:8080/#/c/4412/15/src/kudu/tools/CMakeLists.txt File

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#15). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] KUDU-1363: Add IN-list predicate type

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1363: Add IN-list predicate type .. Patch Set 18: (3 comments) http://gerrit.cloudera.org:8080/#/c/2986/18/src/kudu/common/column_predicate-test.cc File

[kudu-CR] [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section .. Patch Set 6: (7 comments)

[kudu-CR] KUDU-1657: read-only FsManager::Open on active tablet can crash

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash .. Patch Set 2: > that being said, maybe you could just insert the block that causes > the test

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 15: (5 comments) http://gerrit.cloudera.org:8080/#/c/4412/15/src/kudu/tools/CMakeLists.txt

[kudu-CR] KUDU-1363: Add IN-list predicate type

2016-09-28 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2986 to look at the new patch set (#19). Change subject: KUDU-1363: Add IN-list predicate type

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/4412/15/src/kudu/tools/kudu-test.cc File

[kudu-CR] thirdparty: clean up llvm cmake files too

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: thirdparty: clean up llvm cmake files too .. thirdparty: clean up llvm cmake files too The upgrade to LLVM 3.9 changed the list of targets provided in

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 13: (2 comments) http://gerrit.cloudera.org:8080/#/c/4412/12/src/kudu/tools/tool_action_test.cc

[kudu-CR] KUDU-1612 - [python] Enable setting of read mode for scanning

2016-09-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1612 - [python] Enable setting of read mode for scanning .. Patch Set 2: (1 comment)

[kudu-CR] docs: add master permanent failure recovery workflow

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: docs: add master permanent failure recovery workflow .. Patch Set 2: Verified+1 Overriding Jenkins, the build failures were due to other thirdparty patches in

[kudu-CR] docs: add master permanent failure recovery workflow

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: docs: add master permanent failure recovery workflow .. docs: add master permanent failure recovery workflow While testing this I filed KUDU-1620; this

[kudu-CR] docs: add master permanent failure recovery workflow

2016-09-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: docs: add master permanent failure recovery workflow .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4436 To unsubscribe,

[kudu-CR] KUDU-1657: read-only FsManager::Open on active tablet can crash

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash .. Patch Set 2: Two reasons: It's typically about 35 seconds to repro on ve0158; when we fix it

[kudu-CR] KUDU-1657: read-only FsManager::Open on active tablet can crash

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4551 To

[kudu-CR] thirdparty: stifle unused argument warnings when building with clang

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: stifle unused argument warnings when building with clang .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4514 To

[kudu-CR] thirdparty: use libc++ instead libstdc++ for TSAN builds

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: use libc++ instead libstdc++ for TSAN builds .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4511/2/CMakeLists.txt File

[kudu-CR] thirdparty: use libc++ instead libstdc++ for TSAN builds

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: use libc++ instead libstdc++ for TSAN builds .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/4511/2/CMakeLists.txt File

[kudu-CR] thirdparty: use libc++ instead libstdc++ for TSAN builds

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: use libc++ instead libstdc++ for TSAN builds .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4511 To unsubscribe,

[kudu-CR] c++ client: stop requiring the old gcc ABI

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: c++ client: stop requiring the old gcc ABI .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4515 To unsubscribe, visit

[kudu-CR] thirdparty: fix up libtool scripts if needed

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: fix up libtool scripts if needed .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4512 To unsubscribe, visit

[kudu-CR] KUDU-1612 - [python] Enable setting of read mode for scanning

2016-09-28 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: KUDU-1612 - [python] Enable setting of read mode for scanning .. Patch Set 3: (1 comment)

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4560 Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. [docs] Release note for

[kudu-CR] KUDU-1563. Add support for INSERT IGNORE

2016-09-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1563. Add support for INSERT IGNORE .. Patch Set 7: (8 comments) http://gerrit.cloudera.org:8080/#/c/4491/7/src/kudu/integration-tests/fuzz-itest.cc

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

2016-09-28 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: (3 comments)

[kudu-CR] thirdparty: reorganize tree

2016-09-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4550 to look at the new patch set (#2). Change subject: thirdparty: reorganize tree .. thirdparty: reorganize tree

[kudu-CR] thirdparty: fix up libtool scripts if needed

2016-09-28 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4512 to look at the new patch set (#2). Change subject: thirdparty: fix up libtool scripts if needed ..

[kudu-CR] thirdparty: use libc++ instead libstdc++ for TSAN builds

2016-09-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4511 to look at the new patch set (#2). Change subject: thirdparty: use libc++ instead libstdc++ for TSAN builds ..

[kudu-CR] [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section

2016-09-28 Thread Ninad Shringarpure (Code Review)
Ninad Shringarpure has posted comments on this change. Change subject: [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section .. Patch Set 6: (6 comments)

[kudu-CR] [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section

2016-09-28 Thread Ninad Shringarpure (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4526 to look at the new patch set (#7). Change subject: [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section

[kudu-CR] KUDU-1363: Add IN-list predicate type

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1363: Add IN-list predicate type .. Patch Set 19: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2986 To unsubscribe, visit

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 16: (7 comments) http://gerrit.cloudera.org:8080/#/c/4412/16/src/kudu/tools/kudu-tool-test.cc

[kudu-CR] KUDU-1363: Add IN-list predicate type

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1363: Add IN-list predicate type .. KUDU-1363: Add IN-list predicate type Adds support in the C++ client for providing a set of equalities for a

[kudu-CR] [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-1612 - [python] Enable setting of read mode for scanning

2016-09-28 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4520 to look at the new patch set (#3). Change subject: KUDU-1612 - [python] Enable setting of read mode for scanning ..

[kudu-CR] thirdparty: stifle unused argument warnings when building with clang

2016-09-28 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4514 to look at the new patch set (#2). Change subject: thirdparty: stifle unused argument warnings when building with clang

[kudu-CR] c++ client: stop requiring the old gcc ABI

2016-09-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4515 to look at the new patch set (#2). Change subject: c++ client: stop requiring the old gcc ABI .. c++ client: stop

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4513 to look at the new patch set (#2). Change subject: thirdparty: split into dependency groups .. thirdparty: split

[kudu-CR] thirdparty: fix up libtool scripts if needed

2016-09-28 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4512 to look at the new patch set (#3). Change subject: thirdparty: fix up libtool scripts if needed ..

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4513 to look at the new patch set (#3). Change subject: thirdparty: split into dependency groups .. thirdparty: split

[kudu-CR] thirdparty: reorganize tree

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: reorganize tree .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4550 To unsubscribe, visit

[kudu-CR] [WIP]KUDU-1640 - [python] Add IN-list predicate support

2016-09-28 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4548 to look at the new patch set (#3). Change subject: [WIP]KUDU-1640 - [python] Add IN-list predicate support ..

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4560/1/docs/release_notes.adoc File

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4560 to look at the new patch set (#3). Change subject: [docs] Release note for OperationResponse.getWriteTimestamp ..

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. Patch Set 1: (2 comments)

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4560 to look at the new patch set (#4). Change subject: [docs] Release note for OperationResponse.getWriteTimestamp ..

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4560/1/docs/release_notes.adoc

[kudu-CR] thirdparty: fix up libtool scripts if needed

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: fix up libtool scripts if needed .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4512 To unsubscribe, visit

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4560/1/docs/release_notes.adoc File

[kudu-CR] KUDU-1654 - [python] Python 3 Client Test Failure: test table column

2016-09-28 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4543 to look at the new patch set (#2). Change subject: KUDU-1654 - [python] Python 3 Client Test Failure: test_table_column

[kudu-CR] KUDU-1654 - [python] Python 3 Client Test Failure: test table column

2016-09-28 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: KUDU-1654 - [python] Python 3 Client Test Failure: test_table_column .. Patch Set 2: (1 comment)

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#17). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#18). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 16: (3 comments) http://gerrit.cloudera.org:8080/#/c/4412/16/src/kudu/tools/kudu-tool-test.cc

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4560 To unsubscribe,

[kudu-CR] KUDU-1654 - [python] Python 3 Client Test Failure: test table column

2016-09-28 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4543 to look at the new patch set (#3). Change subject: KUDU-1654 - [python] Python 3 Client Test Failure: test_table_column

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 16: (7 comments) http://gerrit.cloudera.org:8080/#/c/4412/16/src/kudu/tools/kudu-tool-test.cc

[kudu-CR] thirdparty: use libc++ instead libstdc++ for TSAN builds

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: use libc++ instead libstdc++ for TSAN builds .. Patch Set 2: Verified+1 Overriding Jenkins, flakiness in ITClient. -- To view, visit

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 17: (3 comments) http://gerrit.cloudera.org:8080/#/c/4412/16/src/kudu/tools/kudu-tool-test.cc

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#13). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-28 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 1: (13 comments) http://gerrit.cloudera.org:8080/#/c/4533/1//COMMIT_MSG Commit Message:

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-28 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#2). Change subject: [tools] Implement a manual leader_step_down for a tablet

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 12: (20 comments)

[kudu-CR] [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section

2016-09-28 Thread Ninad Shringarpure (Code Review)
Ninad Shringarpure has posted comments on this change. Change subject: [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section .. Patch Set 6: (1 comment)

[kudu-CR] c++ client: stop requiring the old gcc ABI

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: c++ client: stop requiring the old gcc ABI .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4515 To unsubscribe, visit

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/4513/2/thirdparty/build-thirdparty.sh File

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/4513/2/thirdparty/build-thirdparty.sh File

[kudu-CR] thirdparty: fix up libtool scripts if needed

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: fix up libtool scripts if needed .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4512/2/thirdparty/build-thirdparty.sh File