[GitHub] [kafka] C0urante commented on pull request #14064: KAFKA-15030: Add connect-plugin-path command-line tool.

2023-08-03 Thread via GitHub
C0urante commented on PR #14064: URL: https://github.com/apache/kafka/pull/14064#issuecomment-1664304087 I also understand the rationale for listing compatible plugin locations, and if we can accurately report this, then I think it'd be worth it to include that information. However, I'm

[GitHub] [kafka] C0urante commented on pull request #14064: KAFKA-15030: Add connect-plugin-path command-line tool.

2023-08-03 Thread via GitHub
C0urante commented on PR #14064: URL: https://github.com/apache/kafka/pull/14064#issuecomment-1664301147 Hmm... seems like we're still failing to detect missing manifests for plugins that implement multiple interfaces. I added a `Converter` manifest for the JSON converter (but not a

[GitHub] [kafka] C0urante commented on pull request #14064: KAFKA-15030: Add connect-plugin-path command-line tool.

2023-08-01 Thread via GitHub
C0urante commented on PR #14064: URL: https://github.com/apache/kafka/pull/14064#issuecomment-1660814545 Hmmm... getting an NPE when I try to run the command now: ``` Cannot invoke "java.util.Set.removeIf(java.util.function.Predicate)" because the return value of