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:
No Dispose only calls CloseOperation from Statement, not from reader.
Correct me if I'm wrong, but users should not be making different queries from
the same statement?
So if used properly, this should only add 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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
