review SPARK-8730

2015-07-24 Thread Eugen Cepoi
Hey,

I've opened a PR to fix ser/de issue of primitives classes in the java
serializer.
I already encountered this problem in different scenarios, so I am bringing
it up.
Would be great if someone wants to have a look at it! :)

https://issues.apache.org/jira/browse/SPARK-8730
https://github.com/apache/spark/pull/7122


Thanks,
Eugen


Re: review SPARK-8730

2015-07-24 Thread Sean Owen
It looks like you have a number of review comments on the PR that you
have not replied to. The PR does not merge at the moment either.

On Fri, Jul 24, 2015 at 12:03 PM, Eugen Cepoi cepoi.eu...@gmail.com wrote:
 Hey,

 I've opened a PR to fix ser/de issue of primitives classes in the java
 serializer.
 I already encountered this problem in different scenarios, so I am bringing
 it up.
 Would be great if someone wants to have a look at it! :)

 https://issues.apache.org/jira/browse/SPARK-8730
 https://github.com/apache/spark/pull/7122


 Thanks,
 Eugen

-
To unsubscribe, e-mail: dev-unsubscr...@spark.apache.org
For additional commands, e-mail: dev-h...@spark.apache.org



Re: review SPARK-8730

2015-07-24 Thread Eugen Cepoi
I just got those comments and it doesn't merge since few time, the code
evolved since I opened the pr

2015-07-24 14:12 GMT+02:00 Sean Owen so...@cloudera.com:

 It looks like you have a number of review comments on the PR that you
 have not replied to. The PR does not merge at the moment either.

 On Fri, Jul 24, 2015 at 12:03 PM, Eugen Cepoi cepoi.eu...@gmail.com
 wrote:
  Hey,
 
  I've opened a PR to fix ser/de issue of primitives classes in the java
  serializer.
  I already encountered this problem in different scenarios, so I am
 bringing
  it up.
  Would be great if someone wants to have a look at it! :)
 
  https://issues.apache.org/jira/browse/SPARK-8730
  https://github.com/apache/spark/pull/7122
 
 
  Thanks,
  Eugen