SandishKumarHN commented on code in PR #38286:
URL: https://github.com/apache/spark/pull/38286#discussion_r998669256
##########
connector/protobuf/src/test/scala/org/apache/spark/sql/protobuf/ProtobufCatalystDataConversionSuite.scala:
##########
@@ -35,18 +36,32 @@ class ProtobufCatalystDataConversionSuite
with SharedSparkSession
with ExpressionEvalHelper {
- private def checkResult(
+ private val testFileDesc =
testFile("protobuf/catalyst_types.desc").replace("file:/", "/")
Review Comment:
@rangadi correct I'm also seeing sbt test failure, in the spark build the
spark-protobuf tests are not running. I have made the changes here
https://github.com/apache/spark/pull/38212/files#diff-48c0ee97c53013d18d6bbae44648f7fab9af2e0bf5b0dc1ca761e18ec5c478f2
can you take this change so that PR build runs with sbt 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]
