xiedeyantu commented on PR #4584: URL: https://github.com/apache/calcite/pull/4584#issuecomment-3411634730
> We should probably remove other tpcds stubs that appear in the code. That should probably be a different issue. I agree. I also briefly checked the previous TPCDS tests and noticed they have been in a disabled state all along. I'm not quite sure why that is. However, the test case written by @bestbeforetoday looks pretty good. Should we consider standardizing its format and keeping it, or should we modify the existing ones? I don't have many strong opinions on this. -- 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]
