[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Start a background thread to run ResultTracker GC .. Start a background thread to run ResultTracker GC Change-Id: Ia34ce95e78920596eb8b9db53643845f637c8e6

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3961 To unsubscribe, visit http:/

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3961 to look at the new patch set (#5). Change subject: Start a background thread to run ResultTracker GC ..

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2928/ -- To view, visit http://gerrit.clou

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3961 To unsubscribe, visit http:/

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2924/ -- To view, visit http://gerrit.clou

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3961 to look at the new patch set (#4). Change subject: Start a background thread to run ResultTracker GC .. Start a bac

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3961/3/src/kudu/rpc/result_tracker.cc File src/ku

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3961/3/src/kudu/rpc/result_tracker.cc File src/k

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3961/3/src/kudu/rpc/result_tracker.cc File src/ku

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2891/ -- To view, visit http://gerrit.clou

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-14 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3961 to look at the new patch set (#3). Change subject: Start a background thread to run ResultTracker GC .. Start a bac

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-14 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 2: (1 comment) will try to write a test. fwiw I've been testing this on a cluster ~2 days now and

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 2: (1 comment) How about a unit test to prove that it does what it says on the tin? http://gerrit.c

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 2: I looped exactly_once_writes-itest with this patch 1000 times (100% pass): http://dist-test.clou

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2855/ -- To view, visit http://gerrit.clou

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2849/ -- To view, visit http://gerrit.clou

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3961 Change subject: Start a background thread to run ResultTracker GC .. Start a background thread to run ResultTracker GC Change-Id: Ia34ce95