[GitHub] [nifi] joewitt commented on pull request #4641: NIFI-6394 - Ability to export contents of a List Queue

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

joewitt commented on pull request #4641:
URL: https://github.com/apache/nifi/pull/4641#issuecomment-726155889


   My concern here is similar in spirt to the statements/assumptions being made on https://github.com/apache/nifi/pull/4221 and its associated PR....
   
   Changing this requires a full understanding of all implications.  Being able to access information beyond what is swapped out *cannot* result in swapped out flowfiles being swapped in or it would defeat the point of swapping which is all about memory blowout prevention.  An alternative solution also could mean we now have a new access vector for swapped out flowfiles so we can look at their information to respond with this 'view all'.  
   
   I totally understand the spirit here but the solution might well not be to actually return everything as there are very good scalability reasons why these things dont occur.  We could update the UI to say 'View All Active' (not swapped) and/or we can update the documentation to make the limitations of the convenience functions more clear.  There is/was never any intent to provide complete access to the queues.


----------------------------------------------------------------
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:
users@infra.apache.org