[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-19 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. [Java] Use Yetus annotations in place of

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 7: (2 comments)

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 7: (1 comment)

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-19 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 7: Code-Review+2 (2 comments)

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 7: Code-Review+1 -- To view, visit

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 7: (1 comment)

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-18 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10077 to look at the new patch set (#7). Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 6: (1 comment)

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-17 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/10077 To

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-17 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 6: Verified+1 -- To view, visit

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-17 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10077 to look at the new patch set (#5). Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 4: Code-Review+1 (2 comments) I'd also like

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 3: (3 comments)

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-16 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10077 to look at the new patch set (#3). Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-16 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 1: (1 comment)

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-16 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10077 to look at the new patch set (#2). Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-16 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10077 ) Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. Patch Set 1: (1 comment)

[kudu-CR] [Java] Use Yetus annotations in place of @VisibleForTesting

2018-04-16 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10077 Change subject: [Java] Use Yetus annotations in place of @VisibleForTesting .. [Java] Use Yetus annotations in place of