yzeng1618 commented on PR #10977:
URL: https://github.com/apache/seatunnel/pull/10977#issuecomment-4585403376
> @yzeng1618 Can you help confirm if the support of OptionRulesService to
return transform has a negative impact on AI generated configuration? Thank you
very much.
I took a rough look at the CLI (#10789) consumption side and wanted to
double-check two points with you — I may not have the full picture, so please
help confirm.
First, on transform: in connectors.py, get_connector_detail() currently
defaults query_types to ["source", "sink"] only, so transform doesn't seem to
be requested at this stage. So even if OptionRulesService now returns
transform, the CLI side likely won't use it yet — supporting it properly would
probably need some follow-up adaptation on the CLI side.
Second, on the new valueConstraints field in this PR: from what I can tell
it's just returned in the response for now, and the CLI side doesn't yet have
parsing/validation logic to consume it. So it shouldn't affect the existing AI
config generation, but I'd appreciate it if you could confirm whether I'm
missing anything @SEZ9
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
