LuciferYang commented on code in PR #47855:
URL: https://github.com/apache/spark/pull/47855#discussion_r1728643211
##########
connector/protobuf/src/test/scala/org/apache/spark/sql/protobuf/ProtobufFunctionsSuite.scala:
##########
@@ -2101,7 +2102,9 @@ class ProtobufFunctionsSuite extends QueryTest with
SharedSparkSession with Prot
queryContext = Array(ExpectedContext(
fragment = s"to_protobuf(complex_struct, 42, '$testFileDescFile',
map())",
start = 10,
- stop = 153))
+ // The subsequent test scenarios also follow the following rules to
define `stop`:
+ // stop = (start - 1) + prefixLength + fileNameLength + suffixLength
+ stop = 9 + 33 + fileNameLength + 9))
Review Comment:
@itholic If you haven't started investigating yet, could you please help
check if this fix is feasible? Thanks
also cc @HyukjinKwon
--
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]
