HyukjinKwon commented on issue #27599: [SPARK-30846][CORE] Add AccumulatorV2 
API in JavaSparkContext
URL: https://github.com/apache/spark/pull/27599#issuecomment-590681870
 
 
   Agree with Sean's point in general. However, from taking a cursory look, it 
does duplicate methods where it doesn't even look specific to Java. How many 
method are missing in `JavaSparkContext` compared to `SparkContext`? Seems not 
so many.
   
   If there are too many instances missing, let's stick to "add Java-friendly 
versions of methods where they are different". If it's not too many, let's add 
it to `JavaSparkContext`, considering that we won't likely add a lot of APIs 
into `SparkContext`.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to