Re: [2/4] sentry git commit: SENTRY-1429, SENTRY-1455: fix flaky TestHDFSIntegration, refactor this huge test class, improve its cleanUp and setUp for temp HDFS dirs; divide test class into smoke test

2016-08-29 Thread Sravya Tirukkovalur
The commit message should instead be SENTRY-1454: Fix intermittent time out issue for TestHDFSIntegration On Mon, Aug 29, 2016 at 2:54 PM, Sravya Tirukkovalur wrote: > Isn't this commit for https://issues.apache.org/jira/browse/SENTRY-1454. > Looks like commit message

Re: [2/4] sentry git commit: SENTRY-1429, SENTRY-1455: fix flaky TestHDFSIntegration, refactor this huge test class, improve its cleanUp and setUp for temp HDFS dirs; divide test class into smoke test

2016-08-29 Thread Sravya Tirukkovalur
Isn't this commit for https://issues.apache.org/jira/browse/SENTRY-1454. Looks like commit message mentions wrong jira numbers? On Mon, Aug 29, 2016 at 1:10 PM, wrote: > http://git-wip-us.apache.org/repos/asf/sentry/blob/ >

Re: Review Request 51471: SENTRY-1429, SENTRY-1455: fix flaky TestHDFSIntegration and unblock post-commit job.

2016-08-29 Thread Anne Yu
> On Aug. 29, 2016, 6:22 p.m., Sravya Tirukkovalur wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java, > > line 447 > > > > > > User needs to

Re: Review Request 51471: SENTRY-1429, SENTRY-1455: fix flaky TestHDFSIntegration and unblock post-commit job.

2016-08-29 Thread Anne Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51471/ --- (Updated Aug. 29, 2016, 6:41 p.m.) Review request for sentry, Hao Hao and

Re: Review Request 51471: SENTRY-1429, SENTRY-1455: fix flaky TestHDFSIntegration and unblock post-commit job.

2016-08-29 Thread Sravya Tirukkovalur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51471/#review147180 --- Fix it, then Ship it!