[SENTRY-1945] getPathsUpdatesFrom() got its boolean logic inversed

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Duplicate
• Affects Version/s: 2.0.0
• Component/s: Sentry
• Assignee: Sergio Peña
• Reporter: Alex Kolbasov
• Created: 12/Sep/17 00:36
• Updated: 12/Sep/17 00:52
• Resolved: 12/Sep/17 00:52

Description

Sentry-1919 introduced the following code in getPathsUpdatesFrom():

    if (pathsRetrieverBusy.compareAndSet(false, true)) {
      LOGGER.debug("PATHS updates are not available because another request is in progress.");
      return Collections.emptyList();
    }

It is easy to see that the code check is inversed and we are doing quite the opposite of what we want to do.

Issue Links

• duplicates: SENTRY-1946 getPathsUpdatesFrom() got its boolean logic inversed which results in sending two snapshots at the same time