JDBCJNDIDataStoreFactory misses the schema parameter

Andrea Aime added a comment - 26/May/09 8:33 AM The fix should be to just add parameters.put(SCHEMA.key, SCHEMA); inside protected void setupParameters(Map parameters) Was there any reason why the schema was not included among the params thought?

Justin Deoliveira added a comment - 26/May/09 9:08 AM Probably just missed. Christian, any objecetions?

Christian Mueller added a comment - 26/May/09 9:25 AM The used db schema is specified with the datasource (like user, password, connection pool settings, .....) . At least for DB2 I am absolutely sure about that fact. No need for specifying a db schema name in the JDBCJNDIDataStoreFactory. If no schema name is given, the user id is the current schema. Look at the the vendor specific DataSource properties how to specify a different schema name. I think, we have no bug here or I have to learn something :-)

Andrea Aime added a comment - 26/May/09 9:36 AM The JDBCDataStore uses the schema parameter to build up queries. I would not be surprised if setting up a container shared pool the admin would still want to have each application hit a different database schema. In fact, DBCP does not have a schema property in its configuration ( http://commons.apache.org/dbcp/configuration.html ) and the PostGIS driver does not allow for a schema specification either: http://jdbc.postgresql.org/documentation/83/connect.html#connection-parameters It seems to me for DB2 (and Oracle) we're seeing a side effect that a user has a its own default schema (usually with the same name as the user) but this is not the rule in other databases.

Justin Deoliveira added a comment - 26/May/09 9:42 AM That makes sense... sometimes the schema is "built-in" (for lack of a better term) to the connection, and sometimes not. So... I think the best way forward would be to add the schema parameter (optional) and db2 and oracle can ignore it.

Christian Mueller added a comment - 26/May/09 10:12 AM In postgres a user has also his default schema http://sql-info.de/en/postgresql/schemas.html#Practical%20schema%20usage Anyways, specifying the schema name along with the datasource is not supported by all databases, we should add the schema parameter(optional) Who ?

Andrea Aime added a comment - 27/May/09 2:12 AM Do we want it to be general, and thus show up in DB2 as well, or not? I guess the simplest way to go, since apparently schema support in the connection is not common, would be to have the schema optional as you say, and eventually have db2 jndi factory override the setupParameters method to skip it?

Christian Mueller added a comment - 27/May/09 5:43 AM Make it general and slimple , DB2 should offer both possibilities.

Christian Mueller added a comment - 28/May/09 10:31 AM Fixed on 2.5.x and on 2.6.x, added SCHEMA parameter