You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by "tigrulya-exe (via GitHub)" <gi...@apache.org> on 2024/02/28 16:08:09 UTC
[I] [Bug] Do not throw exception in KyuubiStatement#getMoreResults(int) in case of closing current result set [kyuubi]
tigrulya-exe opened a new issue, #6114:
URL: https://github.com/apache/kyuubi/issues/6114
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
### Search before asking
- [X] I have searched in the [issues](https://github.com/apache/kyuubi/issues?q=is%3Aissue) and found no similar issues.
### Describe the bug
After fixing [HIVE-7680](https://issues.apache.org/jira/browse/HIVE-7680) issue the method `getMoreResults()` of `HiveStatement` and respectively `KyuubiStatement`classes returns false instead of throwing exception. The javadoc of the `Statement#getMoreResults()`method says, that it should implicitly close any current ResultSet object, i.e. is similar to calling the `KyuubiStatement#getMoreResults(int)` method with `Statement.CLOSE_CURRENT_RESULT` argument. Therefore, in the latter case, we should also return false instead of failing with an exception.
### Affects Version(s)
master
### Kyuubi Server Log Output
_No response_
### Kyuubi Engine Log Output
_No response_
### Kyuubi Server Configurations
_No response_
### Kyuubi Engine Configurations
_No response_
### Additional context
_No response_
### Are you willing to submit PR?
- [X] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to fix.
- [ ] No. I cannot submit a PR at this time.
--
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: notifications-unsubscribe@kyuubi.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
Re: [I] [Bug] Do not throw exception in KyuubiStatement#getMoreResults(int) in case of closing current result set [kyuubi]
Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 closed issue #6114: [Bug] Do not throw exception in KyuubiStatement#getMoreResults(int) in case of closing current result set
URL: https://github.com/apache/kyuubi/issues/6114
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
Re: [I] [Bug] Do not throw exception in KyuubiStatement#getMoreResults(int) in case of closing current result set [kyuubi]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #6114:
URL: https://github.com/apache/kyuubi/issues/6114#issuecomment-1969317571
Hello @tigrulya-exe,
Thanks for finding the time to report the issue!
We really appreciate the community's efforts to improve Apache Kyuubi.
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org