Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-10189

Code cleanup in LeveldbRMStateStore

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3.0, 3.2.2, 3.4.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Some things can be improved:

      • throws Exception declaration can be removed from LeveldbRMStateStore.initInternal method 
      • key variable is redundant in LeveldbRMStateStore.dbStoreVersion
      • try can use automatic Resource management in LeveldbRMStateStore.loadReservationState/loadRMDTSecretManagerKeys/loadRMDTSecretManagerTokens/loadRMApps/... etc
      • there were some methods which were copied to LeveldbConfigurationStore (ie: openDatabase, storeVersion, loadVersion, CompactionTimerClass nested class), a helper class could be created to reduce the duplicated code
      • Any other cleanup

        Attachments

        1. YARN-10189.POC001.patch
          37 kB
          Benjamin Teke
        2. YARN-10189.POC002.patch
          38 kB
          Benjamin Teke
        3. YARN-10189.001.patch
          32 kB
          Benjamin Teke
        4. YARN-10189.branch-3.2.001.patch
          55 kB
          Benjamin Teke
        5. YARN-10189.branch-3.3.001.patch
          32 kB
          Benjamin Teke
        6. YARN-10189.branch-3.2.002.patch
          31 kB
          Benjamin Teke

        Issue Links

          Activity

          $i18n.getText('security.level.explanation', $currentSelection) Viewable by All Users
          Cancel

            People

            • Assignee:
              bteke Benjamin Teke Assign to me
              Reporter:
              bteke Benjamin Teke

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment