LuciferYang commented on PR #12889: URL: https://github.com/apache/gluten/pull/12889#issuecomment-5409509926
Folding this into #12888 and closing here. The split made sense while this was an open question: it removed four ClickHouse exclusions to find out from CI whether they were still needed, and bundling an experiment with an unknown outcome into a PR that was ready to merge would have held that one up. The experiment is done, the ClickHouse CI failed all four, and what is left is eight lines of comments recording why. That is the same activity as #12888, on a file #12888 already modifies, so keeping them apart only buys a second review and a rebase for whichever lands second. The CI evidence and the resulting comments are carried over verbatim. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
