davidhcoe commented on code in PR #3022:
URL: https://github.com/apache/arrow-adbc/pull/3022#discussion_r2187788880
##########
csharp/src/Drivers/BigQuery/BigQueryParameters.cs:
##########
@@ -68,5 +69,10 @@ internal class BigQueryConstants
// default value per
https://pkg.go.dev/cloud.google.com/go/bigquery#section-readme
public const string DetectProjectId = "*detect-project-id*";
+
+ // What ODBC uses
Review Comment:
I updated the comment, but I wonder if using _bqadbc_temp_tables might
actually be better? On the one hand, assuming the dataset had already been
created, it would just reuse that. On the other hand, having a specific one
dedicated to ADBC could be useful in some scenarios if people want to
understand the usage patterns.
--
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]
