[GitHub] [nifi] exceptionfactory commented on pull request #5711: NIFI-9625 Refactor Distributed Cache Server and Client Tests

2022-01-25 Thread GitBox
exceptionfactory commented on pull request #5711: URL: https://github.com/apache/nifi/pull/5711#issuecomment-1020641887 -- 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

[GitHub] [nifi] exceptionfactory commented on pull request #5711: NIFI-9625 Refactor Distributed Cache Server and Client Tests

2022-01-24 Thread GitBox
exceptionfactory commented on pull request #5711: URL: https://github.com/apache/nifi/pull/5711#issuecomment-1020684595 Pushed another commit that improves the behavior of the other test classes in the `nifi-distributed-cache-server` module to use `NetworkUtils.getAvailableTcpPort()` to

[GitHub] [nifi] exceptionfactory commented on pull request #5711: NIFI-9625 Refactor Distributed Cache Server and Client Tests

2022-01-24 Thread GitBox
exceptionfactory commented on pull request #5711: URL: https://github.com/apache/nifi/pull/5711#issuecomment-1020682277 Interesting that the `DistributedSetCacheTest` happened to fail, since it is not directly related to these changes. However, it looks like it uses the default port,

[GitHub] [nifi] exceptionfactory commented on pull request #5711: NIFI-9625 Refactor Distributed Cache Server and Client Tests

2022-01-24 Thread GitBox
exceptionfactory commented on pull request #5711: URL: https://github.com/apache/nifi/pull/5711#issuecomment-1020641887 Thanks for the feedback @joewitt, it looks like the local test run was relying on the previous existence of the cache data directory. I pushed an update to check for the