Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-9267

Remove highly unnecessary accumulators stringify methods

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Trivial
    • Resolution: Fixed
    • 1.0.0
    • 1.5.0
    • Spark Core
    • None

    Description

      def stringifyPartialValue(partialValue: Any): String = "%s".format(partialValue)
      def stringifyValue(value: Any): String = "%s".format(value)
      

      These are only used in 1 place (DAGScheduler). The level of indirection actually makes the code harder to read without an editor. We should just inline them...

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            huitseeker François Garillot
            andrewor14 Andrew Or
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment