[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-29 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1890 Allow renaming of primary key column .. Patch Set 13: Addressed Todd's comments. The jenkins testcase failure is unrelated to this change. -- To view,

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-29 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#13). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-29 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#12). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-15 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1890 Allow renaming of primary key column .. Patch Set 11: Merged with latest master branch -- To view, visit http://gerrit.cloudera.org:8080/6078 To

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-15 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#10). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-15 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1890 Allow renaming of primary key column .. Patch Set 9: (5 comments) Addressed code review comments

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-15 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#9). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-07 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1890 Allow renaming of primary key column .. Patch Set 8: Test failures are unrelated -- To view, visit http://gerrit.cloudera.org:8080/6078 To unsubscribe,

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-07 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1890 Allow renaming of primary key column .. Patch Set 8: (5 comments) Addressed Todd's comments

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-07 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#8). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-02-27 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1890 Allow renaming of primary key column .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-02-27 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#5). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-02-26 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1890 Allow renaming of primary key column .. Patch Set 4: (4 comments) Added test cases from alter_table-test.cc and alter_table-randomized-test.cc as

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-02-26 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#4). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-02-20 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#3). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-02-20 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6078 to look at the new patch set (#2). Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-02-19 Thread Ram Mettu (Code Review)
Ram Mettu has uploaded a new change for review. http://gerrit.cloudera.org:8080/6078 Change subject: KUDU-1890 Allow renaming of primary key column .. KUDU-1890 Allow renaming of primary key column Removed the checks

[kudu-CR] Changes: In kudu-spark, added support to delete rows from Kudu table in DataSource API. This is done using the SaveMode.ErrorIfExists.

2016-09-15 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: Changes: In kudu-spark, added support to delete rows from Kudu table in DataSource API. This is done using the SaveMode.ErrorIfExists. .. Patch Set 2: No this has

[kudu-CR] KUDU-1581 Fix DataFrame read failure when table has Binary Col

2016-08-29 Thread Ram Mettu (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4145 to look at the new patch set (#3). Change subject: KUDU-1581 Fix DataFrame read failure when table has Binary Col

[kudu-CR] KUDU-1581 Fix DataFrame read failure when table has Binary Col For Binary Cols, kudu-spark is returning a ByteBuffer object when Spark expects to receive Array[Byte], so change is to return

2016-08-29 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1581 Fix DataFrame read failure when table has Binary Col For Binary Cols, kudu-spark is returning a ByteBuffer object when Spark expects to receive Array[Byte], so change is to return a copy of the byte array. Modified

[kudu-CR] KUDU-1581 Fix DataFrame read failure when table has Binary Col For Binary Cols, kudu-spark is returning a ByteBuffer object when Spark expects to receive Array[Byte], so change is to return

2016-08-27 Thread Ram Mettu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4145 to look at the new patch set (#2). Change subject: KUDU-1581 Fix DataFrame read failure when table has Binary Col For Binary Cols, kudu-spark is returning a ByteBuffer object when

[kudu-CR] KUDU-1581 Fix DataFrame read failure when table has Binary Col

2016-08-27 Thread Ram Mettu (Code Review)
Ram Mettu has uploaded a new change for review. http://gerrit.cloudera.org:8080/4145 Change subject: KUDU-1581 Fix DataFrame read failure when table has Binary Col .. KUDU-1581 Fix DataFrame read failure when table has Binary

[kudu-CR] KUDU-1522 Fix AlterTable Timestamp col with default value

2016-08-15 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1522 Fix AlterTable Timestamp col with default value .. Patch Set 1: @Todd I fixed your comments, it created a new gerrit code review - may be something wrong

[kudu-CR] KUDU-1522 Fix AlterTable Timestamp col with default value

2016-08-15 Thread Ram Mettu (Code Review)
Ram Mettu has uploaded a new change for review. http://gerrit.cloudera.org:8080/3980 Change subject: KUDU-1522 Fix AlterTable Timestamp col with default value .. KUDU-1522 Fix AlterTable Timestamp col with default value

[kudu-CR] KUDU-1522 Fix AlterTable Timestamp col with default value

2016-08-13 Thread Ram Mettu (Code Review)
Ram Mettu has uploaded a new change for review. http://gerrit.cloudera.org:8080/3970 Change subject: KUDU-1522 Fix AlterTable Timestamp col with default value .. KUDU-1522 Fix AlterTable Timestamp col with default value

[kudu-CR] KUDU-1533 Spark Kudu Rdd/Dataframe upsert

2016-08-12 Thread Ram Mettu (Code Review)
Ram Mettu has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 4: Changes and the documentation looks good. -- To view, visit http://gerrit.cloudera.org:8080/3871 To