rafaelsmf commented on PR #1570: URL: https://github.com/apache/fluss/pull/1570#issuecomment-3218277294
> Hi @rafaelsmf, thanks for your contribution! > > I noticed the main change in this PR is renaming `Engine Flink` to `Flink Engine`. While we appreciate the suggestion, the current naming (`Engine Flink`) is intentional. > > As we're expanding support to other engines like Spark and Trino, using a consistent prefix — such as `Engine Flink`, `Engine Spark`, `Engine Trino` — helps keep the naming structure uniform and more organized across the project. > > Let’s keep it as `Engine Flink` for now to maintain consistency. Thanks again for your input! Hi @wuchong , Thanks for the review. I now understand the structure of the Flink engine section and have reverted the proposed rename. I also fixed several typos and improved parts of the doc for readability. Additionally, I updated the URL from Alibaba to Apache as suggested. -- 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]
