Alexey Serbin has posted comments on this change.
Change subject: [client] performance optimizations
..
Patch Set 2:
(1 comment)
Thank you for the review! Will address the rest of the comments after
addressing the tests
Adar Dembo has posted comments on this change.
Change subject: [client] avoid circular deps in time-based flusher
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/4395
To unsubscribe, visit
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 (#3).
Change subject: [client] avoid circular deps in time-based flusher
..
[client]
Kudu Jenkins has posted comments on this change.
Change subject: [client] avoid circular deps in time-based flusher
..
Patch Set 3:
Build Started http://104.196.14.100/job/kudu-gerrit/3402/
--
To view, visit
Alexey Serbin has posted comments on this change.
Change subject: [client] avoid circular deps in time-based flusher
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4395/2/src/kudu/client/session-internal.cc
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 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
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++
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
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
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
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:
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
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:
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
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
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 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 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,
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 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
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
---
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
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
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
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
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
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
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
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 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
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]
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
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
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
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
Todd Lipcon has posted comments on this change.
Change subject: [docs/doxygen] removed auto-generated comments
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/4397
To unsubscribe, visit
Kudu Jenkins has posted comments on this change.
Change subject: [docs/doxygen] removed auto-generated comments
..
Patch Set 1:
Build Started http://104.196.14.100/job/kudu-gerrit/3395/
--
To view, visit
Alexey Serbin has uploaded a new change for review.
http://gerrit.cloudera.org:8080/4397
Change subject: [docs/doxygen] removed auto-generated comments
..
[docs/doxygen] removed auto-generated comments
The template
Mike Percy has posted comments on this change.
Change subject: Add two RAT excludes
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/4396
To unsubscribe, visit
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
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
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
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:
took care of all the nits
--
To view, visit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4392
to look at the new patch set (#3).
Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes
mid-copy
Kudu Jenkins has posted comments on this change.
Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes
mid-copy
..
Patch Set 3:
Build Started http://104.196.14.100/job/kudu-gerrit/3393/
--
To view,
Kudu Jenkins has posted comments on this change.
Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes
mid-copy
..
Patch Set 2:
Build Started http://104.196.14.100/job/kudu-gerrit/3392/
--
To view,
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4392
to look at the new patch set (#2).
Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes
mid-copy
Todd Lipcon has posted comments on this change.
Change subject: tool: port ts-cli
..
Patch Set 5:
Just rebased
--
To view, visit http://gerrit.cloudera.org:8080/4373
To unsubscribe, visit
Todd Lipcon has posted comments on this change.
Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool
..
Patch Set 8:
Just rebased and put this in a series with Dinesh's patch. Will commit if it
passes
--
To
Hello Adar Dembo, 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 (#14).
Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool
Kudu Jenkins has posted comments on this change.
Change subject: tool: port kudu-fs_dump, remove kudu-fs_list, fs_tool
..
Patch Set 14:
Build Started http://104.196.14.100/job/kudu-gerrit/3391/
--
To view, visit
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 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
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1135 (part 1): avoid flushing cmeta to disk twice when
voting
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/4333
Todd Lipcon has posted comments on this change.
Change subject: tool: port ts-cli
..
Patch Set 4: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/4373
To unsubscribe, visit
Mike Percy has posted comments on this change.
Change subject: Remove a spurious warning left in raft_consensus_state.cc
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/4391
To unsubscribe,
Hello Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4394
to look at the new patch set (#2).
Change subject: KUDU-1090: relax MemTracker uniqueness constraint
..
KUDU-1090:
Kudu Jenkins has posted comments on this change.
Change subject: KUDU-1090: relax MemTracker uniqueness constraint
..
Patch Set 2:
Build Started http://104.196.14.100/job/kudu-gerrit/3388/
--
To view, visit
Adar Dembo has abandoned this change.
Change subject: WIP: KUDU-1090 test
..
Abandoned
Oops, meant to squash this in with the other patch.
--
To view, visit http://gerrit.cloudera.org:8080/4393
To unsubscribe, visit
Kudu Jenkins has posted comments on this change.
Change subject: WIP: KUDU-1090 test
..
Patch Set 1:
Build Started http://104.196.14.100/job/kudu-gerrit/3386/
--
To view, visit http://gerrit.cloudera.org:8080/4393
To
Kudu Jenkins has posted comments on this change.
Change subject: KUDU-1090: relax MemTracker uniqueness constraint
..
Patch Set 1:
Build Started http://104.196.14.100/job/kudu-gerrit/3387/
--
To view, visit
Hello Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4394
to review the following change.
Change subject: KUDU-1090: relax MemTracker uniqueness constraint
..
KUDU-1090: relax
Hello Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4393
to review the following change.
Change subject: WIP: KUDU-1090 test
..
WIP: KUDU-1090 test
Change-Id:
Kudu Jenkins has posted comments on this change.
Change subject: Remove a spurious warning left in raft_consensus_state.cc
..
Patch Set 1:
Build Started http://104.196.14.100/job/kudu-gerrit/3384/
--
To view, visit
Hello Mike Percy,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4392
to review the following change.
Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes
mid-copy
..
Hello Mike Percy,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4391
to review the following change.
Change subject: Remove a spurious warning left in raft_consensus_state.cc
..
Remove a
Kudu Jenkins has posted comments on this change.
Change subject: KUDU-1605. Blocks can be incorrectly deleted if TS crashes
mid-copy
..
Patch Set 1:
Build Started http://104.196.14.100/job/kudu-gerrit/3385/
--
To view,
Mike Percy has submitted this change and it was merged.
Change subject: Take 2: fix flakiness in tablet_history_gc-itest
..
Take 2: fix flakiness in tablet_history_gc-itest
3a9ea63b9f210b64124b2d9d2b5f14ed1214f272 attempted to
Jean-Daniel Cryans has posted comments on this change.
Change subject: Blogpost describing predicate evaluation pushdown
..
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/4384/2/_posts/predicate-pushdown.md
Mike Percy has posted comments on this change.
Change subject: Take 2: fix flakiness in tablet_history_gc-itest
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/4388
To unsubscribe, visit
David Ribeiro Alves has posted comments on this change.
Change subject: [c++ client] AUTO_FLUSH_BACKGROUND optimizations
..
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/4308/2//COMMIT_MSG
Commit Message:
Todd Lipcon has submitted this change and it was merged.
Change subject: Add more release notes for new features in 1.0
..
Add more release notes for new features in 1.0
Change-Id: Ideac208aa377b52cc9910c05a9fa4d25a333d49a
Kudu Jenkins has posted comments on this change.
Change subject: Take 2: fix flakiness in tablet_history_gc-itest
..
Patch Set 1:
Build Started http://104.196.14.100/job/kudu-gerrit/3383/
--
To view, visit
Hello Mike Percy,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4388
to review the following change.
Change subject: Take 2: fix flakiness in tablet_history_gc-itest
..
Take 2: fix
David Ribeiro Alves has posted comments on this change.
Change subject: [client] performance optimizations
..
Patch Set 2:
(7 comments)
http://gerrit.cloudera.org:8080/#/c/4385/2//COMMIT_MSG
Commit Message:
Line 10: which
David Ribeiro Alves has submitted this change and it was merged.
Change subject: release_notes: improve explanation of repartitioning limitation
..
release_notes: improve explanation of repartitioning limitation
Change-Id:
Mike Percy has posted comments on this change.
Change subject: Create base class for MiniCluster and ExternalMiniCluster
..
Patch Set 10:
Filed a cpplint issue @ https://github.com/google/styleguide/issues/176
--
To view,
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 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 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
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
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
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
-
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
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
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
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,
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
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 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
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
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 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
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
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 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
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 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
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
1 - 100 of 138 matches
Mail list logo