sumitagrawl commented on PR #4505: URL: https://github.com/apache/ozone/pull/4505#issuecomment-1507097074
> Thanks for working on this @sumitagrawl. > > * There are some places where a raw `LeaseManager` object is used, and others where it is `LeaseManager<Object>`. I think we can use the later in all places to avoid compiler warnings. > * Can we do tests with and without this patch on a live cluster to verify it fixes the issue in most cases? > * Can we add some sort of unit test to verify the delay is respected? @errose28 1. LeaseManager<Object> changes are done 2. I have tested with local m/c with container size 256MB, freon with 5 client -- able to reproduce and verified with fix 3. Added a UT case for same -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
