Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/22009#discussion_r209699841
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/SessionConfigSupport.java
---
@@ -27,10 +27,10 @@
@InterfaceStability.Evolving
public interface SessionConfigSupport extends DataSourceV2 {
--- End diff --
If we are using `TableCatalog` and an equivalent `PathCatalog` then there
is no need to pass these in options anyway. The table or path is passed
directly to the table created by the catalog. It's fine for now, unless you
want to get #21306 first and then update this to rely on the string-to-string
map.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
