markap14 commented on issue #3387: NIFI-6009 ScanKudu Processor & KuduPut Processor Delete Operation URL: https://github.com/apache/nifi/pull/3387#issuecomment-481402688 @SandishKumarHN Thanks for responding so quickly. Re: the Controller Service - @granthenke can correct me if I am wrong, but I don't know that reusing the client provides a huge upside here. It probably doesn't hurt but I don't know that we really gain much either. In terms of code reuse, I think we can easily accomplish that in other ways, perhaps with an abstract class or helper methods. A change that makes a flow invalid is pretty much a last resort. If there were some sort of security vulnerability or something like that, or if we find that we're doing something egregious to the server side then it would make sense to consider the change. Can you explain your third thought there, regarding CDC operations? Can you outline the use case where this helps? As you're describing it, I think such a race condition could still exist. If you wanted to INSERT some records and DELETE some records, you'd still need a PutKudu processor configured to INSERT and another PutKudu Processor configured to DELETE - unless I'm missing something?
---------------------------------------------------------------- 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] With regards, Apache Git Services
