zabetak commented on PR #3145: URL: https://github.com/apache/calcite/pull/3145#issuecomment-1537089353
> But I would appreciate help to decide how to package this tool. @mihaibudiu From my perspective (and Julian's I think) this PR should contain something equivalent to what is now `TestCalcite` inside the `plus` module and allow all SQL logic tests that pass to run. It should probably contain `CalciteExecutor` but nothing more. We just want to run to be able to run the tests as part of the Calcite suite but nothing more for now. If you want I can push the relevant commits following this idea in this PR. -- 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]
