aem-testing-clients icon indicating copy to clipboard operation
aem-testing-clients copied to clipboard

Wrong delay in TemporaryContentAuthorGroup class

Open hrayrpapikyan opened this issue 2 years ago • 1 comments

The timeout is 1000000 milliseconds instead of 1000. p.poll(DEFAULT_TIMEOUT, SECONDS.toMillis(DEFAULT_RETRY_DELAY)); DEFAULT_RETRY_DELAY equals to 1000 and the result is million milliseconds (~16 minutes), which does not seem logical. The code can be replaced with: p.poll(DEFAULT_TIMEOUT, DEFAULT_RETRY_DELAY); or p.poll(DEFAULT_TIMEOUT, SECONDS.toMillis(1));

hrayrpapikyan avatar Mar 28 '23 10:03 hrayrpapikyan

@hrayrpapikyan do you have a PR to see the change in the context?

volteanu avatar Apr 04 '23 09:04 volteanu