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

Use `text.StringEscapeUtils` instead `lang3.StringEscapeUtils`

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

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.0.0
    • 3.0.0
    • Build, Spark Core, SQL
    • None

    Description

      `org.apache.commons.lang3.StringEscapeUtils` is deprecated over two years ago at LANG-1316.

      /**
 * <p>Escapes and unescapes {@code String}s for
 * Java, Java Script, HTML and XML.</p>
 *
 * <p>#ThreadSafe#</p>
 * @since 2.0
 * @deprecated as of 3.6, use commons-text
 * <a href="https://commons.apache.org/proper/commons-text/javadocs/api-release/org/apache/commons/text/StringEscapeUtils.html">
 * StringEscapeUtils</a> instead
 */
@Deprecated
public class StringEscapeUtils {

      This issue aims to use the latest one from `commons-text` module which has more bug fixes like
      TEXT-100, TEXT-118 and TEXT-120.

      -import org.apache.commons.lang3.StringEscapeUtils
+import org.apache.commons.text.StringEscapeUtils

      This will add a new dependency to `hadoop-2.7` profile distribution.

      +commons-text-1.6.jar

      Attachments

        Issue Links

        relates to

        Bug - A problem which impairs or prevents the functions of the product. TEXT-120 StringEscapeUtils#unescapeJson does not unescape double quotes and forward slash

        • Major - Major loss of function.
        • Resolved

        Bug - A problem which impairs or prevents the functions of the product. TEXT-118 JSON escaping incorrect for the delete control character

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Resolved

        Bug - A problem which impairs or prevents the functions of the product. TEXT-100 StringEscapeUtils#UnEscapeJson doesn't recognize escape signs correctly

        • Major - Major loss of function.
        • Closed

        Task - A task that needs to be done. LANG-1316 Deprecate classes/methods moved to commons-text

        • Major - Major loss of function.
        • Closed
        links to

        Web Link GitHub Pull Request #25281

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

          People

            dongjoon Dongjoon Hyun
            dongjoon Dongjoon Hyun
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment