Uploaded image for project: 'Geode'
  1. Geode
  2. GEODE-5151 AbstractRegionMap.txApplyPut should be refactored
  3. GEODE-5164

refactor the three blocks of code in txApplyPut into one method

    XMLWordPrintableJSON

Details

    Description

      AbstractRegionMap.txApplyPut currently has three blocks of code that do the put:
      1. applyTxUpdateOnReplicateOrRedundantCopy is called and much of the code in it is the same as the oldRe block
      2. oldRe block: if an existing entry is found, do the put by changing it
      3. newRe block: if no existing entry is found, create a new entry and do the put by changing it

      It would help future refactoring of txApplyPut if these 3 blocks of code could be consolidated into a single method.

      Attachments

        Issue Links

          Activity

            People

              dschneider Darrel Schneider
              dschneider Darrel Schneider
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 1h 20m
                  1h 20m