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 01:00:00 UTC
[jira] [Updated] (SENTRY-1946) getPathsUpdatesFrom() got its
boolean logic inversed which results in sending two snapshots at the same
time
[ https://issues.apache.org/jira/browse/SENTRY-1946?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Vamsee Yarlagadda updated SENTRY-1946:
--------------------------------------
Summary: getPathsUpdatesFrom() got its boolean logic inversed which results in sending two snapshots at the same time (was: Conditional error in SENTRY-1931 results in sending two snapshots at the same time)
> getPathsUpdatesFrom() got its boolean logic inversed which results in sending two snapshots at the same time
> ------------------------------------------------------------------------------------------------------------
>
> Key: SENTRY-1946
> URL: https://issues.apache.org/jira/browse/SENTRY-1946
> Project: Sentry
> Issue Type: Bug
> Affects Versions: 2.0.0
> Reporter: Vamsee Yarlagadda
> Assignee: Vamsee Yarlagadda
> Attachments: SENTRY-1946.01-master.patch
>
>
> 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.
> {code}
> // 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();
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)