tpalfy commented on PR #10136: URL: https://github.com/apache/nifi/pull/10136#issuecomment-3127841278
@exceptionfactory Your suggestion is exactly (well, almot) what I'm trying to do! I only touched the `PutSQL` and no other class. I added a PutSQL-specific FlowFileFilter implementation there. It does this separation in one filter cycle, but the result is basically same. Maybe the solution being complex confuscates what is really going in, but I deemed it necessary due to the fact that the `database.name` requirement only exists for `PutSQL` when the `DBCPService` is of the type `DBCPConnectionPoolLookup`. Otherwise the requirement is not there. I didn't want to add a check for this specific combination. Not to mention that this exception-based filtering is theoretically possible now for any `DBCPService` implementation. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
