Commit c88df2c
[SPARK-28331][SQL] Catalogs.load() should be able to load built-in catalogs
## What changes were proposed in this pull request?
In `Catalogs.load`, the `pluginClassName` in the following code
```
String pluginClassName = conf.getConfString("spark.sql.catalog." + name, null);
```
is always null for built-in catalogs, e.g there is a SQLConf entry `spark.sql.catalog.session`.
This is because of #18852: SQLConf.conf.getConfString(key, null) always returns null.
## How was this patch tested?
Apply code changes of #24768 and tried loading session catalog.
Closes #25094 from gengliangwang/fixCatalogLoad.
Authored-by: Gengliang Wang <[email protected]>
Signed-off-by: Burak Yavuz <[email protected]>1 parent 8617bf6 commit c88df2c
File tree
1 file changed
+5
-2
lines changed- sql/catalyst/src/main/java/org/apache/spark/sql/catalog/v2
1 file changed
+5
-2
lines changedLines changed: 5 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
| 29 | + | |
29 | 30 | | |
30 | 31 | | |
31 | 32 | | |
| |||
50 | 51 | | |
51 | 52 | | |
52 | 53 | | |
53 | | - | |
54 | | - | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
55 | 58 | | |
56 | 59 | | |
57 | 60 | | |
| |||
0 commit comments