wForget commented on code in PR #5161:
URL: https://github.com/apache/kyuubi/pull/5161#discussion_r1296992037
##########
externals/kyuubi-jdbc-engine/src/main/scala/org/apache/kyuubi/engine/jdbc/util/ResultSetWrapper.scala:
##########
@@ -30,6 +30,7 @@ class ResultSetWrapper(statement: Statement)
private lazy val metadata = currentResult.getMetaData
override def hasNext: Boolean = {
+ if (currentResult.isClosed) return false
Review Comment:
`FetchResult` may be called again when the previous resultSet has ended, so
we need to determine whether the resultSet has been closed.
https://github.com/apache/kyuubi/blob/f05aefb86643e69c64237ebfb9db208ec509f3b2/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiQueryResultSet.java#L328
--
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]
