advancedxy opened a new pull request, #285:
URL: https://github.com/apache/arrow-datafusion-comet/pull/285
## Which issue does this PR close?
Closes #.
## Rationale for this change
When running `org.apache.spark.sql.CometTPCHQueriesList` directly from
console/IDEA, I notice it failed with NPE. The exception track is showed as
below:
```
Exception in thread "main" java.lang.NullPointerException
at
org.apache.spark.sql.catalyst.util.package$.resourceToBytes(package.scala:67)
at
org.apache.spark.sql.catalyst.util.package$.resourceToString(package.scala:75)
at
org.apache.spark.sql.CometTPCQueryListBase.$anonfun$runQueries$1(CometTPCQueryListBase.scala:82)
at
org.apache.spark.sql.CometTPCQueryListBase.$anonfun$runQueries$1$adapted(CometTPCQueryListBase.scala:79)
at scala.collection.immutable.List.foreach(List.scala:431)
at
org.apache.spark.sql.CometTPCQueryListBase.runQueries(CometTPCQueryListBase.scala:79)
at
org.apache.spark.sql.CometTPCQueryListBase.runQueries$(CometTPCQueryListBase.scala:68)
at
org.apache.spark.sql.CometTPCHQueriesList$.runQueries(CometTPCHQueriesList.scala:43)
at
org.apache.spark.sql.CometTPCHQueriesList$.runSuite(CometTPCHQueriesList.scala:69)
at
org.apache.spark.sql.CometTPCQueryListBase.main(CometTPCQueryListBase.scala:59)
at
org.apache.spark.sql.CometTPCQueryListBase.main$(CometTPCQueryListBase.scala:42)
at
org.apache.spark.sql.CometTPCHQueriesList$.main(CometTPCHQueriesList.scala:43)
at
org.apache.spark.sql.CometTPCHQueriesList.main(CometTPCHQueriesList.scala)
```
The reason is that there's only a `q1` query in the
`spark/src/test/resources/tpch-extended` dir, loading other queries such as
`q2`, `q3` will result an NPE.
## What changes are included in this PR?
Make sure only "q1" is in the tpch extended queries list.
## How are these changes tested?
Manul test
--
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]
