Details

    • Type: Wish
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.2.0
    • Fix Version/s: 1.3.0
    • Component/s: JobManager
    • Labels:
      None

      Description

      In JobManager.scala
      var userCodeLoader = libraryCacheManager.getClassLoader(jobGraph.getJobID)
      this var could be a val

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user zhengcanbin opened a pull request:

          https://github.com/apache/flink/pull/3682

          FLINK-6269[JobManager]var could be val

          In JobManager.scala
          var userCodeLoader = libraryCacheManager.getClassLoader(jobGraph.getJobID)
          this var could be a val

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/zhengcanbin/flink FLINK-6269

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/flink/pull/3682.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #3682


          commit 03dd6b53ee08b1843019a2faf37b420fb8b27a3e
          Author: zcb <zhengcanbin@huawei.com>
          Date: 2017-04-06T07:31:46Z

          var could be val


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user zhengcanbin opened a pull request: https://github.com/apache/flink/pull/3682 FLINK-6269 [JobManager] var could be val In JobManager.scala var userCodeLoader = libraryCacheManager.getClassLoader(jobGraph.getJobID) this var could be a val You can merge this pull request into a Git repository by running: $ git pull https://github.com/zhengcanbin/flink FLINK-6269 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/3682.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #3682 commit 03dd6b53ee08b1843019a2faf37b420fb8b27a3e Author: zcb <zhengcanbin@huawei.com> Date: 2017-04-06T07:31:46Z var could be val
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user zentol commented on the issue:

          https://github.com/apache/flink/pull/3682

          merging.

          Show
          githubbot ASF GitHub Bot added a comment - Github user zentol commented on the issue: https://github.com/apache/flink/pull/3682 merging.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/flink/pull/3682

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3682
          Hide
          Zentol Chesnay Schepler added a comment -

          1.3: 2cb60043d8fce4a291fbefeddce6598b442b8f01

          Show
          Zentol Chesnay Schepler added a comment - 1.3: 2cb60043d8fce4a291fbefeddce6598b442b8f01

            People

            • Assignee:
              canbinzheng CanBin Zheng
              Reporter:
              canbinzheng CanBin Zheng
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development