You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@sentry.apache.org by "Vamsee Yarlagadda (JIRA)" <ji...@apache.org> on 2017/09/12 00:53:04 UTC
[jira] [Resolved] (SENTRY-1945) getPathsUpdatesFrom() got its
boolean logic inversed
[ https://issues.apache.org/jira/browse/SENTRY-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Vamsee Yarlagadda resolved SENTRY-1945.
---------------------------------------
Resolution: Duplicate
> 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)