laglangyue commented on code in PR #43526:
URL: https://github.com/apache/spark/pull/43526#discussion_r1402950365
##########
sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamingJoinSuite.scala:
##########
@@ -541,7 +542,7 @@ class StreamingInnerJoinSuite extends StreamingJoinSuite {
Utils.createDirectory(tempDir.getAbsolutePath,
Random.nextFloat().toString).toString
val stateInfo = StatefulOperatorStateInfo(path, queryId, opId, 0L, 5)
- implicit val sqlContext = spark.sqlContext
+ implicit val sqlContext: SQLContext = spark.sqlContext
val coordinatorRef = sqlContext.streams.stateStoreCoordinator
Review Comment:
well, maybe. For local variable, we don't need to explicit type, I revert
some change. Under the this case, just remove line 545, it's ok. what do you
think?
--
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]
