Skip to content

Introduce base blaze sql test suite#674

Merged
richox merged 1 commit intoapache:masterfrom
wForget:dev
Dec 2, 2024
Merged

Introduce base blaze sql test suite#674
richox merged 1 commit intoapache:masterfrom
wForget:dev

Conversation

@wForget
Copy link
Member

@wForget wForget commented Nov 29, 2024

Which issue does this PR close?

no

Rationale for this change

Introduce base blaze sql test suite to facilitate writing test cases

What changes are included in this PR?

  • Move native lib to native-engine/_build (avoid being cleaned up by mvn clean)
  • Place native lib to spark-extension module resource (for test classpatch)
  • Add skipBuildNative property to allow skipping native build
  • Add BaseBlazeSQLSuite and BlazeAdaptiveQueryExecSuite

Are there any user-facing changes?

import testImplicits._

// Copy from spark/sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala
test("SPARK-35725: Support optimize skewed partitions in RebalancePartitions") {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A test case for #662

@richox richox merged commit 0b68ae5 into apache:master Dec 2, 2024
wForget added a commit to wForget/blaze that referenced this pull request Dec 5, 2024
richox pushed a commit that referenced this pull request Dec 6, 2024
@richox richox mentioned this pull request Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants