doris-flink-connector icon indicating copy to clipboard operation
doris-flink-connector copied to clipboard

[Enhance](cdc) Add support for environment variable resolution in parameter mapping

Open buu-nguyen opened this issue 7 months ago • 3 comments

Proposed changes

Problem Summary:

This PR adds support for resolving environment variables within the parameter mapping for CDC tools. It updates the getConfigMap method to substitute placeholders like $VAR or ${VAR} with their corresponding environment variable values.

Checklist(Required)

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

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

buu-nguyen avatar May 21 '25 04:05 buu-nguyen

Thank you for your contribution, can this be added as a case, something like this? https://github.com/apache/doris-flink-connector/blob/master/flink-doris-connector/src/test/resources/container/e2e/mysql2doris/testMySQL2Doris.txt

JNSimba avatar May 27 '25 06:05 JNSimba

Thank you for your contribution, can this be added as a case, something like this? https://github.com/apache/doris-flink-connector/blob/master/flink-doris-connector/src/test/resources/container/e2e/mysql2doris/testMySQL2Doris.txt

Hi @JNSimba, i already added an E2E test

buu-nguyen avatar May 30 '25 15:05 buu-nguyen

ci failed, PTAL.@buu-nguyen

JNSimba avatar Jun 11 '25 06:06 JNSimba