Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-12956 Inevitable Log4j2 migration via slf4j
  3. HADOOP-14542

Add IOUtils.cleanupWithLogger that accepts slf4j logger API

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.9.0, 3.0.0-alpha4
    • None
    • None

    Description

      Split from HADOOP-14539.
      Now IOUtils.cleanup only accepts commons-logging logger API. Now we are migrating the APIs to slf4j, slf4j logger API should be accepted as well. Adding IOUtils.cleanup(Logger, Closeable...) causes IOUtils.cleanup(null, Closeable...) to fail (incompatible change), so it's better to change the method name to avoid the conflict.

      Attachments

        1. HADOOP-14542.001.patch
          2 kB
          Chen Liang
        2. HADOOP-14542.002.patch
          2 kB
          Chen Liang
        3. HADOOP-14542.003.patch
          2 kB
          Chen Liang

        Issue Links

        Activity

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

          People

            vagarychen Chen Liang
            aajisaka Akira Ajisaka
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment