ulysses-you commented on code in PR #57603:
URL: https://github.com/apache/spark/pull/57603#discussion_r3670707825
##########
sql/core/src/test/scala/org/apache/spark/sql/execution/ProjectedOrderingAndPartitioningSuite.scala:
##########
@@ -31,6 +32,8 @@ class ProjectedOrderingAndPartitioningSuite
extends SharedSparkSession with AdaptiveSparkPlanHelper {
import testImplicits._
+ setupTestData()
Review Comment:
Good point, done in e16b666. Since `setupTestData()` only flips the
`loadTestDataBeforeTests` flag consumed in `beforeAll`, I register the view by
touching the `testData` lazy val (which calls `createOrReplaceTempView` as a
side effect) at the start of each of the four new tests.
--
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]
