[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-02-22 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: I'm still keen to merge https://gerrit.cloudera.org/#/c/5636/ instead of this one. Will

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-02-22 Thread Chris George (Code Review)
Chris George has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: Code-Review+1 overall looks good... I don't think this is an issue if someone is using ky

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: I've put up a new gerrit based on Kousuke's suggestion: https://gerrit.cloudera.org/#/c/5

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: https://issues.apache.org/jira/browse/KUDU-1824 -- To view, visit http://gerrit.cloudera

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: I've filed an issue and added some notes in https://issues.apache.org/jira/browse/KUDU-18

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-01-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: Chris and Dan -- now that the holidays are behind us, any further thoughts on this discus

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-15 Thread Kousuke Saruta (Code Review)
Kousuke Saruta has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: > > Can we have any compatibility issue? Do you have any examples? > > Yes, by imple

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-15 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: > Can we have any compatibility issue? Do you have any examples? Yes, by implementing io.

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-14 Thread Kousuke Saruta (Code Review)
Kousuke Saruta has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: Hi Dan, Thank you for the review! I have a question and a comment. > I'm not keen on

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: Code-Review-1 Hi Kousuke, I'm not keen on adding io.Serializable to the java client class

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-13 Thread Kousuke Saruta (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5496 to look at the new patch set (#4). Change subject: KuduRDD.collect fails because of NoSerializableException .. Kudu

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-13 Thread Kousuke Saruta (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5496 to look at the new patch set (#3). Change subject: KuduRDD.collect fails because of NoSerializableException .. Kudu

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-13 Thread Kousuke Saruta (Code Review)
Kousuke Saruta has abandoned this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Abandoned This is duplicate PR. -- To view, visit http://gerrit.cloudera.org:8080/5497 To unsubscribe, visi

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-13 Thread Kousuke Saruta (Code Review)
Kousuke Saruta has uploaded a new change for review. http://gerrit.cloudera.org:8080/5497 Change subject: KuduRDD.collect fails because of NoSerializableException .. KuduRDD.collect fails because of NoSerializableException Ro

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-13 Thread Kousuke Saruta (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5496 to look at the new patch set (#2). Change subject: KuduRDD.collect fails because of NoSerializableException .. Kudu

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2016-12-13 Thread Kousuke Saruta (Code Review)
Kousuke Saruta has uploaded a new change for review. http://gerrit.cloudera.org:8080/5496 Change subject: KuduRDD.collect fails because of NoSerializableException .. KuduRDD.collect fails because of NoSerializableException Ro