search

syndesisio / connectors

Apache Camel Connectors for Syndesis
Apache License 2.0
6 stars 12 forks source link

Refactor the SQL Stored connector #84

Closed zregvart closed 6 years ago

zregvart commented 6 years ago

We currently fiddle with the AutoConfiguration to create DataSource, this could be moved to be the responsibility of the SQL Stored connector component.

The template parameter could be generated on endpoint creation in the SQL Stored connector component.

This would allow us to use the Camel Maven tooling to build the JSON descriptors and remove the workaround that removes the options that are not declared on the endpoint.

zregvart commented 6 years ago

See the discussion on https://github.com/syndesisio/connectors/pull/79

KurtStam commented 6 years ago

@zregvart I think all of this has been done, except for maybe the template generation. Is think we can leave that the way it is?