weiqingy commented on PR #2409: URL: https://github.com/apache/auron/pull/2409#issuecomment-5054877010
@Tartarus0zm Thanks for the review. Both commits already start with `[AURON #1863]`, but the PR body previously said `Part of #1863`, which only mentioned the issue without creating a formal link. I’ve changed it to `Closes #1863`, so the PR now appears as a linked PR under the issue. One thing to note: this PR only covers the native Rust implementation. The Flink converter that emits the function will come in a separate PR, and the zero-argument form is still pending. As a result, #1863 will not be fully complete when this PR merges. Would you prefer that we keep #1863 open after the merge to track the converter and zero-argument follow-ups, or should I create a dedicated sub-issue for the remaining work? Either approach works for me. -- 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]
