This is an automated email from the ASF dual-hosted git repository.

ntimofeev pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cayenne.git


The following commit(s) were added to refs/heads/master by this push:
     new cdc961b  Use constant lists instead of dynamic creation
cdc961b is described below

commit cdc961b3c631bbe320b016b88e00399e3e71813d
Author: Nikita Timofeev <stari...@gmail.com>
AuthorDate: Tue Apr 14 13:42:24 2020 +0300

    Use constant lists instead of dynamic creation
---
 .../src/main/java/org/apache/cayenne/dba/mysql/MySQLAdapter.java    | 4 ++--
 .../src/main/java/org/apache/cayenne/dba/oracle/OracleAdapter.java  | 6 +++---
 .../main/java/org/apache/cayenne/dba/postgres/PostgresAdapter.java  | 4 ++--
 .../java/org/apache/cayenne/dba/sqlserver/SQLServerAdapter.java     | 6 +++---
 4 files changed, 10 insertions(+), 10 deletions(-)

diff --git 
a/cayenne-server/src/main/java/org/apache/cayenne/dba/mysql/MySQLAdapter.java 
b/cayenne-server/src/main/java/org/apache/cayenne/dba/mysql/MySQLAdapter.java
index b1792f4..d9bf587 100644
--- 
a/cayenne-server/src/main/java/org/apache/cayenne/dba/mysql/MySQLAdapter.java
+++ 
b/cayenne-server/src/main/java/org/apache/cayenne/dba/mysql/MySQLAdapter.java
@@ -83,7 +83,7 @@ public class MySQLAdapter extends JdbcAdapter {
 
        protected String storageEngine;
 
-       private String[] SYSTEM_CATALOGS = new String[]{"sys", 
"information_schema", "mysql", "performance_schema"};
+       private List<String> SYSTEM_CATALOGS = Arrays.asList("sys", 
"information_schema", "mysql", "performance_schema");
 
        public MySQLAdapter(@Inject RuntimeProperties runtimeProperties,
                                                
@Inject(Constants.SERVER_DEFAULT_TYPES_LIST) List<ExtendedType> 
defaultExtendedTypes,
@@ -358,7 +358,7 @@ public class MySQLAdapter extends JdbcAdapter {
 
        @Override
        public List<String> getSystemCatalogs() {
-               return Arrays.asList(SYSTEM_CATALOGS);
+               return SYSTEM_CATALOGS;
        }
 
        final class PKComparator implements Comparator<DbAttribute> {
diff --git 
a/cayenne-server/src/main/java/org/apache/cayenne/dba/oracle/OracleAdapter.java 
b/cayenne-server/src/main/java/org/apache/cayenne/dba/oracle/OracleAdapter.java
index 453aae8..06485eb 100644
--- 
a/cayenne-server/src/main/java/org/apache/cayenne/dba/oracle/OracleAdapter.java
+++ 
b/cayenne-server/src/main/java/org/apache/cayenne/dba/oracle/OracleAdapter.java
@@ -83,13 +83,13 @@ public class OracleAdapter extends JdbcAdapter {
 
        protected static boolean supportsOracleLOB;
 
-       private String[] SYSTEM_SCHEMAS = new String[]{
+       private List<String> SYSTEM_SCHEMAS = Arrays.asList(
                        "ANONYMOUS", "APPQOSSYS", "AUDSYS", "CTXSYS", 
"DBSFWUSER",
                        "DBSNMP", "DIP", "DVF", "GGSYS", "DVSYS", 
"GSMADMIN_INTERNAL",
                        "GSMCATUSER", "GSMUSER", "LBACSYS", "MDDATA", "MDSYS", 
"OJVMSYS",
                        "OLAPSYS", "ORACLE_OCM", "ORDDATA", "ORDPLUGINS", 
"ORDSYS", "OUTLN",
                        "REMOTE_SCHEDULER_AGENT", "SYSTEM", "WMSYS", 
"SI_INFORMTN_SCHEMA",
-                       "SYS", "SYSBACKUP", "SYSDG", "SYSKM", "SYSRAC", 
"SYS$UMF", "XDB", "XS$NULL"};
+                       "SYS", "SYSBACKUP", "SYSDG", "SYSKM", "SYSRAC", 
"SYS$UMF", "XDB", "XS$NULL");
 
        static {
                // TODO: as CAY-234 shows, having such initialization done in a 
static
@@ -302,7 +302,7 @@ public class OracleAdapter extends JdbcAdapter {
 
        @Override
        public List<String> getSystemSchemas() {
-               return Arrays.asList(SYSTEM_SCHEMAS);
+               return SYSTEM_SCHEMAS;
        }
 
        /**
diff --git 
a/cayenne-server/src/main/java/org/apache/cayenne/dba/postgres/PostgresAdapter.java
 
b/cayenne-server/src/main/java/org/apache/cayenne/dba/postgres/PostgresAdapter.java
index edf5e79..2bf1c9e 100644
--- 
a/cayenne-server/src/main/java/org/apache/cayenne/dba/postgres/PostgresAdapter.java
+++ 
b/cayenne-server/src/main/java/org/apache/cayenne/dba/postgres/PostgresAdapter.java
@@ -67,7 +67,7 @@ public class PostgresAdapter extends JdbcAdapter {
 
        public static final String BYTEA = "bytea";
 
-       private String[] SYSTEM_SCHEMAS = new String[]{"information_schema", 
"pg_catalog"};
+       private List<String> SYSTEM_SCHEMAS = 
Arrays.asList("information_schema", "pg_catalog");
 
        public PostgresAdapter(@Inject RuntimeProperties runtimeProperties,
                                                   
@Inject(Constants.SERVER_DEFAULT_TYPES_LIST) List<ExtendedType> 
defaultExtendedTypes,
@@ -272,7 +272,7 @@ public class PostgresAdapter extends JdbcAdapter {
 
        @Override
        public List<String> getSystemSchemas() {
-               return Arrays.asList(SYSTEM_SCHEMAS);
+               return SYSTEM_SCHEMAS;
        }
 
 }
diff --git 
a/cayenne-server/src/main/java/org/apache/cayenne/dba/sqlserver/SQLServerAdapter.java
 
b/cayenne-server/src/main/java/org/apache/cayenne/dba/sqlserver/SQLServerAdapter.java
index 470444f..9237cd5 100644
--- 
a/cayenne-server/src/main/java/org/apache/cayenne/dba/sqlserver/SQLServerAdapter.java
+++ 
b/cayenne-server/src/main/java/org/apache/cayenne/dba/sqlserver/SQLServerAdapter.java
@@ -81,10 +81,10 @@ public class SQLServerAdapter extends SybaseAdapter {
        @Deprecated
        public static final String TRIM_FUNCTION = "RTRIM";
 
-       private String[] SYSTEM_SCHEMAS = new String[]{"db_accessadmin", 
"db_backupoperator",
+       private List<String> SYSTEM_SCHEMAS = Arrays.asList("db_accessadmin", 
"db_backupoperator",
                        "db_datareader", "db_datawriter", "db_ddladmin", 
"db_denydatareader",
                        "db_denydatawriter","dbo", "sys", "db_owner", 
"db_securityadmin", "guest",
-                       "INFORMATION_SCHEMA"};
+                       "INFORMATION_SCHEMA");
 
        public SQLServerAdapter(@Inject RuntimeProperties runtimeProperties,
                                                        
@Inject(Constants.SERVER_DEFAULT_TYPES_LIST) List<ExtendedType> 
defaultExtendedTypes,
@@ -125,7 +125,7 @@ public class SQLServerAdapter extends SybaseAdapter {
 
        @Override
        public List<String> getSystemSchemas() {
-               return Arrays.asList(SYSTEM_SCHEMAS);
+               return SYSTEM_SCHEMAS;
        }
 
 }

Reply via email to