Mercurial > hg > monetdb-java
comparison src/main/java/nl/cwi/monetdb/jdbc/MonetDatabaseMetaData.java @ 356:eabbc5bf7cdb
Corrected the return value of getCatalogTerm();
author | Martin van Dinther <martin.van.dinther@monetdbsolutions.com> |
---|---|
date | Wed, 19 Feb 2020 16:50:24 +0100 (2020-02-19) |
parents | f62137dd6050 |
children | 34ce6e1b1be8 |
comparison
equal
deleted
inserted
replaced
355:69803e5a5a49 | 356:eabbc5bf7cdb |
---|---|
951 return "procedure"; | 951 return "procedure"; |
952 } | 952 } |
953 | 953 |
954 /** | 954 /** |
955 * What is the database vendor's preferred term for "catalog"? | 955 * What is the database vendor's preferred term for "catalog"? |
956 * MonetDB doesn't really have them (from driver accessible) but | 956 * |
957 * from the monetdb.conf file the term "database" sounds best | 957 * MonetDB doesn't support the SQL catalog naming concept. |
958 * Only SQL schemas are supported. | |
959 * We also do not allow the name of a database to be included in | |
960 * a fully qualified name, so we shouldn't return "database" here. | |
961 * A database is a different concept than an SQL catalog. | |
958 * | 962 * |
959 * @return the vendor term | 963 * @return the vendor term |
960 */ | 964 */ |
961 @Override | 965 @Override |
962 public String getCatalogTerm() { | 966 public String getCatalogTerm() { |
963 return "database"; | 967 // MonetDB does NOT support catalogs, so also no catalog term |
968 // Some generic SQL clients (e.g. DbVisualiser) use this string in their GUI | |
969 // To avoid NPE, just return an abrev string: "cat" (we like cats) | |
970 return "cat"; | |
964 } | 971 } |
965 | 972 |
966 /** | 973 /** |
967 * Does a catalog appear at the start of a qualified table name? | 974 * Does a catalog appear at the start of a qualified table name? |
968 * (Otherwise it appears at the end). | 975 * (Otherwise it appears at the end). |