[
https://issues.apache.org/jira/browse/SENTRY-1429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16001615#comment-16001615
]
Alexander Kolbasov commented on SENTRY-1429:
--------------------------------------------
[~kalyanadupa] These are fixed in sentry-ha-redesign patch but not in master -
correct?
> TestHDFSIntegration improvements
> --------------------------------
>
> Key: SENTRY-1429
> URL: https://issues.apache.org/jira/browse/SENTRY-1429
> Project: Sentry
> Issue Type: Test
> Components: Sentry, Test
> Affects Versions: 1.8.0, sentry-ha-redesign
> Reporter: Sravya Tirukkovalur
> Fix For: sentry-ha-redesign
>
> Attachments: SENTRY-1429.001-sentry-ha-redesign.patch
>
>
> There are a few problems with this test that would be good to fix:
> 1. NN pulls the Sentry acls periodically, which means tests cannot verify
> acls as soon as it sets up the required privileges. This requires having
> sleeps in the test but it leads to some flakyness. We need to make sure we
> are actually sleeping after setting the permissions and before veryfying. Or
> explore other options to make this robust.
> 2. We need to make the tests more granular. There is a bunch of stuff
> happening in a single test, so it is hard to pinpoint why a test failed
> without looking at the stacktrace and debugging further.
> 3. Fix inter test dependencies if any. I see that some tests pass if run
> alone, but fail when the entire test class is run, so there might be some
> dependencies which we need to clean up.
> 4. Make these tests runnable on real cluster. So that we can catch any
> integration and configuration issues.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
