Uploaded image for project: 'AdMax'
  1. AdMax
  2. ADMAX-2206

SearchEngineSummarizer: creating WarehouseSummarizer instances unnecessarily

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: unspecified
    • Fix Version/s: unspecified
    • Component/s: Data Summarization
    • Labels:
      None
    • Environment:

      Operating System: Linux
      Platform: PC

    • Bugzilla Id:
      3214

      Description

      We only need to create a WarehouseSummarizer in the
      WarehouseSummarizerSearchEngineAccountDispatcher if the # of total rows of data
      summarized by the SearchEngineAccountWorkers for a particular account > 0.

      To fix this, we need to keep track of the # of rows in summarized, and in the
      summarizeWarehouseData() method we can say

      if (thePreventWHSummFlag) {
      ...
      }
      else if (getNumDataRowsForCurrentAccount() > 0) {
      WarehouseSummarizer whsumm = new WarehouseSummarizer();
      ...
      }
      else {
      // Skipping Account getAccountIDs();
      }

      Note: This might cause issues with Warehouse data availability being gray, so we
      may need to do some additional work to set warehouse data availability for that
      distribution, so data availability will still be set the same way it is now.

        Attachments

          Activity

            People

            • Assignee:
              squadrim Mike Squadrito (Inactive)
              Reporter:
              therouxj Jeff Theroux
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: