MikeThomsen commented on pull request #3317:
URL: https://github.com/apache/nifi/pull/3317#issuecomment-716047476


   @adamfisher I had to fix a few things locally. Not sure if it was due to API 
changes over the last year, but I have your contribution built and passing the 
tests locally. Overall, I'm +1 on your contribution. However, since we have two 
implementations of the same idea there needs to be some reconciliation of the 
two. I'm going to work on that, possibly under my branch/Jira ticket.
   
   In the end, I think what we're going to do here is have three strategies:
   
   1. Hash everything.
   2. Bloom Filter
   3. The UUID strategy I mentioned.
   
   I'm going to talk to a colleague of mine who is working with me on a related 
use case and see what sort of implementation makes sense for #3.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to