You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by "nandorsoma (via GitHub)" <gi...@apache.org> on 2023/02/28 21:25:08 UTC

[GitHub] [nifi] nandorsoma commented on pull request #6990: NIFI-11205 Adding deprecation log for redundant Kerberos properties in Kafka processors

nandorsoma commented on PR #6990:
URL: https://github.com/apache/nifi/pull/6990#issuecomment-1448936748

   Thank you for the review, @exceptionfactory! I didn't know that the logger should reference the componentClass, but it is a good idea to pass it instead of the processorName.
   Regarding the new properties in the 2_0 processors... Maybe I misunderstood the concept of the deprecation logs. I didn’t know we needed to provide an alternative for the deprecated component on the same main version. Is it really a requirement? The message is unlucky because it suggests that, but with a little more detail, it would be clear. If it is unacceptable, I will favor removing 2_0 processors because I think it would be wrong to create a precedence to add a non-bugfix change to this branch.


-- 
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-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org