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.Serializable a class representation is constrained to 
the fields in the original format.  See Effective Java item 74 for more reasons 
why the Serializable API should be avoided.

>  Kryo can't serialize/deserialize guava's ImmutableList

Kryo support for guava collections is provided by 
https://github.com/magro/kryo-serializers, if you add that jar to the classpath 
I believe that error should go away.

    <dependency>
        <groupId>de.javakaffee</groupId>
        <artifactId>kryo-serializers</artifactId>
        <version>0.41</version>
    </dependency>

-- 
To view, visit http://gerrit.cloudera.org:8080/5496
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If0463424481a33c66fd7464345c305062420cfe9
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Kousuke Saruta <saru...@oss.nttdata.co.jp>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Kousuke Saruta <saru...@oss.nttdata.co.jp>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: No

Reply via email to