Uploaded image for project: 'Apache Tez'
  1. Apache Tez
  2. TEZ-690 Tez API Ease of Use
  3. TEZ-1137

Move TezJobConfig to runtime-library and rename to TezRuntimeConfiguration

Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.5.0
    • None
    • None
    • Incompatible change, Reviewed

    Description

      Its currently sitting in the API project. Some values from here may need to move to TezConfiguration.

      Attachments

        1. TEZ-1137.1.patch
          284 kB
          Bikas Saha
        2. TEZ-1137.2.patch
          284 kB
          Bikas Saha

        Activity

          bikassaha Bikas Saha added a comment -

          1) Renames files to TezRuntimeConfiguration.
          2) Moves it to tez-runtime-library/org.apache.tez.runtime.library.api
          3) Moves parameters from TezRuntimeConfiguration to TezConfiguration where needed. None of them were IPO specific.
          4) Some config names changed to read better. Should be obvious from TezConfiguration.

          sseth hitesh please review or comment if you want to. Since this is mostly an Eclipse refactoring and a fragile change, I want to commit it by noon tomorrow. Will do so if there are no comments.

          bikassaha Bikas Saha added a comment - 1) Renames files to TezRuntimeConfiguration. 2) Moves it to tez-runtime-library/org.apache.tez.runtime.library.api 3) Moves parameters from TezRuntimeConfiguration to TezConfiguration where needed. None of them were IPO specific. 4) Some config names changed to read better. Should be obvious from TezConfiguration. sseth hitesh please review or comment if you want to. Since this is mostly an Eclipse refactoring and a fragile change, I want to commit it by noon tomorrow. Will do so if there are no comments.
          sseth Siddharth Seth added a comment -

          +1. Looks good.
          TEZ_TASK_SCALE_TASK_MEMORY_ADDITIONAL_RESERVATION_FRACTION_MAX - just noticed that the parameter name has always has an unintentional space in it. If you don't mind, please fix before committing.

          sseth Siddharth Seth added a comment - +1. Looks good. TEZ_TASK_SCALE_TASK_MEMORY_ADDITIONAL_RESERVATION_FRACTION_MAX - just noticed that the parameter name has always has an unintentional space in it. If you don't mind, please fix before committing.
          bikassaha Bikas Saha added a comment -

          Done. Attaching commit patch. Thanks!

          bikassaha Bikas Saha added a comment - Done. Attaching commit patch. Thanks!
          bikassaha Bikas Saha added a comment -

          commit 2050f50e9e7d25c7b48ccc71fb507b4bf8bba23e
          Author: Bikas Saha <bikas@apache.org>
          Date: Thu Jul 24 10:42:02 2014 -0700

          TEZ-1137. Move TezJobConfig to runtime-library and rename to TezRuntimeConfiguration (bikas)

          bikassaha Bikas Saha added a comment - commit 2050f50e9e7d25c7b48ccc71fb507b4bf8bba23e Author: Bikas Saha <bikas@apache.org> Date: Thu Jul 24 10:42:02 2014 -0700 TEZ-1137 . Move TezJobConfig to runtime-library and rename to TezRuntimeConfiguration (bikas)
          bikassaha Bikas Saha added a comment -

          Bulk close for jiras fixed in 0.5.0.

          bikassaha Bikas Saha added a comment - Bulk close for jiras fixed in 0.5.0.

          People

            bikassaha Bikas Saha
            bikassaha Bikas Saha
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: