toddmeng-db commented on code in PR #2842:
URL: https://github.com/apache/arrow-adbc/pull/2842#discussion_r2099079415
##########
csharp/test/Drivers/Databricks/StatementTests.cs:
##########
@@ -436,13 +436,12 @@ protected override void
PrepareCreateTableWithPrimaryKeys(out string sqlUpdate,
// NOTE: this is a thirty minute test. As of writing, databricks
commands have 20 minutes of idle time (and checked every 5 mintues)
[SkippableTheory]
- [InlineData(true, "CloudFetch enabled")]
- [InlineData(false, "CloudFetch disabled")]
+ [InlineData(true, "CloudFetch enabled")] // TODO: test cloudfetch
disabled
public async Task StatusPollerKeepsQueryAlive(bool useCloudFetch,
string configName)
Review Comment:
Discussed offline: we'll need to seperately take a look into handling url
expiration, then we should re-enable this test for cloudfetch.
--
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]
