[kudu-CR] KUDU-2542: add initial authorization token impl

2018-10-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11750 ) Change subject: KUDU-2542: add initial authorization token impl .. Patch Set 3: (2 comments)

[kudu-CR] KUDU-2542: add initial authorization token impl

2018-10-24 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11750 to look at the new patch set (#3). Change subject: KUDU-2542: add initial authorization token impl

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [client] expose location via internal API .. Patch Set 8: Code-Review+2 -- To view, visit

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11727 to look at the new patch set (#7). Change subject: [location_awareness] Add ts location to TSInfoPB

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 6: (1 comment)

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [client] expose location via internal API .. Patch Set 7: (1 comment)

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#8). Change subject: [client] expose location via internal API

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [client] expose location via internal API .. Patch Set 7: (1 comment) Looks good: just one small nit.

[kudu-CR] [c++ client] clean up inline doxygen documentation

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11780 ) Change subject: [c++ client] clean up inline doxygen documentation .. [c++ client] clean up inline doxygen documentation This

[kudu-CR] [c++ client] KuduTable::ListPartitions() is private API

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11779 ) Change subject: [c++ client] KuduTable::ListPartitions() is private API .. [c++ client] KuduTable::ListPartitions() is private

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [client] expose location via internal API .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 6: Code-Review+1 Will defer to Alexey. -- To view, visit

[kudu-CR] [c++ client] clean up inline doxygen documentation

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11780 ) Change subject: [c++ client] clean up inline doxygen documentation .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [c++ client] KuduTable::ListPartitions() is private API

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11779 ) Change subject: [c++ client] KuduTable::ListPartitions() is private API .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 6: (1 comment) Looks good, just one small nit.

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [client] expose location via internal API .. Patch Set 2: (1 comment)

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#7). Change subject: [client] expose location via internal API

[kudu-CR] [c++ client] clean up inline doxygen documentation

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11780 ) Change subject: [c++ client] clean up inline doxygen documentation .. Patch Set 1: (1 comment)

[kudu-CR] [c++ client] KuduTable::ListPartitions() is private API

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11779 ) Change subject: [c++ client] KuduTable::ListPartitions() is private API .. Patch Set 1: Verified+1 Unrelated flake in

[kudu-CR] [c++ client] KuduTable::ListPartitions() is private API

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11779 ) Change subject: [c++ client] KuduTable::ListPartitions() is private API .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] [c++ client] clean up inline doxygen documentation

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11780 Change subject: [c++ client] clean up inline doxygen documentation .. [c++ client] clean up inline doxygen documentation This patch

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11727 to look at the new patch set (#6). Change subject: [location_awareness] Add ts location to TSInfoPB

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 6: (6 comments)

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11772 ) Change subject: Updating web site for Kudu 1.8.0 release .. Patch Set 4: (1 comment)

[kudu-CR] [c++ client] KuduTable::ListPartitions() is private API

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11779 Change subject: [c++ client] KuduTable::ListPartitions() is private API .. [c++ client] KuduTable::ListPartitions() is private API

[kudu-CR] KUDU-2542: add initial authorization token impl

2018-10-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11750 ) Change subject: KUDU-2542: add initial authorization token impl .. Patch Set 2: Failures are in LINT and IWYU. -- To view, visit

[kudu-CR] KUDU-2542: add initial authorization token impl

2018-10-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11750 ) Change subject: KUDU-2542: add initial authorization token impl .. Patch Set 2: (1 comment)

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Hello Mike Percy, Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Andrew Wong, helifu, Adar Dembo, Anonymous Coward (314), Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11772 to look at the new patch set (#4). Change subject:

[kudu-CR] KUDU-2542: add initial authorization token impl

2018-10-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11750 ) Change subject: KUDU-2542: add initial authorization token impl .. Patch Set 2: (26 comments)

[kudu-CR] KUDU-2542: add initial authorization token impl

2018-10-24 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11750 to look at the new patch set (#2). Change subject: KUDU-2542: add initial authorization token impl

[kudu-CR] docs: Add scripts to automate web site update for releases

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/8800 ) Change subject: docs: Add scripts to automate web site update for releases .. Patch Set 1: (1 comment)

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11772 ) Change subject: Updating web site for Kudu 1.8.0 release .. Patch Set 3: Code-Review+1 (1 comment)

