You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@sentry.apache.org by "Alexander Kolbasov (JIRA)" <ji...@apache.org> on 2017/09/12 00:37:00 UTC

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

Alexander Kolbasov created SENTRY-1945:
------------------------------------------

             Summary: getPathsUpdatesFrom() got its boolean logic inversed
                 Key: SENTRY-1945
                 URL: https://issues.apache.org/jira/browse/SENTRY-1945
             Project: Sentry
          Issue Type: Bug
          Components: Sentry
    Affects Versions: 2.0.0
            Reporter: Alexander Kolbasov
            Assignee: Sergio Peña
            Priority: Blocker


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

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

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



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)