pvillard31 commented on pull request #5692: URL: https://github.com/apache/nifi/pull/5692#issuecomment-1032805239
@tpalfy - while I understand the willingness to not overcomplicate the code in the parent classes, I believe this is worth the effort as Snowflake may not be the only implementation we'll be seeing in the future. We can definitely think about additional implementations that would make things easier for NiFi users when interacting with SaaS-based databases. Can we keep as much as possible in the parent objects? I do share the concern about modifying and complicating the parent code but I think we have to if looking at this on the long run. Thoughts? Happy to hear additional thoughts though. @joewitt - you did comment this PR, how do you feel about it? -- 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-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