[kudu-CR](gh-pages) [blog] Add post about 1.8.0 release

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11776 ) Change subject: [blog] Add post about 1.8.0 release .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR](gh-pages) [blog] Add post about 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Hello Mike Percy, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11776 to look at the new patch set (#2). Change subject: [blog] Add post about 1.8.0 release ..

[kudu-CR](gh-pages) [blog] Add post about 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/11776 ) Change subject: [blog] Add post about 1.8.0 release .. Patch Set 2: (2 comments)

[kudu-CR](gh-pages) [blog] Add post about 1.8.0 release

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11776 ) Change subject: [blog] Add post about 1.8.0 release .. Patch Set 1: (2 comments)

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11772 ) Change subject: Updating web site for Kudu 1.8.0 release .. Patch Set 3: Code-Review+1 -- To view, visit

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11774 ) Change subject: Bump version to 1.8.1-SNAPSHOT .. Bump version to 1.8.1-SNAPSHOT Change-Id:

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has removed a vote on this change. Change subject: Bump version to 1.8.1-SNAPSHOT .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11774 To unsubscribe, visit

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/11774 ) Change subject: Bump version to 1.8.1-SNAPSHOT .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/11774 To

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/11772 ) Change subject: Updating web site for Kudu 1.8.0 release .. Patch Set 3: > Patch Set 2: > > Good question. Not sure why I didn't do

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Hello Mike Percy, Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Andrew Wong, helifu, Adar Dembo, Anonymous Coward (314), Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11772 to look at the new patch set (#3). Change subject:

[kudu-CR](gh-pages) [blog] Add post about 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Hello Mike Percy, Grant Henke, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11776 to review the following change. Change subject: [blog] Add post about 1.8.0 release ..

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11772 ) Change subject: Updating web site for Kudu 1.8.0 release .. Patch Set 2: Good question. Not sure why I didn't do /docs. -- To view,

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11774 ) Change subject: Bump version to 1.8.1-SNAPSHOT .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/11774

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11774 ) Change subject: Bump version to 1.8.1-SNAPSHOT .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/11774/1/java/pom.xml

[kudu-CR](gh-pages) Add archived releases section

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has abandoned this change. ( http://gerrit.cloudera.org:8080/11295 ) Change subject: Add archived releases section .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/11295 To unsubscribe, visit

[kudu-CR] [docs] Update release notes after 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11773 ) Change subject: [docs] Update release notes after 1.8.0 release .. [docs] Update release notes after 1.8.0 release - add 1.8.0

[kudu-CR] WIP [kudu-jepsen] updated jepsen and clojureToolsCli

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11775 Change subject: WIP [kudu-jepsen] updated jepsen and clojureToolsCli .. WIP [kudu-jepsen] updated jepsen and clojureToolsCli Updated a

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 5: (2 comments)

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/11774 ) Change subject: Bump version to 1.8.1-SNAPSHOT .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/11774/1/java/pom.xml

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Attila Bukor (Code Review)
Hello Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11774 to look at the new patch set (#2). Change subject: Bump version to 1.8.1-SNAPSHOT .. Bump

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Hello Mike Percy, Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Andrew Wong, helifu, Adar Dembo, Anonymous Coward (314), Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11772 to look at the new patch set (#2). Change subject:

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/11772 ) Change subject: Updating web site for Kudu 1.8.0 release .. Patch Set 1: > Patch Set 1: > > (1 comment) sure, good idea. why not docs

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11774 ) Change subject: Bump version to 1.8.1-SNAPSHOT .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/11774/1/java/pom.xml

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11772 ) Change subject: Updating web site for Kudu 1.8.0 release .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.8.x) Bump version to 1.8.1-SNAPSHOT

2018-10-24 Thread Attila Bukor (Code Review)
Hello Mike Percy, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11774 to review the following change. Change subject: Bump version to 1.8.1-SNAPSHOT .. Bump version to

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11772 ) Change subject: Updating web site for Kudu 1.8.0 release .. Patch Set 1: (1 comment)

[kudu-CR] [docs] Update release notes after 1.8.0 release

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11773 ) Change subject: [docs] Update release notes after 1.8.0 release .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [docs] Update release notes after 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Hello Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11773 to review the following change. Change subject: [docs] Update release notes after 1.8.0 release .. [docs] Update

