toddmeng-db commented on code in PR #3241:
URL: https://github.com/apache/arrow-adbc/pull/3241#discussion_r2261648291
##########
csharp/src/Drivers/Apache/Hive2/HiveServer2Statement.cs:
##########
@@ -395,6 +403,9 @@ public override void Dispose()
OperationHandle = null;
}
+ // Dispose the current query result if it exists
+ ClearCurrentQueryResult();
Review Comment:
The only potential added latency is that Reader.Dispose() may delay on
certain requests to complete.
Correct me if I'm wrong, but users should not be making more than one query
from the same statement?
So if used properly, this should only add potential latency to
Statement.Dispose, which should not be an issue?
--
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]
