xiedeyantu commented on PR #4724: URL: https://github.com/apache/calcite/pull/4724#issuecomment-3782027829
> > > I hope we can add a test case in SqlToRelTest. It's not necessary to have completely identical metadata; using existing table names is fine, and this test case can clearly show how $cor0 is defined. > > > > > > If you find the name confusing, can I change emps to emps_tmp? > > I suggest adding test cases to SqlToRelTest. You can directly use the metadata from SqlToRelTest instead of simulating the same table and column names.The current test cases in .iq are fine for me. I have added test cases and change the table name in blank.iq; please check if they meet your requirements. -- 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]
