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

2016-08-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 6: Thanks, Will! -- To view, visit http://gerrit.cloudera.org:8080/3871 To unsubscribe, visit http://gerri

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

2016-08-12 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. KUDU-1533 Spark Kudu Rdd/Dataframe upsert This patch improves the Kudu SparkSQL integration in two ways: 1)

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

2016-08-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3871 To unsubscribe, visit http://gerrit

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

2016-08-12 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/3871/4/java/kudu-spark/src/main/scala/org/apache/kudu

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

2016-08-12 Thread Will Berkeley (Code Review)
Hello Chris George, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3871 to look at the new patch set (#5). Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. KUDU-

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

2016-08-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2839/ -- To view, visit http://gerrit.cloudera.org

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

2016-08-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 4: (3 comments) Looking really good now, sorry I sent you down a rabbit hole with the CreatableRelation st

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

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

2016-08-12 Thread Will Berkeley (Code Review)
Hello Chris George, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3871 to look at the new patch set (#4). Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. KUDU-

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

2016-08-11 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2831/ -- To view, visit http://gerrit.cloudera.org

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

2016-08-11 Thread Chris George (Code Review)
Chris George has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 3: Specifically this documentation: http://kudu.apache.org/docs/developing.html#_kudu_integration_with_spa

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

2016-08-11 Thread Chris George (Code Review)
Chris George has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 3: Code-Review-1 I would prefer that we not drop CreatableRelationProvider. I'm using cassandra spark conn

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

2016-08-11 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3871/2/java/kudu-spark/src/main/scala/org/apache/kudu/

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

2016-08-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/3871/2//COMMIT_MSG Commit Message: Line 19: The change

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

2016-08-11 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2816/ -- To view, visit http://gerrit.cloudera.org

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

2016-08-11 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3871 to look at the new patch set (#3). Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. KUDU-1533 Spark Kud

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

2016-08-11 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 2: (8 comments) http://gerrit.cloudera.org:8080/#/c/3871/2//COMMIT_MSG Commit Message: Line 13: for Kudu

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

2016-08-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 2: (8 comments) This is looking great. Sorry I've been so slow on reviews. I'd love to get this in for

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

2016-08-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/3871/1/java/kudu-spark/src/main/scala/org/apache/kudu

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

2016-08-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2785/ -- To view, visit http://gerrit.cloudera.org

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

2016-08-09 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3871 to look at the new patch set (#2). Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. KUDU-1533 Spark Kud

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

2016-08-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/3871/1/java/kudu-spark/src/main/scala/org/apache/kudu/s

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

2016-08-09 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/3871 Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. KUDU-1533 Spark Kudu Rdd/Dataframe upsert This patch adds support for upser

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

2016-08-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1533 Spark Kudu Rdd/Dataframe upsert .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2761/ -- To view, visit http://gerrit.cloudera.org