Uploaded image for project: 'Sentry'
  1. Sentry
  2. SENTRY-1946

getPathsUpdatesFrom() got its boolean logic inversed which results in sending two snapshots at the same time

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.0
    • Component/s: None
    • Labels:
      None

      Description

      As part of SENTRY-1931, we added check not to send two NN snaphsots at the same time. But because of a conditional typo, it results in sending back two snapshots.

          // If another paths updates request is in progress by another HDFS NameNode, then we
          // return an empty image for now to avoid a large memory consumption
          if (pathsRetrieverBusy.compareAndSet(false, true)) {
            LOGGER.debug("PATHS updates are not available because another request is in progress.");
            return Collections.emptyList();
          }
      

        Attachments

        1. SENTRY-1946.01-master.patch
          1.0 kB
          Vamsee K. Yarlagadda

          Issue Links

            Activity

              People

              • Assignee:
                vamsee Vamsee K. Yarlagadda
                Reporter:
                vamsee Vamsee K. Yarlagadda
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: