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

    https://github.com/apache/spark/pull/3262#discussion_r20400601
  
    --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
    @@ -1427,46 +1427,56 @@ object SparkContext extends Logging {
     
       private[spark] val DRIVER_IDENTIFIER = "<driver>"
     
    -  implicit object DoubleAccumulatorParam extends AccumulatorParam[Double] {
    +  @deprecated("An API for backforward compatibility", "1.2.0")
    +  object DoubleAccumulatorParam extends AccumulatorParam[Double] {
         def addInPlace(t1: Double, t2: Double): Double = t1 + t2
         def zero(initialValue: Double) = 0.0
       }
     
    -  implicit object IntAccumulatorParam extends AccumulatorParam[Int] {
    +  @deprecated("An API for backforward compatibility", "1.2.0")
    +  object IntAccumulatorParam extends AccumulatorParam[Int] {
         def addInPlace(t1: Int, t2: Int): Int = t1 + t2
         def zero(initialValue: Int) = 0
       }
     
    -  implicit object LongAccumulatorParam extends AccumulatorParam[Long] {
    +  @deprecated("An API for backforward compatibility", "1.2.0")
    +  object LongAccumulatorParam extends AccumulatorParam[Long] {
         def addInPlace(t1: Long, t2: Long) = t1 + t2
         def zero(initialValue: Long) = 0L
       }
     
    -  implicit object FloatAccumulatorParam extends AccumulatorParam[Float] {
    +  @deprecated("An API for backforward compatibility", "1.2.0")
    +  object FloatAccumulatorParam extends AccumulatorParam[Float] {
         def addInPlace(t1: Float, t2: Float) = t1 + t2
         def zero(initialValue: Float) = 0f
       }
     
       // TODO: Add AccumulatorParams for other types, e.g. lists and strings
     
    -  implicit def rddToPairRDDFunctions[K, V](rdd: RDD[(K, V)])
    +  @deprecated("An API for backforward compatibility", "1.2.0")
    --- End diff --
    
    update these accordingly too


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to