David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3288
Change subject: Avoid unused var warning in deltafile-test.cc
..
Avoid unused var warning in deltafile-test.cc
Now that in memory
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1469. Fix handling of fully-deduped requests after a
leader change
..
Patch Set 4: Code-Review+2
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: RaftConsensus: Trigger election at startup if single node
..
Patch Set 2:
(1 comment)
can you add a test for this? something simple like setting the
David Ribeiro Alves has posted comments on this change.
Change subject: Add WaitUntilLeader() to Consensus interface
..
Patch Set 2:
where is this used?
--
To view, visit http://gerrit.cloudera.org:8080/3345
To unsubscribe,
David Ribeiro Alves has posted comments on this change.
Change subject: Remove the LocalConsensus implementation
..
Patch Set 3:
general wondering. Should we just kill the consensus.h interface and refactor
David Ribeiro Alves has posted comments on this change.
Change subject: Remove the LocalConsensus implementation
..
Patch Set 3:
yeah was not saying that we needed to do it here. Not sure there is much to
discuss though,
David Ribeiro Alves has posted comments on this change.
Change subject: Do not run (g)addr2line translator on MacOS X
..
Patch Set 2: Code-Review+2
Ran the tests in my mac. When running on ctest all tests that injected
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3179
to look at the new patch set (#3).
Change subject: Allow to set RequestId in the RPC RequestHeader
..
Allow to set
David Ribeiro Alves has submitted this change and it was merged.
Change subject: Add unique id generation to the client
..
Add unique id generation to the client
This adds unique id generation to the client, which takes the
David Ribeiro Alves has posted comments on this change.
Change subject: Fix stray memory writes due to tcmalloc profiling
..
Patch Set 1: Code-Review+2
good catch
--
To view, visit http://gerrit.cloudera.org:8080/3445
To
David Ribeiro Alves has posted comments on this change.
Change subject: RaftConsensus: Trigger election at startup if single node
..
Patch Set 5: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3344
To
David Ribeiro Alves has posted comments on this change.
Change subject: Use RaftConsensus instead of LocalConsensus in tests
..
Patch Set 4:
(3 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: Remove the LocalConsensus implementation
..
Patch Set 5:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/3350/5/src/kudu/consensus/metadata.proto
File
David Ribeiro Alves has posted comments on this change.
Change subject: Avoid missing 'override' keyword warnings in
raft_consensus-test.cc
..
Patch Set 2: Verified+1
unrelated failure on
David Ribeiro Alves has submitted this change and it was merged.
Change subject: Mark the Response accessors in transactions/transaction's state
const
..
Mark the Response accessors in transactions/transaction's state const
David Ribeiro Alves has posted comments on this change.
Change subject: Avoid missing 'override' keyword warnings in
raft_consensus-test.cc
..
Patch Set 2:
was looking into updating gmock and it seems like it has moved to a
David Ribeiro Alves has posted comments on this change.
Change subject: Added more to the documentation for botched installation. Users
should uninstall the dependencies and install them again after cleaning the
project.
..
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3403
to look at the new patch set (#3).
Change subject: WIP: Exactly once semantics for writes
..
WIP: Exactly once
David Ribeiro Alves has posted comments on this change.
Change subject: Integrate the ResultTracker into the rpc subsystem and add a
test
..
Patch Set 12: Verified+1
(12 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: Add a ResultTracker class that will track server side results
..
Patch Set 13: Verified+1
unrelated flake ReplicatedAlterTableTest.TestReplicatedAlter
--
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 (#12).
Change subject: Integrate the ResultTracker into the rpc subsystem and add a
test
Hello Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3179
to look at the new patch set (#9).
Change subject: Allow to set RequestId in the RPC RequestHeader
..
David Ribeiro Alves has posted comments on this change.
Change subject: Integrate the request tracker with the client
..
Patch Set 8:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3080/8//COMMIT_MSG
Commit Message:
Line
David Ribeiro Alves has posted comments on this change.
Change subject: catalog_manager: fix a locking error
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3461
To unsubscribe, visit
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1477. Pending COMMIT message for failed write operation
can prevent tablet startup
..
Patch Set 3: Code-Review+2
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: Add a ResultTracker class that will track server side results
..
Patch Set 12:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: Integrate the result tracker with writes
..
Patch Set 1:
this is good to be reviewed, but it's still missing a flag to disable it until
we've finished
David Ribeiro Alves has abandoned this change.
Change subject: WIP: Exactly once semantics for writes
..
Abandoned
superceded by other patches
--
To view, visit http://gerrit.cloudera.org:8080/3403
To unsubscribe, visit
David Ribeiro Alves has uploaded a new patch set (#8).
Change subject: Integrate the request tracker with the client
..
Integrate the request tracker with the client
This integrates the request tracker with the client, making
David Ribeiro Alves has posted comments on this change.
Change subject: Add a RpcContext::RespondFailure() method
..
Patch Set 6:
(2 comments)
bq. wonder whether it would be a clearer API to instead have a flag like
Hello Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3179
to look at the new patch set (#4).
Change subject: Allow to set RequestId in the RPC RequestHeader
..
Allow to set
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 (#8).
Change subject: Add a ResultTracker class that will track server side results
..
David Ribeiro Alves has posted comments on this change.
Change subject: Avoid missing 'override' keyword warnings in
raft_consensus-test.cc
..
Patch Set 2:
we can will do (disregard the update of this patch)
--
To view,
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3409
Change subject: Clarify the different between 'call_id' and 'request_id'
..
Clarify the different between 'call_id' and
David Ribeiro Alves has posted comments on this change.
Change subject: Allow to set RequestId in the RPC RequestHeader
..
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3179/3/src/kudu/rpc/rpc_controller.h
File
David Ribeiro Alves has posted comments on this change.
Change subject: Allow crcutil* symbols in the client
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3412/4/src/kudu/client/symbols.map
File
David Ribeiro Alves has posted comments on this change.
Change subject: Allow crcutil* symbols in the client
..
Patch Set 3:
don't follow, isn't it ok if it follows this one on the sequence?
--
To view, visit
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 (#9).
Change subject: Integrate the ResultTracker into the rpc subsystem and add a
test
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3412
Change subject: Allow crcutil* symbols in the client
..
Allow crcutil* symbols in the client
A previous patch moved PB tracing
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 (#9).
Change subject: Add a ResultTracker class that will track server side results
..
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 (#10).
Change subject: Integrate the ResultTracker into the rpc subsystem and add a
test
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 (#10).
Change subject: Add a ResultTracker class that will track server side results
David Ribeiro Alves has posted comments on this change.
Change subject: Clarify the difference between 'call_id' and 'request_id'
..
Patch Set 2:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: Allow crcutil* symbols in the client
..
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3412/3/src/kudu/client/symbols.map
File
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3191
to look at the new patch set (#10).
Change subject: Add a RpcContext::RespondFailure() method
..
Add a
David Ribeiro Alves has posted comments on this change.
Change subject: Add a RpcContext::RespondFailure() method
..
Patch Set 10: Verified+1
unrelated flake MasterTest.TestMasterMetadataConsistentDespiteFailures
--
To view,
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3408
to look at the new patch set (#4).
Change subject: Make 'num_attempts' in RequestIdPB required
..
Make
David Ribeiro Alves has posted comments on this change.
Change subject: Make 'num_attempts' in RequestIdPB required
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3408/2//COMMIT_MSG
Commit Message:
Line 15:
David Ribeiro Alves has posted comments on this change.
Change subject: RaftConsensus: Trigger election at startup if single node
..
Patch Set 4:
(2 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: Add blog post about removing LocalConsensus
..
Patch Set 1:
(3 comments)
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3403
to look at the new patch set (#2).
Change subject: WIP: Exactly once semantics for writes
..
WIP: Exactly once
David Ribeiro Alves has posted comments on this change.
Change subject: alter_schema_transaction: don't crash in ToString() when no
timestamp
..
Patch Set 1: Code-Review+2
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: Make RequestTracker not return Status on FirstIncomplete()
..
Patch Set 2:
unrelated flake
David Ribeiro Alves has posted comments on this change.
Change subject: tablet: change default bloom filter FP rate to 0.01%
..
Patch Set 1:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/3517/1/docs/release_notes.adoc
File
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 (#13).
Change subject: Integrate the request tracker with the client
..
Integrate the
David Ribeiro Alves has uploaded a new patch set (#3).
Change subject: Avoid missing 'override' keyword warnings in
raft_consensus-test.cc
..
Avoid missing 'override' keyword warnings in raft_consensus-test.cc
In this test
David Ribeiro Alves has posted comments on this change.
Change subject: Avoid missing 'override' keyword warnings in
raft_consensus-test.cc
..
Patch Set 2:
(1 comment)
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3505
to look at the new patch set (#5).
Change subject: Add a test for the integration of RequestTracker with the
client and ResultTracker with the server
David Ribeiro Alves has posted comments on this change.
Change subject: tablet: change default bloom filter FP rate to 0.01%
..
Patch Set 2: Code-Review+2
thanks for adding the additional info
--
To view, visit
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3519
Change subject: WIP: Integration test for replay cache
..
WIP: Integration test for replay cache
This adds a new integration test
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 (#5).
Change subject: Disable exactly once semantics by default and add a flag to
enable it for tests
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 (#7).
Change subject: Integrate the result tracker with writes
..
Integrate the
David Ribeiro Alves has submitted this change and it was merged.
Change subject: Avoid missing 'override' keyword warnings in
raft_consensus-test.cc
..
Avoid missing 'override' keyword warnings in raft_consensus-test.cc
In
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 (#16).
Change subject: Integrate the ResultTracker into the rpc subsystem
..
David Ribeiro Alves has uploaded a new patch set (#14).
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
David Ribeiro Alves has uploaded a new patch set (#2).
Change subject: Integrate the result tracker with writes
..
Integrate the result tracker with writes
This patch integrates the result tracker with write transactions,
David Ribeiro Alves has posted comments on this change.
Change subject: Remove the LocalConsensus implementation
..
Patch Set 6: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3350
To unsubscribe, visit
David Ribeiro Alves has submitted this change and it was merged.
Change subject: Add request id and sequence number to the rpc request header
..
Add request id and sequence number to the rpc request header
This adds a new
David Ribeiro Alves has submitted this change and it was merged.
Change subject: Move the PbTracer in rpc_context.cc to pb_util
..
Move the PbTracer in rpc_context.cc to pb_util
We'll need to use this outside of RpcContext so
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/3190
Change subject: Add a ResultTracker class that will track server side results
..
Add a ResultTracker class that will track server
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 (#2).
Change subject: Integrate the ResultTracker into the rpc subsystem and add a
test
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3191
to look at the new patch set (#2).
Change subject: Add a RpcContext::RespondFailure() method
..
Add a
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3191
to look at the new patch set (#4).
Change subject: Add a RpcContext::RespondFailure() method
..
Add a
David Ribeiro Alves has posted comments on this change.
Change subject: Add unique id generation to the client
..
Patch Set 11:
(3 comments)
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 (#4).
Change subject: Add a ResultTracker class that will track server side results
..
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
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
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
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
..
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
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
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)
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)
David Ribeiro Alves has posted comments on this change.
Change subject: Add integration tests for replay cache with writes
..
Patch Set 29:
(8 comments)
i moved these to a new itest, also moved a couple of methods out of raft
David Ribeiro Alves has uploaded a new patch set (#30).
Change subject: Add integration tests for replay cache with writes
..
Add integration tests for replay cache with writes
This adds a couple of new integration tests for
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3628
to look at the new patch set (#5).
Change subject: WIP: Add garbage collection to ResultTracker
..
WIP: Add
David Ribeiro Alves has posted comments on this change.
Change subject: c++ client: various fixes to DDL operations
..
Patch Set 8: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3608
To unsubscribe, visit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3628
to look at the new patch set (#4).
Change subject: WIP: Add garbage collection to ResultTracker
..
WIP: Add
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 15:
(1 comment)
adressed the nit, keeping the +2
David Ribeiro Alves has submitted this change and it was merged.
Change subject: Add integration tests for replay cache with writes
..
Add integration tests for replay cache with writes
This adds a couple of new integration
David Ribeiro Alves has submitted this change and it was merged.
Change subject: Integrate the ResultTracker into the rpc subsystem
..
Integrate the ResultTracker into the rpc subsystem
This integrates the ResultTracker into
David Ribeiro Alves has posted comments on this change.
Change subject: Integrate the result tracker with writes
..
Patch Set 25:
(13 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: Integrate the result tracker with writes
..
Patch Set 25:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: Move the maintenance manager to util
..
Patch Set 1:
(16 comments)
This change is required as we'll need a server-wide MM for the result tracker.
Added
David Ribeiro Alves has submitted this change and it was merged.
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
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 (#31).
Change subject: Add integration tests for replay cache with writes
..
Add
David Ribeiro Alves has posted comments on this change.
Change subject: Memory tracking for result tracker
..
Patch Set 6:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3627/6//COMMIT_MSG
Commit Message:
PS6, Line 10:
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3627
to look at the new patch set (#8).
Change subject: Memory tracking for result tracker
..
Memory tracking for
David Ribeiro Alves has posted comments on this change.
Change subject: Add integration tests for replay cache with writes
..
Patch Set 24:
(1 comment)
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 (#27).
Change subject: Integrate the result tracker with writes
..
Integrate the
1 - 100 of 259 matches
Mail list logo