junyeong0619 commented on PR #66022: URL: https://github.com/apache/airflow/pull/66022#issuecomment-4435321470
> Thanks for working on this — the layered architecture is right, and the tests cover all four layers (API, supervisor, SDK, version-gate). I have a few concerns inline, ranging from blockers (cadwyn version date placement, SQL `LIKE` metacharacter handling, return-type annotation, silent error masking, unbounded result set) to nice-to-haves (dead branch, routing-order fragility, docstring clarity). Happy to discuss any of them. > > --- > Drafted-by: Claude Code (Opus 4.7); reviewed by @potiuk before posting I’ve completed all the tasks. Could you please take a look. -- 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]
