Hello Adar Dembo, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3287
to review the following change.
Change subject: increase table_locations-itest timeout
..
increase
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3255
to look at the new patch set (#6).
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
Dan Burkert has posted comments on this change.
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
..
Patch Set 2:
(1 comment)
Dan Burkert has posted comments on this change.
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
..
Patch Set 4:
(8 comments)
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3255
to look at the new patch set (#5).
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3255
to look at the new patch set (#7).
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
Hello Misty Stanley-Jones,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3277
to look at the new patch set (#5).
Change subject: No default partitioning blog post
..
No default
Hello Jean-Daniel Cryans, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3303
to review the following change.
Change subject: Update Java client for new master GetTableLocations semantics
Dan Burkert has posted comments on this change.
Change subject: Update Java client for new master GetTableLocations semantics
..
Patch Set 1:
This change will cause a 1.0 Java client to misinterpret an empty
GetTableLocations
Dan Burkert has posted comments on this change.
Change subject: [java client] Fix a race in TabletClient cleanup
..
Patch Set 1: Code-Review+2
(1 comment)
Hello Jean-Daniel Cryans, Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3329
to look at the new patch set (#3).
Change subject: [java-client] disconnect all tablets in
TestAsyncKuduSession.disconnectAndWait
Hello Jean-Daniel Cryans, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3332
to review the following change.
Change subject: update dist-test configuration for new client.py location
Dan Burkert has posted comments on this change.
Change subject: [java-client] disconnect all tablets in
TestAsyncKuduSession.disconnectAndWait
..
Patch Set 1:
(1 comment)
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3332
to look at the new patch set (#3).
Change subject: update dist-test configuration for new client.py location
..
Dan Burkert has posted comments on this change.
Change subject: KUDU-1353: remove per-tablet replica locations cache
..
Patch Set 9:
(1 comment)
Dan Burkert has submitted this change and it was merged.
Change subject: [java-client] RPC timeout may sometimes be reported as max
attempts violation
..
[java-client] RPC timeout may sometimes be reported as max attempts
Hello Jean-Daniel Cryans, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3329
to review the following change.
Change subject: [java-client] disconnect all tablets in
TestAsyncKuduSession.disconnectAndWait
Dan Burkert has uploaded a new patch set (#2).
Change subject: [java-client] RPC timeout may sometimes be reported as max
attempts violation
..
[java-client] RPC timeout may sometimes be reported as max attempts violation
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3330
to look at the new patch set (#3).
Change subject: [java-client] RPC timeout may sometimes be reported as max
attempts violation
Dan Burkert has submitted this change and it was merged.
Change subject: Add weekly update for 06/06
..
Add weekly update for 06/06
Change-Id: I6814ef94ab522da5d1224fc5d86f975c3978c062
Reviewed-on:
Dan Burkert has posted comments on this change.
Change subject: Update Java client for new master GetTableLocations semantics
..
Patch Set 1:
I think this may be a breaking change. Still thinking about how to do it
backwards
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3131
to look at the new patch set (#10).
Change subject: Remove default table partitioning
..
Remove default table
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3131
to look at the new patch set (#11).
Change subject: Remove default table partitioning
..
Remove default table
Dan Burkert has posted comments on this change.
Change subject: Reduce verbosity of Java logs
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3203/1/java/kudu-client/src/test/resources/log4j.properties
File
Dan Burkert has submitted this change and it was merged.
Change subject: python: add support for specifying partitioning
..
python: add support for specifying partitioning
This adds support for range and hash partitioning when
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3203
to look at the new patch set (#2).
Change subject: Reduce verbosity of Java logs
..
Reduce verbosity of Java logs
Dan Burkert has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3218
Change subject: python: add support for specifying partitioning
..
python: add support for specifying partitioning
This adds support for
Dan Burkert has posted comments on this change.
Change subject: python: add support for specifying partitioning
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3196
To unsubscribe, visit
Dan Burkert has posted comments on this change.
Change subject: Update Java client for new master GetTableLocations semantics
..
Patch Set 1:
(1 comment)
Updated to make the similar change for the locate tablets codepath,
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3303
to look at the new patch set (#2).
Change subject: Update Java client for new master GetTableLocations semantics
..
Hello Jean-Daniel Cryans, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3388
to review the following change.
Change subject: KUDU-1309: [java client] support tables with non-covering
partition-key ranges
Dan Burkert has posted comments on this change.
Change subject: [java client] fix reruns of TestKuduTable.testGetLocations
..
Patch Set 2:
(1 comment)
LGTM, just the small nit.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2806
to look at the new patch set (#12).
Change subject: KUDU-1307 [master] support tables with range partition bounds
Dan Burkert has posted comments on this change.
Change subject: KUDU-1307 [master] support tables with range partition bounds
..
Patch Set 11:
(1 comment)
Dan Burkert has posted comments on this change.
Change subject: Specify guaranteed semantics of GetTableLocations RPC
..
Patch Set 1:
actually it's missing tests for a few of the guarantees, I'll add specific
tests for those.
Dan Burkert has submitted this change and it was merged.
Change subject: KUDU-1307 [master] support tables with range partition bounds
..
KUDU-1307 [master] support tables with range partition bounds
This commit adds support
Dan Burkert has posted comments on this change.
Change subject: fix compile error when compiling column_predicate-test.cc
..
Patch Set 8:
Hi Bruce,
This issue looks like a bug in Boost versions 1.56 and 1.57:
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3176
to look at the new patch set (#5).
Change subject: Kudu 0.9.0 release notes edit
..
Kudu 0.9.0 release notes edit
Dan Burkert has posted comments on this change.
Change subject: KUDU-1471 NPE in AsyncKuduClient from Spark
..
Patch Set 3:
LGTM with the caveat of updating the comment per Todd's suggestion. I think
it's worth rushing this
Hello Internal Jenkins, Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/1868
to look at the new patch set (#13).
Change subject: Replace uses of boost::thread with std::thread
Dan Burkert has posted comments on this change.
Change subject: Fix license issues
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3254
To unsubscribe, visit
Hello Internal Jenkins, Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/1868
to look at the new patch set (#14).
Change subject: Replace uses of boost::thread with std::thread
Hello Adar Dembo, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3255
to review the following change.
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3203
to look at the new patch set (#3).
Change subject: Reduce verbosity of Java logs
..
Reduce verbosity of Java logs
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3131
to look at the new patch set (#12).
Change subject: Remove default table partitioning
..
Remove default table
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3131
to look at the new patch set (#13).
Change subject: Remove default table partitioning
..
Remove default table
Dan Burkert has posted comments on this change.
Change subject: Remove default table partitioning
..
Patch Set 9:
(2 comments)
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3131
to look at the new patch set (#15).
Change subject: Remove default table partitioning
..
Remove default table
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3131
to look at the new patch set (#16).
Change subject: Remove default table partitioning
..
Remove default table
Dan Burkert has posted comments on this change.
Change subject: KUDU-1307 [master] support tables with range partition bounds
..
Patch Set 10:
(9 comments)
http://gerrit.cloudera.org:8080/#/c/2806/10//COMMIT_MSG
Commit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2806
to look at the new patch set (#11).
Change subject: KUDU-1307 [master] support tables with range partition bounds
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3255
to look at the new patch set (#2).
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
Dan Burkert has uploaded a new patch set (#4).
Change subject: No default partitioning blog post
..
No default partitioning blog post
Change-Id: I14ace416b3eb5b61bb522a388e564be2e7dce3a9
---
A
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3131
to look at the new patch set (#17).
Change subject: Remove default table partitioning
..
Remove default table
Dan Burkert has posted comments on this change.
Change subject: Remove default table partitioning
..
Patch Set 15:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3131/15/docs/schema_design.adoc
File docs/schema_design.adoc:
Dan Burkert has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3229
Change subject: Remove default table partitioning
..
Remove default table partitioning
This commit removes the current default of
Dan Burkert has posted comments on this change.
Change subject: Document advice about max columns and record size
..
Patch Set 4: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/2778
To unsubscribe, visit
Dan Burkert has posted comments on this change.
Change subject: Non-covering Range Partitions design doc
..
Patch Set 11: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/2772
To unsubscribe, visit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3176
to look at the new patch set (#4).
Change subject: Kudu 0.9.0 release notes edit
..
Kudu 0.9.0 release notes edit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3275
to look at the new patch set (#2).
Change subject: Add TableCreator::add_range_split and deprecate
TableCreator::split_rows
Hello Jean-Daniel Cryans, Misty Stanley-Jones, Adar Dembo, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3277
to review the following change.
Change subject: No default partitioning blog post
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3271
to look at the new patch set (#2).
Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents
Dan Burkert has posted comments on this change.
Change subject: Replace boost::{lock, unique_lock, mutex} with std lib
equivalents
..
Patch Set 2:
(3 comments)
It wasn't a straight find/replace because I did a lot of import
Hello Mike Percy, Adar Dembo, Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3262
to look at the new patch set (#3).
Change subject: Replace boost::{lock, unique_lock, mutex} with std lib
equivalents
Hello Mike Percy, Adar Dembo, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3271
to review the following change.
Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents
Dan Burkert has posted comments on this change.
Change subject: Replace boost::{lock, unique_lock, mutex} with std lib
equivalents
..
Patch Set 3:
(2 comments)
Hello Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3262
to look at the new patch set (#4).
Change subject: Replace boost::{lock, unique_lock, mutex} with std lib
equivalents
Dan Burkert has posted comments on this change.
Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents
..
Patch Set 5:
We already have quite a few std::lock_guard instances in the codebase (mostly
Dan Burkert has posted comments on this change.
Change subject: KUDU-1471 NPE in AsyncKuduClient from Spark
..
Patch Set 4: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3243
To unsubscribe, visit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3271
to look at the new patch set (#6).
Change subject: Replace kudu::{lock_guard, unique_lock} with std lib equivalents
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3255
to look at the new patch set (#3).
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
Hello Adar Dembo, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3275
to review the following change.
Change subject: Add TableCreator::add_range_split and deprecate
TableCreator::split_rows
Dan Burkert has posted comments on this change.
Change subject: KUDU-1308 [c++-client]: support tables with non-covering range
partitions
..
Patch Set 2:
(1 comment)
Dan Burkert has uploaded a new patch set (#2).
Change subject: No default partitioning blog post
..
No default partitioning blog post
Change-Id: I14ace416b3eb5b61bb522a388e564be2e7dce3a9
---
A
Dan Burkert has posted comments on this change.
Change subject: KUDU-1470 Exceptions on getting a column value should return
the column name not the column number
..
Patch Set 3: Code-Review+2
Looks good, I'm retrying the
Dan Burkert has posted comments on this change.
Change subject: Update Java client for new master GetTableLocations semantics
..
Patch Set 2:
(1 comment)
The changes I made today have been made significantly simpler by
Dan Burkert has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3386
Change subject: [java-client] use tablet cache for locateTablet calls
..
[java-client] use tablet cache for locateTablet calls
This
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3303
to look at the new patch set (#3).
Change subject: Update Java client for new master GetTableLocations semantics
..
Hello Jean-Daniel Cryans, Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3386
to look at the new patch set (#5).
Change subject: [java-client] use tablet cache for locateTablet calls
Dan Burkert has posted comments on this change.
Change subject: Add required Debian version to installation page
..
Patch Set 1:
I thought you had? Debian is mentioned along with Ubuntu everywhere in the
docs, except in the
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3303
to look at the new patch set (#4).
Change subject: Update Java client for new master GetTableLocations semantics
..
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3386
to look at the new patch set (#2).
Change subject: [java-client] use tablet cache for locateTablet calls
..
Dan Burkert has uploaded a new patch set (#2).
Change subject: KUDU-1309: [java client] support tables with non-covering
partition-key ranges
..
KUDU-1309: [java client] support tables with non-covering partition-key ranges
Dan Burkert has posted comments on this change.
Change subject: Update Java client for new master GetTableLocations semantics
..
Patch Set 4:
(2 comments)
Hello Jean-Daniel Cryans, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3389
to review the following change.
Change subject: Add required Debian version to installation page
..
Dan Burkert has posted comments on this change.
Change subject: Add required Debian version to installation page
..
Patch Set 1:
Ah, so we should probably hold of on this till we can verify that it works. Or
we should remove
Hello Jean-Daniel Cryans, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3386
to look at the new patch set (#3).
Change subject: [java-client] use tablet cache for locateTablet calls
Dan Burkert has posted comments on this change.
Change subject: [java-client] use tablet cache for locateTablet calls
..
Patch Set 2:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/3386/2//COMMIT_MSG
Commit Message:
PS2,
Dan Burkert has submitted this change and it was merged.
Change subject: Add required Debian version to installation page
..
Add required Debian version to installation page
Change-Id: I65cef323648d7934cb140046cf391d56867eeb56
Dan Burkert has posted comments on this change.
Change subject: Add required Debian version to installation page
..
Patch Set 1: -Code-Review
JD, are you ok with pushing this now with your recent deb 8 testing?
--
To view,
Hello Jean-Daniel Cryans, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3477
to review the following change.
Change subject: [java-client] refactor AsyncKuduSession
..
Hello Jean-Daniel Cryans, Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3388
to look at the new patch set (#3).
Change subject: KUDU-1309: [java client] support tables with non-covering
partition-key ranges
Dan Burkert has posted comments on this change.
Change subject: thirdparty: add boost and switch to header-only build
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3500
To unsubscribe, visit
Dan Burkert has posted comments on this change.
Change subject: KUDU-1496. NoLeaderMasterFoundException are mishandled
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3516
To unsubscribe, visit
Dan Burkert has posted comments on this change.
Change subject: KUDU-1496. NoLeaderMasterFoundException are mishandled
..
Patch Set 1:
(1 comment)
Dan Burkert has posted comments on this change.
Change subject: locks: switch from boost::shared_mutex to new read-write mutex
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:8080/3499
To
Dan Burkert has posted comments on this change.
Change subject: locks: change kudu::shared_lock constructor to pass by ref
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3497
To unsubscribe,
Dan Burkert has posted comments on this change.
Change subject: locks: change kudu::shared_lock constructor to pass by ref
..
Patch Set 2: Code-Review+1
LGTM except for Todd's comment
--
To view, visit
Dan Burkert has posted comments on this change.
Change subject: thirdparty: add boost and switch to header-only build
..
Patch Set 1:
(5 comments)
I'm +1 on moving boost into thirdparty. I like that it removes complexity
Dan Burkert has posted comments on this change.
Change subject: locks: add new read-write mutex
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3496/2/src/kudu/util/rw_mutex.cc
File src/kudu/util/rw_mutex.cc:
1 - 100 of 196 matches
Mail list logo