[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/2738/ -- To view, visit

[kudu-CR](gh-pages) Add weekly update for 8/8

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3864 Change subject: Add weekly update for 8/8 .. Add weekly update for 8/8 Change-Id: Idd1dcb28082b476c18a92a91f6f84af4c69df66b --- A

[kudu-CR](gh-pages) Add weekly update for 8/8

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new patch set (#2). Change subject: Add weekly update for 8/8 .. Add weekly update for 8/8 Change-Id: Idd1dcb28082b476c18a92a91f6f84af4c69df66b --- A _posts/2016-08-08-weekly-update.md 1 file changed,

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-08 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 1: I'm using it in the rust client. There are a couple of cases where the field could be used in the C++ and

[kudu-CR](gh-pages) Add weekly update for 8/8

2016-08-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add weekly update for 8/8 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3864/1/_posts/2016-08-08-weekly-update.md File

[kudu-CR](gh-pages) Add weekly update for 8/8

2016-08-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add weekly update for 8/8 .. Patch Set 2: Code-Review+2 Glad I noticed the more important error. -- To view, visit http://gerrit.cloudera.org:8080/3864 To

[kudu-CR] Persistent cache support for NVM

2016-08-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Persistent cache support for NVM .. Patch Set 18: Build Started http://104.196.14.100/job/kudu-gerrit/2745/ -- To view, visit

[kudu-CR] Persistent cache support for NVM

2016-08-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Persistent cache support for NVM .. Patch Set 18: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2747/ -- To view, visit

[kudu-CR] Upgrade to NVML 1.1

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Upgrade to NVML 1.1 .. Upgrade to NVML 1.1 Change-Id: I9b28130407b4ec30bc13a9acfc2390cee7a69de4 Reviewed-on: http://gerrit.cloudera.org:8080/3866

[kudu-CR] [CMakeLists.txt] dot is mandatory for doxygen

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [CMakeLists.txt] dot is mandatory for doxygen .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3855 To unsubscribe, visit

[kudu-CR] catalog manager: cleanup status messages

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: catalog manager: cleanup status messages .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3858 To unsubscribe, visit

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 1: you don't want to use this new field in the client? seems strange to add it with no use case -- To view,

[kudu-CR] Adding some missing 0.10.0 release notes

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Adding some missing 0.10.0 release notes .. Patch Set 1: (8 comments) http://gerrit.cloudera.org:8080/#/c/3802/1/docs/release_notes.adoc File

[kudu-CR] [java client] Support add/remove partition

2016-08-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Support add/remove partition .. Patch Set 1: (18 comments)

[kudu-CR] [java-client] repackage to org.apache.kudu (Part 3)

2016-08-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] repackage to org.apache.kudu (Part 3) .. Patch Set 3: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2750/ -- To view, visit

[kudu-CR] [C++ client] doxygenized all C++ client API

2016-08-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [C++ client] doxygenized all C++ client API .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/3840/2/src/kudu/client/callbacks.h File

[kudu-CR] [java-client] repackage to org.apache.kudu (Part 3)

2016-08-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] repackage to org.apache.kudu (Part 3) .. Patch Set 3: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2753/ -- To view, visit

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: C++ client: deprecating KuduPartialRow::SetString() .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/3723/3//COMMIT_MSG Commit Message: Line

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: C++ client: deprecating KuduPartialRow::SetString() .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2757/ -- To view, visit

[kudu-CR] WIP: Adding some missing 0.10.0 release notes

2016-08-08 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3802 to look at the new patch set (#2). Change subject: WIP: Adding some missing 0.10.0 release notes ..

[kudu-CR] WIP: Adding some missing 0.10.0 release notes

2016-08-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Adding some missing 0.10.0 release notes .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2751/ -- To view, visit

[kudu-CR] client/sample.cc: fixed a couple of crashes

2016-08-08 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3685 to look at the new patch set (#3). Change subject: client/sample.cc: fixed a couple of crashes ..

[kudu-CR] client/sample.cc: fixed a couple of crashes

2016-08-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: client/sample.cc: fixed a couple of crashes .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2752/ -- To view, visit

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: C++ client: deprecating KuduPartialRow::SetString() .. Patch Set 3: Can we try to close this one out for 0.10? Seems like it's basically ready and a good API