[kudu-CR] Reorganize and fill out release notes for 1.0

2016-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Reorganize and fill out release notes for 1.0 .. Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/4369/1/docs/release_notes.adoc File

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: port ts-cli .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3367/ -- To view, visit http://gerrit.cloudera.org:8080/4373 To

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4373 to look at the new patch set (#3). Change subject: tool: port ts-cli .. tool: port ts-cli I chose to expose

[kudu-CR] Reorganize and fill out release notes for 1.0

2016-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Reorganize and fill out release notes for 1.0 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4369/1/docs/release_notes.adoc File

[kudu-CR] Add more release notes for new features in 1.0

2016-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add more release notes for new features in 1.0 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4370 To unsubscribe, visit

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 9: (14 comments) http://gerrit.cloudera.org:8080/#/c/4305/9/src/kudu/tools/kudu-tool-test.cc

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: port ts-cli .. Patch Set 2: (5 comments) > (5 comments) > > I agree from the user perspective it's nice to expose the generic > stuff in both 'master'

[kudu-CR] dist test: enable sharding on more long tests

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: dist_test: enable sharding on more long tests .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4377/2/build-support/dist_test.py File

[kudu-CR] KUDU-1604 - [python] Fix bug getting table column by index

2016-09-12 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: KUDU-1604 - [python] Fix bug getting table column by index .. Patch Set 2: (1 comment)

[kudu-CR] Change to UNIXTIME MICROS in RegexpKuduOperationsProducer

2016-09-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Change to UNIXTIME_MICROS in RegexpKuduOperationsProducer .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4383 To

[kudu-CR] KUDU-1604 - [python] Fix bug getting table column by index

2016-09-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1604 - [python] Fix bug getting table column by index .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4378 To

[kudu-CR] dist test: enable sharding on more long tests

2016-09-12 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4377 to look at the new patch set (#3). Change subject: dist_test: enable sharding on more long tests .. dist_test:

[kudu-CR] flex partitioning-itest: shard into separate cases

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: flex_partitioning-itest: shard into separate cases .. Patch Set 1: Verified+1 unrelated flakies in Java (under discussion on dev list) -- To view, visit

[kudu-CR] flex partitioning-itest: shard into separate cases

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: flex_partitioning-itest: shard into separate cases .. flex_partitioning-itest: shard into separate cases Rather than doing a loop in the main test case,

[kudu-CR](gh-pages) Blogpost describing predicate evaluation pushdown

2016-09-12 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/4384 Change subject: Blogpost describing predicate evaluation pushdown .. Blogpost describing predicate evaluation pushdown Version with

[kudu-CR] flex partitioning-itest: shard into separate cases

2016-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: flex_partitioning-itest: shard into separate cases .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4376 To unsubscribe, visit

[kudu-CR] Change to UNIXTIME MICROS in RegexpKuduOperationsProducer

2016-09-12 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/4383 Change subject: Change to UNIXTIME_MICROS in RegexpKuduOperationsProducer .. Change to UNIXTIME_MICROS in RegexpKuduOperationsProducer

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 11: (8 comments)

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 12: Build Started http://104.196.14.100/job/kudu-gerrit/3374/ -- To view, visit

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4305 to look at the new patch set (#12). Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 11: (6 comments)

[kudu-CR](gh-pages) Blogpost describing predicate evaluation pushdown

2016-09-12 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Blogpost describing predicate evaluation pushdown .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/4384/2/_posts/predicate-pushdown.md File

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 9: (2 comments)

[kudu-CR] KUDU-1604 - [python] Fix bug getting table column by index

2016-09-12 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4378 to look at the new patch set (#2). Change subject: KUDU-1604 - [python] Fix bug getting table column by index ..

[kudu-CR] [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client

2016-09-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client .. Patch Set 2: (1 comment) How did you hit this originally?

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 11: (1 comment)

[kudu-CR] Change to UNIXTIME MICROS in RegexpKuduOperationsProducer

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Change to UNIXTIME_MICROS in RegexpKuduOperationsProducer .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3372/ -- To view, visit

[kudu-CR] dist test: enable sharding on more long tests

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: dist_test: enable sharding on more long tests .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3373/ -- To view, visit

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4305 to look at the new patch set (#11). Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool

[kudu-CR] [python] Implement Scan Token API

2016-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [python] Implement Scan Token API .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/4367/5/python/kudu/tests/test_scantoken.py File

[kudu-CR] KUDU-1604 - [python] Fix bug getting table column by index

2016-09-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1604 - [python] Fix bug getting table column by index .. Patch Set 1: (1 comment)

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/3370/ -- To view, visit

[kudu-CR] Change to UNIXTIME MICROS in RegexpKuduOperationsProducer

2016-09-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Change to UNIXTIME_MICROS in RegexpKuduOperationsProducer .. Change to UNIXTIME_MICROS in RegexpKuduOperationsProducer Original patch was

[kudu-CR] dist test: enable sharding on more long tests

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: dist_test: enable sharding on more long tests .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4377 To unsubscribe, visit

[kudu-CR] [client] performance optimizations

2016-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/4385 Change subject: [client] performance optimizations .. [client] performance optimizations The major change is on

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 13: (2 comments)

[kudu-CR] [client] performance optimizations

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [client] performance optimizations .. Patch Set 2: Oh, somehow I didn't see the whole patch when I made the previous comment. Please disregrad. -- To view,

[kudu-CR] [client] performance optimizations

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [client] performance optimizations .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4385/2//COMMIT_MSG Commit Message: Line 11: a client pushes

[kudu-CR] Reorganize and fill out release notes for 1.0

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Reorganize and fill out release notes for 1.0 .. Patch Set 1: Will gave me a verbal +1 to move forward with this commit -- To view, visit

[kudu-CR] Reorganize and fill out release notes for 1.0

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Reorganize and fill out release notes for 1.0 .. Reorganize and fill out release notes for 1.0 - moved old version release notes to a separate page -

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: tool: port ts-cli .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/4373/3/src/kudu/tools/ksck_remote.cc File src/kudu/tools/ksck_remote.cc: Line

[kudu-CR] release notes: improve explanation of repartitioning limitation

2016-09-12 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: release_notes: improve explanation of repartitioning limitation .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4387 To

[kudu-CR] release notes: improve explanation of repartitioning limitation

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: release_notes: improve explanation of repartitioning limitation .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3382/ -- To view, visit

[kudu-CR] Add more release notes for new features in 1.0

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add more release notes for new features in 1.0 .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3381/ -- To view, visit

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: tool: port ts-cli .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4373 To unsubscribe, visit

[kudu-CR] [python] Implement Scan Token API

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [python] Implement Scan Token API .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/3375/ -- To view, visit

[kudu-CR] [c++ client] AUTO FLUSH BACKGROUND optimizations

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

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4305 to look at the new patch set (#13). Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 13: Build Started http://104.196.14.100/job/kudu-gerrit/3378/ -- To view, visit

[kudu-CR] release notes: improve explanation of repartitioning limitation

2016-09-12 Thread Todd Lipcon (Code Review)
Hello Will Berkeley, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4387 to review the following change. Change subject: release_notes: improve explanation of repartitioning limitation ..

[kudu-CR] [client] performance optimizations

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

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 13: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4305 To unsubscribe, visit

[kudu-CR] [c++ client] AUTO FLUSH BACKGROUND optimizations

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [c++ client] AUTO_FLUSH_BACKGROUND optimizations .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3376/ -- To view, visit

[kudu-CR] [client] performance optimizations

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [client] performance optimizations .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3379/ -- To view, visit

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4373 to look at the new patch set (#4). Change subject: tool: port ts-cli .. tool: port ts-cli I chose to expose

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: port ts-cli .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/4373/3/src/kudu/tools/ksck_remote.cc File src/kudu/tools/ksck_remote.cc: Line

[kudu-CR] [client] performance optimizations

2016-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [client] performance optimizations .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4385/2//COMMIT_MSG Commit Message: Line 11: a client pushes

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4395 to look at the new patch set (#2). Change subject: [client] avoid circular deps in time-based flusher .. [client]

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [client] avoid circular deps in time-based flusher .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4395/1/src/kudu/client/session-internal.cc

[kudu-CR] KUDU-1605. Blocks can be incorrectly deleted if TS crashes mid-copy

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes mid-copy .. KUDU-1605. Blocks can be incorrectly deleted if TS crashes mid-copy This fixes a

[kudu-CR] Add two RAT excludes

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add two RAT excludes .. Add two RAT excludes - .avsc files can't have a comment header - The HTML template used for the doxygen footer seems small enough

[kudu-CR](branch-1.0.x) Change version to non-SNAPSHOT in branch

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Change version to non-SNAPSHOT in branch .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3398/ -- To view, visit

[kudu-CR] Bump version to 1.1.0-SNAPSHOT

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/4400 Change subject: Bump version to 1.1.0-SNAPSHOT .. Bump version to 1.1.0-SNAPSHOT Change-Id: Ifd9db10a73fecbb73cebe66c16d16e9d11a98750 ---

[kudu-CR] Bump version to 1.1.0-SNAPSHOT

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Bump version to 1.1.0-SNAPSHOT .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3399/ -- To view, visit http://gerrit.cloudera.org:8080/4400

[kudu-CR](branch-1.0.x) Revert "java: fix leak of TabletClient objects in client2tablets map"

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Revert "java: fix leak of TabletClient objects in client2tablets map" .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3397/ -- To view,

[kudu-CR](branch-1.0.x) Change version to non-SNAPSHOT in branch

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/4399 Change subject: Change version to non-SNAPSHOT in branch .. Change version to non-SNAPSHOT in branch Change-Id:

[kudu-CR](branch-1.0.x) Revert "java: fix leak of TabletClient objects in client2tablets map"

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/4398 Change subject: Revert "java: fix leak of TabletClient objects in client2tablets map" .. Revert "java: fix leak of TabletClient objects

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [client] avoid circular deps in time-based flusher .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4395/1/src/kudu/client/session-internal.cc

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [client] avoid circular deps in time-based flusher .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3396/ -- To view, visit

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [client] avoid circular deps in time-based flusher .. Patch Set 2: (1 comment) I left you some comments just as you revved PS2. Not sure if you saw them.

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [client] avoid circular deps in time-based flusher .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4395/1/src/kudu/client/session-internal.h File

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [client] avoid circular deps in time-based flusher .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4395/1/src/kudu/client/session-internal.cc

[kudu-CR] [docs/doxygen] removed auto-generated comments

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [docs/doxygen] removed auto-generated comments .. [docs/doxygen] removed auto-generated comments The template auto-generated comments for doxygen

[kudu-CR] KUDU-1605. Blocks can be incorrectly deleted if TS crashes mid-copy

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes mid-copy .. Patch Set 3: Verified+1 Known-flaky java test -- To view, visit

[kudu-CR] debug-util-test: address flakiness

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: debug-util-test: address flakiness .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4346 To unsubscribe, visit

[kudu-CR] debug-util-test: address flakiness

2016-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: debug-util-test: address flakiness .. debug-util-test: address flakiness In TSAN builds, this test seems to fail sometimes due to not starting the

[kudu-CR](branch-1.0.x) Revert "java: fix leak of TabletClient objects in client2tablets map"

2016-09-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Revert "java: fix leak of TabletClient objects in client2tablets map" .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [c++ client] AUTO FLUSH BACKGROUND optimizations

2016-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] AUTO_FLUSH_BACKGROUND optimizations .. Patch Set 2: (2 comments) Thank you for the review! I posted the updated version.

[kudu-CR] [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3369/

[kudu-CR] [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client

2016-09-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client .. [java] - Synchronize the removal of the TabletClient from

[kudu-CR] KUDU-1604 - [python] Fix bug getting table column by index

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1604 - [python] Fix bug getting table column by index .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3371/ -- To view, visit

[kudu-CR] Add more release notes for new features in 1.0

2016-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add more release notes for new features in 1.0 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4370 To unsubscribe, visit

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [client] avoid circular deps in time-based flusher .. Patch Set 2: > How sure are we that this won't negatively affect the > non-auto-flush code path? i.e is

[kudu-CR](branch-1.0.x) Revert "java: fix leak of TabletClient objects in client2tablets map"

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Revert "java: fix leak of TabletClient objects in client2tablets map" .. Revert "java: fix leak of TabletClient objects in client2tablets map" This

[kudu-CR](branch-1.0.x) Change version to non-SNAPSHOT in branch

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Change version to non-SNAPSHOT in branch .. Change version to non-SNAPSHOT in branch Change-Id: Ibc73006692673591a78c1bf3a101058ad62fc014 Reviewed-on:

[kudu-CR] KUDU-1301 - [python] Tests leak tmp directory

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1301 - [python] Tests leak tmp directory .. KUDU-1301 - [python] Tests leak tmp directory In the past the python tests have been leaking tmp

[kudu-CR] KUDU-1301 - [python] Tests leak tmp directory

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1301 - [python] Tests leak tmp directory .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4372 To unsubscribe, visit

[kudu-CR](branch-1.0.x) Change version to non-SNAPSHOT in branch

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Change version to non-SNAPSHOT in branch .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4399 To unsubscribe, visit

[kudu-CR] Bump version to 1.1.0-SNAPSHOT

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Bump version to 1.1.0-SNAPSHOT .. Bump version to 1.1.0-SNAPSHOT Change-Id: Ifd9db10a73fecbb73cebe66c16d16e9d11a98750 Reviewed-on:

[kudu-CR] [c++ client] AUTO FLUSH BACKGROUND optimizations

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

[kudu-CR] [c++ client] AUTO FLUSH BACKGROUND optimizations

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [c++ client] AUTO_FLUSH_BACKGROUND optimizations .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3401/ -- To view, visit

[kudu-CR] [client] avoid circular deps in time-based flusher

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [client] avoid circular deps in time-based flusher .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3389/ -- To view, visit

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Todd Lipcon (Code Review)
Hello Dinesh Bhat, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4373 to look at the new patch set (#5). Change subject: tool: port ts-cli .. tool: port ts-cli I chose to

[kudu-CR] KUDU-1605. Blocks can be incorrectly deleted if TS crashes mid-copy

2016-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes mid-copy .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4392 To

[kudu-CR] Add two RAT excludes

2016-09-12 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4396 to review the following change. Change subject: Add two RAT excludes .. Add two RAT excludes - .avsc

[kudu-CR] Add two RAT excludes

2016-09-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add two RAT excludes .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3394/ -- To view, visit http://gerrit.cloudera.org:8080/4396 To

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool This change ports fs_dump actions under

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: tool: port ts-cli .. tool: port ts-cli I chose to expose common server functionality in new 'master' and 'tserver' modes rather than consolidating them

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list, fs tool

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool .. Patch Set 14: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4305 To unsubscribe, visit

[kudu-CR] tool: port ts-cli

2016-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: port ts-cli .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4373 To unsubscribe, visit

  1   2   >