[kudu-CR] [kudu-jepsen] standardize layout of test resource files

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11769 ) Change subject: [kudu-jepsen] standardize layout of test resource files .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] [kudu-jepsen] standardize layout of test resource files

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11769 ) Change subject: [kudu-jepsen] standardize layout of test resource files .. [kudu-jepsen] standardize layout of test resource

[kudu-CR] [kudu-jepsen] standardize layout of test resource files

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11769 ) Change subject: [kudu-jepsen] standardize layout of test resource files .. Patch Set 2: Verified+1 Unrelated flake in

[kudu-CR](gh-pages) Updating web site for Kudu 1.8.0 release

2018-10-24 Thread Attila Bukor (Code Review)
Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, helifu, Andrew Wong, Anonymous Coward (314), Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11772 to review the following change. Change subject: Updating web site

[kudu-CR] docs: Add scripts to automate web site update for releases

2018-10-24 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/8800 ) Change subject: docs: Add scripts to automate web site update for releases .. Patch Set 1: (2 comments) Mike, could you also add this

[kudu-CR] [kudu-jepsen] standardize layout of test resource files

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11769 ) Change subject: [kudu-jepsen] standardize layout of test resource files .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [kudu-jepsen] standardize layout of test resource files

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11769 ) Change subject: [kudu-jepsen] standardize layout of test resource files .. Patch Set 1: (1 comment)

[kudu-CR] [kudu-jepsen] standardize layout of test resource files

2018-10-24 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11769 to look at the new patch set (#2). Change subject: [kudu-jepsen] standardize layout of test resource files

[kudu-CR] [java] added derby.log into .gitignore

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11770 Change subject: [java] added derby.log into .gitignore .. [java] added derby.log into .gitignore Change-Id:

[kudu-CR] [kudu-jepsen] added resources to the build rules

2018-10-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11769 ) Change subject: [kudu-jepsen] added resources to the build rules .. Patch Set 1: (1 comment)

[kudu-CR] [kudu-jepsen] added resources to the build rules

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11769 Change subject: [kudu-jepsen] added resources to the build rules .. [kudu-jepsen] added resources to the build rules The kudu-jepsen

[kudu-CR] Follow-up for commit fff0da840

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11768 ) Change subject: Follow-up for commit fff0da840 .. Follow-up for commit fff0da840 Change-Id:

[kudu-CR] Follow-up for commit fff0da840

2018-10-24 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11768 ) Change subject: Follow-up for commit fff0da840 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/11768

[kudu-CR] transaction tracker: add more information to the out-of-memory message

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11765 ) Change subject: transaction_tracker: add more information to the out-of-memory message .. Patch Set 3: (2 comments)

[kudu-CR] Follow-up for commit fff0da840

2018-10-24 Thread Adar Dembo (Code Review)
Hello Will Berkeley, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11768 to review the following change. Change subject: Follow-up for commit fff0da840 .. Follow-up for commit fff0da840

[kudu-CR] transaction tracker: add more information to the out-of-memory message

2018-10-24 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11765 ) Change subject: transaction_tracker: add more information to the out-of-memory message .. Patch Set 2: (2 comments)

[kudu-CR] transaction tracker: add more information to the out-of-memory message

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11765 ) Change subject: transaction_tracker: add more information to the out-of-memory message .. Patch Set 2: (2 comments)

[kudu-CR] transaction tracker: add more information to the out-of-memory message

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11765 ) Change subject: transaction_tracker: add more information to the out-of-memory message .. transaction_tracker: add more

[kudu-CR] transaction tracker: add more information to the out-of-memory message

2018-10-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11765 ) Change subject: transaction_tracker: add more information to the out-of-memory message .. Patch Set 2: Code-Review+2 (1 comment)

[kudu-CR] transaction tracker: add more information to the out-of-memory message

2018-10-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11765 ) Change subject: transaction_tracker: add more information to the out-of-memory message .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR] transaction tracker: add more information to the out-of-memory message

2018-10-24 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11765 to look at the new patch set (#2). Change subject: transaction_tracker: add more information to the out-of-memory message

[kudu-CR] transaction tracker: add more information to the out-of-memory message

2018-10-24 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11765 to review the following change. Change subject: transaction_tracker: add more information to the out-of-memory message

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 5: (3 comments)

[kudu-CR] [sentry] SentryAuthzProvider

2018-10-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] SentryAuthzProvider .. Patch Set 3: (12 comments)