Github user viirya commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22635#discussion_r223251196
  
    --- Diff: python/pyspark/accumulators.py ---
    @@ -109,10 +109,14 @@
     
     def _deserialize_accumulator(aid, zero_value, accum_param):
         from pyspark.accumulators import _accumulatorRegistry
    -    accum = Accumulator(aid, zero_value, accum_param)
    -    accum._deserialized = True
    -    _accumulatorRegistry[aid] = accum
    -    return accum
    +    # If this certain accumulator was deserialized, don't overwrite it.
    +    if aid in _accumulatorRegistry:
    --- End diff --
    
    Yes.


---

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

Reply via email to