rangadi commented on code in PR #38286:
URL: https://github.com/apache/spark/pull/38286#discussion_r998516442
##########
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:
All the sbt builds and tests are passing in ci/cd, but tests fail on my mac.
The reason is these descriptor files are copied to
_target/scala-2.12test-classes/_ rather than to
_target/scala-2.12test-classes/**protobuf**_ .
@SandishKumarHN Do you know what might be going on? I tried the same without
this PR and it works fine.
--
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]
