Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Duplicate
    • Affects Version/s: 0.23.2
    • Fix Version/s: None
    • Component/s: mrv2
    • Labels:
      None

      Description

      This tracks a couple of cleanup issues that were identified in MAPREDUCE-4043:

      • It seems unnecessary to pass the job token and credentials separately when we always combine the job token into the credentials before building the container launch context. The TaskImpl and TaskAttemptImpl constructors could simply take credentials with the job token already added rather than separate job token and credential parameters.
      • It's unclear whether we still need the appTokens file that is placed into HDFS by the job submitter, localized by the NM, and finally read in by the AM. I believe the AM's credentials sent in the AM's container launch context already contains the same information. If that's the case, we should remove the code related to the appTokens file.

        Issue Links

          Activity

          Vinod Kumar Vavilapalli made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Duplicate [ 3 ]
          Vinod Kumar Vavilapalli made changes -
          Field Original Value New Value
          Link This issue duplicates MAPREDUCE-5199 [ MAPREDUCE-5199 ]
          Hide
          Vinod Kumar Vavilapalli added a comment -

          Fixed by MAPREDUCE-5199.

          Show
          Vinod Kumar Vavilapalli added a comment - Fixed by MAPREDUCE-5199 .
          Jason Lowe created issue -

            People

            • Assignee:
              Unassigned
              Reporter:
              Jason Lowe
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development