[GitHub] [kafka] yashmayya commented on pull request #14093: KAFKA-15248 Add BooleanConverter

["yashmayya (via GitHub)" <gi...@apache.org>]

yashmayya commented on PR #14093:
URL: https://github.com/apache/kafka/pull/14093#issuecomment-1731222665

   @hgeraldino the Connect integration tests are failing with errors like:
   
   ```
   org.apache.kafka.connect.errors.ConnectException: One or more plugins are missing ServiceLoader manifests may not be usable with plugin.discovery=service_load: [
   classpath	org.apache.kafka.connect.converters.BooleanConverter	converter	undefined
   classpath	org.apache.kafka.connect.converters.BooleanConverter	header_converter	undefined
   ]
   Read the documentation at https://kafka.apache.org/documentation.html#connect_plugindiscovery for instructions on migrating your plugins to take advantage of the performance improvements of service_load mode. To silence this error, set plugin.discovery=hybrid_warn in the worker config.
   ```
   
   This is related to [KIP-898](https://cwiki.apache.org/confluence/display/KAFKA/KIP-898:+Modernize+Connect+plugin+discovery) and specifically changes that were made in https://github.com/apache/kafka/pull/14055. Could you please update the [Converter](https://github.com/apache/kafka/blob/5bdea94c0582104fd1d471918dc143aa9bee67d3/connect/runtime/src/main/resources/META-INF/services/org.apache.kafka.connect.storage.Converter#L15) and [HeaderConverter](https://github.com/apache/kafka/blob/5bdea94c0582104fd1d471918dc143aa9bee67d3/connect/runtime/src/main/resources/META-INF/services/org.apache.kafka.connect.storage.HeaderConverter#L15) provider-configuration files to include the new `BooleanConverter`?


-- 
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: jira-unsubscribe@kafka.apache.org

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