hequn8128 commented on issue #6236: [FLINK-9699] [table] Add api to replace registered table URL: https://github.com/apache/flink/pull/6236#issuecomment-436275448 @zjffdu Thanks for the PR. I think this feature is very useful. I haven't taken a deep look at the code. Considering the api, how about change these `register()` functions with replace parameter to `registerOrReplaceXXX()` ? It makes our api more alignment with SQL, such as CREATE OR REPLACE. What do you guys think here? @twalthr @yanghua
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
