[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2017-02-22 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1493: Implement SchemaRelationProvider .. KUDU-1493: Implement SchemaRelationProvider Implement SchemaRelationProvider org.apache.kudu.spark.kudu.Def

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2017-02-22 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1493: Implement SchemaRelationProvider .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5167 To unsubscribe, visit http://gerr

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2017-02-22 Thread Chris George (Code Review)
Chris George has posted comments on this change. Change subject: KUDU-1493: Implement SchemaRelationProvider .. Patch Set 5: would be good to get some examples in documentation about using this -- To view, visit http://gerrit

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2017-02-17 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1493: Implement SchemaRelationProvider .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5167/5/java/kudu-spark/src/main/scala/org/apache/kudu/

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2017-02-13 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5167 to look at the new patch set (#5). Change subject: KUDU-1493: Implement SchemaRelationProvider .. KUDU-1493: Implem

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2017-02-13 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1493: Implement SchemaRelationProvider .. Patch Set 4: (7 comments) http://gerrit.cloudera.org:8080/#/c/5167/3/java/kudu-spark/src/main/scala/org/apache/ku

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2017-02-13 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5167 to look at the new patch set (#4). Change subject: KUDU-1493: Implement SchemaRelationProvider .. KUDU-1493: Implem

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2016-12-15 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1493: Implement SchemaRelationProvider .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/5167/3/java/kudu-spark/src/main/scala/org/apache/ku

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2016-12-12 Thread Chris George (Code Review)
Chris George has posted comments on this change. Change subject: KUDU-1493: Implement SchemaRelationProvider .. Patch Set 3: (1 comment) overall I think it looks good... we should update the @param on KuduRelation Would also b

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2016-11-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1493: Implement SchemaRelationProvider .. Patch Set 3: Dan/Chris, can you guys take a look at this when you get a chance? I don't feel qualified to review th

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider

2016-11-23 Thread Andrew Manlove (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5167 to look at the new patch set (#3). Change subject: KUDU-1493: Implement SchemaRelationProvider .. KUDU-1493: Implem

[kudu-CR] KUDU-1493: Implement SchemaRelationProvider for spark kudu DefaultSource

2016-11-23 Thread Andrew Manlove (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5167 to look at the new patch set (#2). Change subject: KUDU-1493: Implement SchemaRelationProvider for spark kudu DefaultSource ...