[kudu-CR] Add a FindPointeeOrNull method to map-util

2016-07-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a FindPointeeOrNull method to map-util .. Patch Set 1: Verified+1 unrelated failures -- To view, visit http://gerrit.cloudera.org:8080/3594 To

[kudu-CR] Add a FindPointeeOrNull method to map-util

2016-07-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a FindPointeeOrNull method to map-util .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2232/ -- To view, visit

[kudu-CR] Add a FindPointeeOrNull method to map-util

2016-07-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/3594 Change subject: Add a FindPointeeOrNull method to map-util .. Add a FindPointeeOrNull method to map-util This adds a new

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add ComputeIfAbsent methods to map-util .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2231/ -- To view, visit

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread David Ribeiro Alves (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3593 to look at the new patch set (#5). Change subject: Add ComputeIfAbsent methods to map-util .. Add

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add ComputeIfAbsent methods to map-util .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3593/4/src/kudu/gutil/map-util.h File

[kudu-CR](gh-pages) kudu flume sink blog post

2016-07-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: kudu flume sink blog post .. Patch Set 1: (27 comments) Sorry for the delay, I was out of town for a while. Because the date already passed, let's shoot for a

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add ComputeIfAbsent methods to map-util .. Patch Set 4: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3593/4/src/kudu/gutil/map-util.h File

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add ComputeIfAbsent methods to map-util .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2230/ -- To view, visit

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add ComputeIfAbsent methods to map-util .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/3593/3/src/kudu/gutil/map-util.h File

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add ComputeIfAbsent methods to map-util .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/3593/3/src/kudu/gutil/map-util.h File

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3593 to look at the new patch set (#3). Change subject: Add ComputeIfAbsent methods to map-util .. Add ComputeIfAbsent

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add ComputeIfAbsent methods to map-util .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2229/ -- To view, visit

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add ComputeIfAbsent methods to map-util .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2228/ -- To view, visit

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3593 to look at the new patch set (#2). Change subject: Add ComputeIfAbsent methods to map-util .. Add ComputeIfAbsent

[kudu-CR] Add ComputeIfAbsent methods to map-util

2016-07-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/3593 Change subject: Add ComputeIfAbsent methods to map-util .. Add ComputeIfAbsent methods to map-util This adds two new

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

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

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

2016-07-07 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 (#3). Change subject: RFC [java client] Redo how we manage exceptions .. RFC [java

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

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: RFC [java client] Redo how we manage exceptions .. Patch Set 1: > Build Failed > > http://104.196.14.100/job/kudu-gerrit/2225/ : FAILURE This is a real

[kudu-CR] master: only use tablet reports to notify master of altered tablets

2016-07-07 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: master: only use tablet reports to notify master of altered tablets .. master: only use tablet reports to notify master of altered tablets One or more

[kudu-CR] master: only use tablet reports to notify master of altered tablets

2016-07-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: master: only use tablet reports to notify master of altered tablets .. Patch Set 2: > would you mind looping alter_table-test a bit to make sure this > doesn't

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java-client] Fix a hang in TabletClient .. [java-client] Fix a hang in TabletClient TabletClient#sendRpc has been changing a lot lately, and it's

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

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: RFC [java client] Redo how we manage exceptions .. Patch Set 2: Latest patch is just a rebase since many things changed in the Java client (although I did

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

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

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

2016-07-07 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 (#2). Change subject: RFC [java client] Redo how we manage exceptions .. RFC [java

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java-client] Fix a hang in TabletClient .. Patch Set 2: Looks fine, but the cascading series of fixes to this code has left me with zero confidence that we got

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] Fix a hang in TabletClient .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3586 To unsubscribe, visit

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] Fix a hang in TabletClient .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2224/ -- To view, visit

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] Fix a hang in TabletClient .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3586/1//COMMIT_MSG Commit Message: Line 9:

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] Fix a hang in TabletClient .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3586/1//COMMIT_MSG Commit Message: Line 9: