[kudu-CR] rw mutex: add try lock methods

2016-07-11 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: rw_mutex: add try lock methods .. rw_mutex: add try lock methods I found these to be useful, so I'm exposing them in our wrapper, and beefing up kudu::shar

[kudu-CR] rw mutex: add try lock methods

2016-07-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rw_mutex: add try lock methods .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3604 To unsubscribe, visit http://gerrit.cloudera.o

[kudu-CR] rw mutex: add try lock methods

2016-07-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: rw_mutex: add try lock methods .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3604 To unsubscribe, visit http://gerrit.cloudera.o

[kudu-CR] rw mutex: add try lock methods

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: rw_mutex: add try lock methods .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2297/ -- To view, visit http://gerrit.cloudera.org:8080/3604

[kudu-CR] rw mutex: add try lock methods

2016-07-10 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3604 to look at the new patch set (#2). Change subject: rw_mutex: add try lock methods .. rw_mutex: add try lock methods

[kudu-CR] rw mutex: add try lock methods

2016-07-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: rw_mutex: add try lock methods .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3604/1/src/kudu/util/locks.h File src/kudu/util/locks.h: Line 235:

[kudu-CR] rw mutex: add try lock methods

2016-07-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rw_mutex: add try lock methods .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3604/1/src/kudu/util/locks.h File src/kudu/util/locks.h: Line 235

[kudu-CR] rw mutex: add try lock methods

2016-07-08 Thread Adar Dembo (Code Review)
Hello Dan Burkert, David Ribeiro Alves, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3604 to review the following change. Change subject: rw_mutex: add try lock methods .. rw

[kudu-CR] rw mutex: add try lock methods

2016-07-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: rw_mutex: add try lock methods .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2265/ -- To view, visit http://gerrit.cloudera.org:8080/3604