CurtHagenlocher commented on code in PR #2672:
URL: https://github.com/apache/arrow-adbc/pull/2672#discussion_r2031440057


##########
csharp/src/Drivers/Apache/Spark/README.md:
##########
@@ -26,7 +26,7 @@ but can also be passed in the call to `AdbcDatabase.Connect`.
 
 | Property               | Description | Default |
 | :---                   | :---        | :---    |
-| `adbc.spark.type`      | (Required) Indicates the Spark server type. One of 
`databricks`, `http` (future: `standard`) | |
+| `adbc.spark.type`      | (Required) Indicates the Spark server type. 
Currently only `http` (future: `standard`) | |

Review Comment:
   I don't think `standard` would apply to Databricks which is likely 
always/only `http`. (I seem to recall that `standard` is a TCP-based Thrift 
transport but might be misremembering.)



-- 
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]

Reply via email to