ppalaga commented on a change in pull request #2748:
URL: https://github.com/apache/camel-quarkus/pull/2748#discussion_r648058770
##########
File path: docs/modules/ROOT/pages/reference/extensions/sql.adoc
##########
@@ -67,6 +67,25 @@ Note that URI schemes such as `file` or `http` do not need
to be listed.
quarkus.camel.sql.script-files = queries.sql, classpath:sql/insert.sql
----
+=== SQL Aggregator ===
+
+If your exchanges in native mode contain objects other than:
+
+* `java.util.LinkedHashMap`
+* `java.util.HashMap`
+* `java.lang.String`
+* `java.lang.Number`
+* `java.lang.Integer`
Review comment:
OK, now I understand much better how this was intended. Thanks for
explaining!
I think that because this is going to be a temporary solution, we should not
invest too much into it and we should try to avoid changes that we'll have to
revert later thus risking regressions in end user apps.
So I'd vote for moving the whole `serialization-config.json` to the test
module and for leaving it up to the end users to maintain their
`serialization-config.json` in their apps. Our SQL extension docs should
contain a full listing of our `serialization-config.json` so that end users can
copy&paste it to their repos. WDYT?
--
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:
[email protected]
