Extend the allowed Debezium properties#31
Extend the allowed Debezium properties#31gayaldassanayake wants to merge 29 commits intoballerina-platform:mainfrom
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is ❌ Your project check has failed because the head coverage (75.30%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #31 +/- ##
============================================
- Coverage 76.56% 75.30% -1.27%
- Complexity 224 226 +2
============================================
Files 30 31 +1
Lines 1084 1466 +382
Branches 170 258 +88
============================================
+ Hits 830 1104 +274
- Misses 176 284 +108
Partials 78 78 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
2bfd18d to
e0484ce
Compare
Purpose
This pull request introduces utility support for extracting additional configuration keys and improves the structure and clarity of configuration handling for connectors.
Configuration separation and refactoring:
externStartfunction and its Java counterpart now accept two separate maps: one for Debezium-specific configs and another for listener-specific configs, instead of a single combined map. This clarifies the distinction between the two types of properties and enables more precise handling in both Ballerina and Java code. [1] [2] [3]mock_listener.balhave been refactored to separately track and initializedebeziumConfigsandlistenerConfigs, and to use new helper functions for populating these maps. [1] [2] [3] [4] [5]New utility for configuration keys:
Utilswith a static methodgetAdditionalConfigKeys, which returns configuration keys present in an options map but not defined in a typedesc (record type). This is exposed to Ballerina via a new external function. [1] [2]Other improvements:
Build tooling:
Fixes: ballerina-platform/ballerina-library#8572
Examples
Checklist