exceptionfactory commented on PR #10031: URL: https://github.com/apache/nifi/pull/10031#issuecomment-3175057032
> @mark-bathori , thank you for updating the plugin! One small thing — mentioning SDK version in the bundle name (couchbase-3-...) makes it feel like it is couchbase version, not the SDK version, and couchbase version 3 is very ancient (we're on major 7 now). That's a good point @chedim. Clearly the Client SDK version follows a different strategy than the Couchbase server version. From a flow designer perspective, naming the component according to the SDK version is probably more confusing. Naming the component with the Couchbase server version could also be unnecessarily limiting. Decoupling the API and implementation is a good approach, so when it comes to naming, perhaps just calling the implementation `StandardCouchbase...` is the best option. If we come to the point where different implementations require a different version, those could be named differently. What do you think @mark-bathori and @turcsanyip? -- 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]
