github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[jOOQ/jOOQ#9574] The PUBLIC pseudo schema should be invisible

Browse Source 
Otherwise, the default catalog will report two distinct default schemas, with all the side effects that ensue
This commit is contained in:
Lukas Eder 2024-10-02 12:39:29 +02:00
parent c3f8894625
commit f9d21eedda
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
jOOQ/src/main/java/org/jooq/impl/Interpreter.java
View File

@ -174,8 +174,8 @@ final class Interpreter {
this.locale = interpreterLocale(configuration.settings());
this.defaultCatalog = new MutableCatalog(NO_NAME);
this.catalogs.put(defaultCatalog.name(), defaultCatalog);
this.defaultSchema = new MutableSchema(NO_NAME, defaultCatalog);
this.publicSchema = new MutableSchema(NO_NAME, defaultCatalog);
this.defaultSchema = new MutableSchema(NO_NAME, defaultCatalog, true);
this.publicSchema = new MutableSchema(NO_NAME, defaultCatalog, false);
}
final Meta meta() {
@ -1456,7 +1456,7 @@ final class Interpreter {
MutableSchema schema = defaultSchema;
if ((schema = find(catalog.schemas, input)) == null && create)
// TODO createSchemaIfNotExists should probably be configurable
schema = new MutableSchema((UnqualifiedName) input.getUnqualifiedName(), catalog);
schema = new MutableSchema((UnqualifiedName) input.getUnqualifiedName(), catalog, true);
return schema;
}
@ -1801,11 +1801,13 @@ final class Interpreter {
MutableSchema(UnqualifiedName name, MutableCatalog catalog) {
MutableSchema(UnqualifiedName name, MutableCatalog catalog, boolean visible) {
super(name);
this.catalog = catalog;
this.catalog.schemas.add(this);
if (visible)
this.catalog.schemas.add(this);
}
@Override