bbende commented on pull request #4630: URL: https://github.com/apache/nifi/pull/4630#issuecomment-755739378
If we had done this from the beginning, I definitely think we should have used a single property with an ordered list of claims, and used "claims" in the name instead of "claim", but since we have the existing property I'm not sure either approach has a major advantage... On one hand, using the existing property is easier for anyone already referencing the property in some way, and it can be done without breaking anything. On the other hand, the work is already done to add the second property, and it does create very specific semantics where the original "claim" property is a single claim, and the second one with "claims" is a list. ---------------------------------------------------------------- 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]
