You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@gluten.apache.org by "baibaichen (via GitHub)" <gi...@apache.org> on 2026/01/05 06:07:47 UTC

[PR] [VL][GLUTEN-11088] Remove excluded tests in VeloxTestSettings following Velox upgrade [incubator-gluten]

baibaichen opened a new pull request, #11359:
URL: https://github.com/apache/incubator-gluten/pull/11359

   ## What changes are proposed in this pull request?
   
   This PR removes excluded tests related to Spark 4.0 in VeloxTestSettings following the Velox upgrade. With the recent Velox upgrade, certain unit tests that were previously failing have been fixed, so they no longer need to be excluded.
   
   Main changes:
   - Remove previous failed tests in VeloxTestSettings that are now passing after Velox upgrade
   - Move GlutenSQLQueryTestSuite.scala to `org.apache.spark.sql` package for better consistency (minor refactoring)
   
   Fixes #11088
   
   ## How was this patch tested?
   
   Tested with Spark 4.0 unit test suite. Previously excluded tests now pass correctly after Velox upgrade.


-- 
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: commits-unsubscribe@gluten.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org


Re: [PR] [VL][GLUTEN-11088] Remove excluded tests in VeloxTestSettings following Velox upgrade [incubator-gluten]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #11359:
URL: https://github.com/apache/incubator-gluten/pull/11359#issuecomment-3709061173

   Run Gluten Clickhouse CI on x86


-- 
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: commits-unsubscribe@gluten.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org


Re: [PR] [GLUTEN-11088][VL] Remove excluded tests in VeloxTestSettings following Velox upgrade [incubator-gluten]

Posted by "baibaichen (via GitHub)" <gi...@apache.org>.
baibaichen merged PR #11359:
URL: https://github.com/apache/incubator-gluten/pull/11359


-- 
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: commits-unsubscribe@gluten.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org