jamesnetherton commented on a change in pull request #1278: URL: https://github.com/apache/camel-quarkus/pull/1278#discussion_r433862805
########## File path: integration-tests/debezium/pom.xml ########## @@ -39,8 +40,11 @@ <!-- mvn process-resources -Pformat from the root directory --> <mvnd.builder.rule>camel-quarkus-debezium-postgres-deployment,camel-quarkus-debezium-mysql-deployment,camel-quarkus-support-policy-deployment</mvnd.builder.rule> Review comment: `camel-quarkus-debezium-sqlserver-deployment` seems to be missing. ########## File path: extensions/debezium-sqlserver/runtime/src/main/doc/usage.adoc ########## @@ -0,0 +1,13 @@ +=== Encryption + +SQL server may require encryption. In that case `quarkus.ssl.native=false` parameter has to be added during execution. +See https://quarkus.io/guides/native-and-ssl for more details. + +=== Integration testing Review comment: Maybe move this section into a dedicated README in the debezium itest module. It's not really required information for the most users. ---------------------------------------------------------------- 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: us...@infra.apache.org