Todd Lipcon has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 13: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5045
To unsubscribe,
Todd Lipcon has submitted this change and it was merged.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
This patch changes tablet copy
Hello Dinesh Bhat, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#13).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 12:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/5045/12/src/kudu/integration-tests/ta
Todd Lipcon has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 12:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/5045/12/src/kudu/integration-tests/t
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 12: Verified+1
Apparently log-rolling-itest is flaky.
--
To view, visit http://gerrit.clo
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 6:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/5045/6/src/kudu/tserver/ts_tablet_mana
Dinesh Bhat has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 6:
(3 comments)
LGTM, couple of nits.
http://gerrit.cloudera.org:8080/#/c/5045/6/src/kud
Hello Dinesh Bhat, Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#11).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Hello Dinesh Bhat, Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#10).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Hello Dinesh Bhat, Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#9).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
...
Hello Dinesh Bhat, Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#8).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
...
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 7:
> I don't have a strong opinion as long as:
> 1) There's some way to wait for the table
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 7:
> 1) There's some way to wait for the tablet copy to finish, at least
> for tests, even
Adar Dembo has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 7:
> > I'm going to take your advice and do this. I think it will also
> > solve the test
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 7: Code-Review-1
I posted this for a test run / sanity check but I need to work on better
Hello Dinesh Bhat, Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#7).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
...
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 6:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/5045/6/src/kudu/integration-tests/tabl
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 6:
> > On second look I see what you mean; the moment we invoke the
> > callback, we respo
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 6:
> On second look I see what you mean; the moment we invoke the
> callback, we respond t
Dinesh Bhat has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 6: Code-Review+1
(3 comments)
http://gerrit.cloudera.org:8080/#/c/5045/6/src/kudu/integra
Adar Dembo has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 6:
> > If I'm understanding this correctly, the semantics of the
> > StartTabletCopy RPC r
Hello Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#6).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 2:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/5045/2/src/kudu/tserver/ts_tablet_mana
Hello Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#5).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
Mike Percy has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 4:
(4 comments)
> If I'm understanding this correctly, the semantics of the
> StartTabletC
Todd Lipcon has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5045/4/src/kudu/tserver/ts_tablet_mana
Todd Lipcon has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 4: Verified-1
i'm trying this out on a cluster that's experiencing the issue, and it crash
Adar Dembo has posted comments on this change.
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
..
Patch Set 4:
(3 comments)
Just passing through and took an interest.
If I'm understanding this corr
Hello Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5045
to look at the new patch set (#4).
Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async
30 matches
Mail list logo