[GitHub] [nifi] ZhewenFu commented on pull request #6596: NIFI-10717 fix inconsistent tests

[GitBox <gi...@apache.org>]

ZhewenFu commented on PR #6596:
URL: https://github.com/apache/nifi/pull/6596#issuecomment-1308099916

   > @ZhewenFu Instead of manually sorting the component identifiers, what about replacing `HashSet` with `LinkedHashSet` in this class? That seems like it would be a less involved change. Would that resolve the non-deterministic issue?
   
   Thank you for the advice! I have tried to replace `HashSet` with `LinkedHashSet` in this class but it doesn't solve the issue. Since during the sync process, the `missingComponents` has been through several transferring. If we want to solve the issue by replacing `HashSet` with `LinkedHashSet`, we may need to do that in several classes/methods. So I use the sort here to avoid making too much change.
   
   Let me know if you think there is a better solve to this issue :)


-- 
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: issues-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org