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

2016-08-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: C++ client: deprecating KuduPartialRow::SetString() .. Abandoned Abandoning this in favor of http://gerrit.cloudera.org:8080/3868 which has been merged already. -- To

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

2016-08-10 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3723 to look at the new patch set (#11). Change subject: C++ client: deprecating KuduPartialRow::SetString()

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

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

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

2016-08-10 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3723 to look at the new patch set (#9). Change subject: C++ client: deprecating KuduPartialRow::SetString()

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

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

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

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

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

2016-08-09 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3723 to look at the new patch set (#5). Change subject: C++ client: deprecating KuduPartialRow::SetString()

[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] 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 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