apache / doris-flink-connector

Flink Connector for Apache Doris
https://doris.apache.org/
Apache License 2.0
315 stars 221 forks source link

[improve]Improve the hardcode of tableConfig #449

Closed DongLiang-0 closed 2 months ago

DongLiang-0 commented 2 months ago

Proposed changes

Issue Number: close #xxx

Problem Summary:

The early table-conf passed a Map<String, String> as a parameter, and the parameters included not only the PROPERTIES() attributes that can be directly used to create doris tables, but also the data that needs to be parsed as a table creation statement. Such as table-buckets. As a result, in our actual use process, these mixed parameters need to be parsed at various stages, and the logic is also very complicated, because we abstract the parameters of table-conf as a DorisTableConfig object

Checklist(Required)

  1. Does it affect the original behavior: (Yes/No/I Don't know)
  2. Has unit tests been added: (Yes/No/No Need)
  3. Has document been added or modified: (Yes/No/No Need)
  4. Does it need to update dependencies: (Yes/No)
  5. Are there any changes that cannot be rolled back: (Yes/No)

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...