[kudu-CR] [java client] Redo how we manage exceptions

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2333/ -- To view, visit

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 17: (18 comments) http://gerrit.cloudera.org:8080/#/c/3449/17/src/kudu/client/batcher.cc File

[kudu-CR] Add a ResultTracker class that will track server side results

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a ResultTracker class that will track server side results .. Patch Set 25: Todd: As we discussed I also changed the TrackRpc/ChangeDriver methods. I

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-12 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3055 to look at the new patch set (#5). Change subject: [java client] Redo how we manage exceptions .. [java client]

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3080 to look at the new patch set (#22). Change subject: Integrate the request tracker with the client .. Integrate the

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the ResultTracker into the rpc subsystem .. Patch Set 23: Build Started http://104.196.14.100/job/kudu-gerrit/2336/ -- To view, visit

[kudu-CR] Make RequestTracker not return Status on FirstIncomplete()

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Make RequestTracker not return Status on FirstIncomplete() .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/2335/ -- To view, visit

[kudu-CR] Add a ResultTracker class that will track server side results

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a ResultTracker class that will track server side results .. Patch Set 26: Build Started http://104.196.14.100/job/kudu-gerrit/2338/ -- To view, visit

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 22: Build Started http://104.196.14.100/job/kudu-gerrit/2334/ -- To view, visit

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3192 to look at the new patch set (#23). Change subject: Integrate the ResultTracker into the rpc subsystem ..

[kudu-CR] Add a ResultTracker class that will track server side results

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3190 to look at the new patch set (#26). Change subject: Add a ResultTracker class that will track server side results

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 21: Build Started http://104.196.14.100/job/kudu-gerrit/2337/ -- To view, visit

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 17: (2 comments) http://gerrit.cloudera.org:8080/#/c/3080/17/src/kudu/rpc/retriable_rpc.h File

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Patch Set 8: (3 comments)

[kudu-CR] Update docs on how to run gcovr

2016-07-12 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Update docs on how to run gcovr .. Update docs on how to run gcovr Change-Id: I494136e20452b76572d753b54fc7a095aa54a69b Reviewed-on:

[kudu-CR] Add a ResultTracker class that will track server side results

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a ResultTracker class that will track server side results .. Patch Set 22: (2 comments)

[kudu-CR] client.h: doxygen comments for C++ API

2016-07-12 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3619 to look at the new patch set (#5). Change subject: client.h: doxygen comments for C++ API .. client.h: doxygen

[kudu-CR] client.h: doxygen comments for C++ API

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: client.h: doxygen comments for C++ API .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2331/ -- To view, visit

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 5: (16 comments) Definitely make a pass over the Javadoc for: 1. Adding @throws where necessary, and 2.

[kudu-CR] client.h: doxygen comments for C++ API

2016-07-12 Thread Anonymous Coward (Code Review)
Anonymous Coward #206 has posted comments on this change. Change subject: client.h: doxygen comments for C++ API .. Patch Set 5: (1 comment) Hi Alexey, This is not a review, only testing my review subscription via this. Also

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Integrate the ResultTracker into the rpc subsystem .. Patch Set 24: (7 comments) http://gerrit.cloudera.org:8080/#/c/3192/24/src/kudu/rpc/protoc-gen-krpc.cc File

[kudu-CR] Add a ResultTracker class that will track server side results

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add a ResultTracker class that will track server side results .. Add a ResultTracker class that will track server side results This adds the initial

[kudu-CR] Add a ResultTracker class that will track server side results

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a ResultTracker class that will track server side results .. Patch Set 26: Code-Review+2 (2 comments)

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3192 to look at the new patch set (#25). Change subject: Integrate the ResultTracker into the rpc subsystem ..

[kudu-CR] Add integration tests for replay cache with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add integration tests for replay cache with writes .. Patch Set 28: Build Started http://104.196.14.100/job/kudu-gerrit/2375/ -- To view, visit

[kudu-CR] Add a way to include request ids in log-dump

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a way to include request ids in log-dump .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2369/ -- To view, visit

[kudu-CR](gh-pages) Make site tool.py support Python 2.6 and Centos 6

2016-07-12 Thread Mike Percy (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3629 to review the following change. Change subject: Make site_tool.py support Python 2.6 and Centos 6 .. Make

[kudu-CR] Add a way to include request ids in log-dump

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a way to include request ids in log-dump .. Patch Set 4: Code-Review+2 just rebases, keeping todd's earlier +2 -- To view, visit

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3506 to look at the new patch set (#11). Change subject: Disable exactly once semantics by default and add a flag to enable it for tests

[kudu-CR] WIP: Add garbage collection to ResultTracker

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/3628 Change subject: WIP: Add garbage collection to ResultTracker .. WIP: Add garbage collection to ResultTracker This still needs

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/2346/ --

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 22: Build Started http://104.196.14.100/job/kudu-gerrit/2347/ -- To view, visit

[kudu-CR] WIP: Add garbage collection to ResultTracker

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Add garbage collection to ResultTracker .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2351/ -- To view, visit

[kudu-CR] Add integration tests for replay cache with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add integration tests for replay cache with writes .. Patch Set 26: Build Started http://104.196.14.100/job/kudu-gerrit/2349/ -- To view, visit

[kudu-CR] WIP: Memory tracking for result tracker

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/3627 Change subject: WIP: Memory tracking for result tracker .. WIP: Memory tracking for result tracker Missing directed test

[kudu-CR] WIP: Memory tracking for result tracker

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Memory tracking for result tracker .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2350/ -- To view, visit

[kudu-CR](gh-pages) Make site tool.py support Python 2.6 and Centos 6

2016-07-12 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new patch set (#2). Change subject: Make site_tool.py support Python 2.6 and Centos 6 .. Make site_tool.py support Python 2.6 and Centos 6 This pulls in kudu_util.py from the Kudu master branch to get

[kudu-CR] WIP: Add garbage collection to ResultTracker

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: WIP: Add garbage collection to ResultTracker .. WIP: Add garbage collection to ResultTracker This still needs testing and hooking up to the mm. Change-Id:

[kudu-CR] WIP: Memory tracking for result tracker

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Memory tracking for result tracker .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2353/ -- To view, visit

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3192 to look at the new patch set (#24). Change subject: Integrate the ResultTracker into the rpc subsystem ..

[kudu-CR] WIP: Add garbage collection to ResultTracker

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Add garbage collection to ResultTracker .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2356/ -- To view, visit

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the ResultTracker into the rpc subsystem .. Patch Set 24: Build Started http://104.196.14.100/job/kudu-gerrit/2362/ -- To view, visit

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 23: Build Started http://104.196.14.100/job/kudu-gerrit/2354/ -- To view, visit

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 23: Build Started http://104.196.14.100/job/kudu-gerrit/2358/ -- To view, visit

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Patch Set 12: Build Started http://104.196.14.100/job/kudu-gerrit/2357/ --

[kudu-CR] Add integration tests for replay cache with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add integration tests for replay cache with writes .. Patch Set 27: Build Started http://104.196.14.100/job/kudu-gerrit/2363/ -- To view, visit

[kudu-CR] Add information about Exactly Once RPC semantics to rpc.md

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add information about Exactly Once RPC semantics to rpc.md .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2361/ -- To view, visit

[kudu-CR] WIP: Memory tracking for result tracker

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Memory tracking for result tracker .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2359/ -- To view, visit

[kudu-CR] Add a ToString() method to Proxy

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add a ToString() method to Proxy .. Add a ToString() method to Proxy ReplicatedRpc takes the server proxy type as a template argument and uses its

[kudu-CR] Add a ToString() method to Proxy

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3502 To unsubscribe, visit

[kudu-CR] Add integration tests for replay cache with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add integration tests for replay cache with writes .. Patch Set 25: Build Started http://104.196.14.100/job/kudu-gerrit/2344/ -- To view, visit

[kudu-CR] Add information about Exactly Once RPC semantics to rpc.md

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add information about Exactly Once RPC semantics to rpc.md .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/2343/ -- To view, visit

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 21: Build Started http://104.196.14.100/job/kudu-gerrit/2341/ -- To view, visit

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/2340/ --

[kudu-CR] Add information about Exactly Once RPC semantics to rpc.md

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add information about Exactly Once RPC semantics to rpc.md .. Patch Set 6: Verified+1 unrelated failure. -- To view, visit

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3080 to look at the new patch set (#24). Change subject: Integrate the request tracker with the client .. Integrate the

[kudu-CR] Add integration tests for replay cache with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add integration tests for replay cache with writes .. Patch Set 27: Verified+1 unrelated flakyness, addressed that in another patch -- To view, visit

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 24: Build Started http://104.196.14.100/job/kudu-gerrit/2364/ -- To view, visit

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 24: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2366/ -- To view, visit

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#10). Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Disable exactly once semantics by default and add a flag to enable it

[kudu-CR] Add integration tests for replay cache with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3519 to look at the new patch set (#25). Change subject: Add integration tests for replay cache with writes .. Add

[kudu-CR] Add a way to dump ResultTracker state

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3569 to look at the new patch set (#7). Change subject: Add a way to dump ResultTracker state .. Add a way to dump

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#21). Change subject: Integrate the result tracker with writes .. Integrate the

[kudu-CR] Add a way to dump ResultTracker state

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a way to dump ResultTracker state .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2342/ -- To view, visit

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 23: Verified+1 unrelated flakyness, addressed that on a previous patch -- To view, visit

[kudu-CR] master: add read-write lock to serialize operations around elections

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: master: add read-write lock to serialize operations around elections .. Patch Set 6: Code-Review+2 (1 comment)

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Patch Set 12: Verified+1 two different unrelated flakes one on the

[kudu-CR] Add information about Exactly Once RPC semantics to rpc.md

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add information about Exactly Once RPC semantics to rpc.md .. Add information about Exactly Once RPC semantics to rpc.md This patch adds some info about

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: Add a RpcContext::RespondFailure() method .. Abandoned ok, abandoning this and opting for a RespondNoCache method. Will add it in the first patch where it's used.

[kudu-CR] Make RequestTracker not return Status on FirstIncomplete()

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Make RequestTracker not return Status on FirstIncomplete() .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/2368/ -- To view, visit

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 24: Build Started http://104.196.14.100/job/kudu-gerrit/2371/ -- To view, visit

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#24). Change subject: Integrate the result tracker with writes .. Integrate the result tracker with writes This patch integrates the result tracker with write transactions,

[kudu-CR] Add a way to dump ResultTracker state

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a way to dump ResultTracker state .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/2373/ -- To view, visit

[kudu-CR] Add a fault injection point after the leader sends a request

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: Add a fault injection point _after_ the leader sends a request .. Abandoned squashed this a couple of days ago -- To view, visit

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 24: Verified+1 bunch of unrelated failures with no logs. guess Im ddos'ing the dist test system --

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Patch Set 13: (2 comments)

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Patch Set 14: Build Started http://104.196.14.100/job/kudu-gerrit/2378/ --

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 26: Build Started http://104.196.14.100/job/kudu-gerrit/2377/ -- To view, visit

[kudu-CR] Integrate the request tracker with the client

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3080 to look at the new patch set (#26). Change subject: Integrate the request tracker with the client .. Integrate the

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3192 to look at the new patch set (#26). Change subject: Integrate the ResultTracker into the rpc subsystem ..

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the ResultTracker into the rpc subsystem .. Patch Set 24: (7 comments)

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the ResultTracker into the rpc subsystem .. Patch Set 26: Build Started http://104.196.14.100/job/kudu-gerrit/2376/ -- To view, visit

[kudu-CR] rw mutex: add configurable priority

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: rw_mutex: add configurable priority .. Patch Set 3: I have experienced in the past an issue where the fairness policy causes a deadlock. See

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 07/11 .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3616 To unsubscribe, visit

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add weekly update for 07/11 .. Add weekly update for 07/11 Change-Id: I5c60067b9e548ff1f75b1cd940646ad6debf462d Reviewed-on:

[kudu-CR] master: fix corruption when AlterTable() races with CreateTable()

2016-07-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: fix corruption when AlterTable() races with CreateTable() .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3607 To

[kudu-CR] rw mutex: add configurable priority

2016-07-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rw_mutex: add configurable priority .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3603 To unsubscribe, visit

[kudu-CR] master: add read-write lock to serialize operations around elections

2016-07-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: add read-write lock to serialize operations around elections .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3550 To

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 1: (4 comments) Thanks a lot Misty for taking my original document to the finish line.

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the ResultTracker into the rpc subsystem .. Patch Set 19: (7 comments) http://gerrit.cloudera.org:8080/#/c/3192/19//COMMIT_MSG Commit Message: