CALL FOR: Removing old supportclasses and MMJDBC2NodeInterface
Called by: Pierre van Rooden Total tally on this call : +8
START OF VOTING: 20-01-2005 12:00 END OF CALL: 25-01-2005 12:00
YEA (8) : Marcel Maatkamp, Andre van Toly, Rico Jansen, Rob Vermeulen, Daniel Ockeloen, Rob van Maris, Kees Jongenburger, Michiel Meeuwissen
ABSTAIN (3) : Nico Klasens, Johannes Verelst, Eduard Witteveen
NAY (0) :
VETO (0) :
No votes, assumed abstained (4): Jaco de Groot, Gerard van Enk, Mark Huijser, Ernst Bunders
Result: call succeeded.
The old code will be removed.
Testing will be required, and fixes provided for any bugs occurring due to the removal of the old support classes.
CALL FOR: Removing old supprtclasses and MMJDBC2NodeInterface
Called by: Pierre van Rooden Total tally on this call : 0
This step involves the removal of all old support code, in particular the package:
* org.mmbase.module.database.support.*
This also involves removing all references to MMJDBC2NodeInterface (acecssible with the MMBase.getDatabase() method), replacing it with calls to either searchquery or the storagemanager.
This step needs a proposal and a vote to allow for the deletion of the support classes.
Notes:
- MultiRelations is replaced with the use of ClusterBuilder
- JDBC2NodeWrapper is obsolete
- XMLDatabaseReader and DatabaseReader are no longer needed
- A lot of utility methods have been removed form MMObjectBuilder. Parsing of MMNODE syntax has been moved to the SCAN application (to the HtmlBase class).
- The parsing and querying of Clusterbuilder has been removed - Clusterbuilder now only uses searchquery, and no longer falls back to the old code.
- SearchQuery, for the moment, still uses it's own connection, but obtains it through the StorageManager's 'datasource' propery. It uses the setValue() method of DatabseStorageManager to read field values from a recordset. This method has been made public for that purpose. In the future we will look for a way to move querying of nodes to the storage layer.
- The build has been adapted to no longer check on a variety of optional packages - either because they are no longer needed (Informix, Oracle), or because they have become mandatory (naming, datasource).
- Support for Oracle will become, for the moment, unavailable in 1.8. This is regretable but unavoidable as there are currently no resources to implement proper support. Efforts to port Oracle support to the new storage layer are welcomed.
Please note that the code changes have been tested, but with a recently created site. We expect there will be some issue swith older sites, which we plan to fix as soon as they surface. We invite people to test their sites if and once the code has been placed in CVS Head.
This change is for MMBase 1.8.
START OF VOTING: 20-01-2005 12:00 END OF CALL: 25-01-2005 12:00
[_] +1 (YES) [_] +0 (ABSTAIN ) [_] -1 (NO), because : [_] VETO, because:
The following is a list of classes that are to be changed or removed.
Note that the changes are extensive and as such I did not include the sources, though I can zip the source tree if people like.
M build.xml
M applications/cloudcontext/src/org/mmbase/security/implementation/cloudcontext/builders/Contexts.java
M applications/cloudcontext/src/org/mmbase/security/implementation/cloudcontext/builders/Groups.java
M applications/packaging/src/org/mmbase/applications/packaging/projects/creators/dataapptools/NodeWriter.java
M applications/scan/src/org/mmbase/module/sessions.java
M applications/scan/src/org/mmbase/module/builders/Cookies.java
M applications/scan/src/org/mmbase/module/gui/html/HtmlBase.java
M applications/scan/src/org/mmbase/module/gui/html/ObjectSelector.java
M applications/vwms/src/org/mmbase/module/builders/Vwmtasks.java
M applications/xmlimporter/src/org/mmbase/applications/xmlimporter/BasicFinder.java
M src/org/mmbase/bridge/implementation/BasicCloud.java
M src/org/mmbase/bridge/implementation/BasicNodeManager.java
M src/org/mmbase/bridge/util/Queries.java
M src/org/mmbase/module/builders/DayMarkers.java
R src/org/mmbase/module/builders/MultiRelations.java
M src/org/mmbase/module/builders/PropertiesProbe.java
M src/org/mmbase/module/builders/VersionCacheNode.java
M src/org/mmbase/module/core/ClusterBuilder.java
M src/org/mmbase/module/core/MMBase.java
M src/org/mmbase/module/core/MMObjectBuilder.java
M src/org/mmbase/module/core/MMObjectNode.java
M src/org/mmbase/module/core/MMTable.java
R src/org/mmbase/module/database/support/BaseJdbc2Node.java
R src/org/mmbase/module/database/support/MMHsqldb2Node.java
R src/org/mmbase/module/database/support/MMHypersonic2Node.java
R src/org/mmbase/module/database/support/MMInformix42Node.java
R src/org/mmbase/module/database/support/MMJdbc2NodeInterface.java
R src/org/mmbase/module/database/support/MMMckoiNode.java
R src/org/mmbase/module/database/support/MMMysql42Node.java
R src/org/mmbase/module/database/support/MMOORel2Node.java
R src/org/mmbase/module/database/support/MMOracle.java
R src/org/mmbase/module/database/support/MMPostgres42Node.java
R src/org/mmbase/module/database/support/MMSQL92Node.java
R src/org/mmbase/module/database/support/PostgreSQL.java
R src/org/mmbase/module/database/support/PostgreSQL71.java
R src/org/mmbase/module/database/support/PostgreSQL72.java
R src/org/mmbase/module/database/support/Sql92SingleFields.java
R src/org/mmbase/module/database/support/Sql92WithViews.java
R src/org/mmbase/module/database/support/dTypeInfo.java
R src/org/mmbase/module/database/support/dTypeInfos.java
R src/org/mmbase/module/database/support/package.html
M src/org/mmbase/module/tools/MMAdmin.java
M src/org/mmbase/security/implementation/cloud/UserBuilder.java
M src/org/mmbase/storage/StorageManagerFactory.java
M src/org/mmbase/storage/implementation/database/DatabaseStorageManager.java
M src/org/mmbase/storage/implementation/database/DatabaseStorageManagerFactory.java
R src/org/mmbase/storage/implementation/database/JDBC2NodeWrapper.java
M src/org/mmbase/storage/search/implementation/database/BasicQueryHandler.java
M src/org/mmbase/storage/search/implementation/database/BasicSqlHandler.java
M src/org/mmbase/storage/search/implementation/database/ChainedSqlHandler.java
M src/org/mmbase/storage/search/implementation/database/HSqlSqlHandler.java
M src/org/mmbase/storage/search/implementation/database/InformixSqlHandler.java
M src/org/mmbase/storage/search/implementation/database/MySqlSqlHandler.java
M src/org/mmbase/storage/search/implementation/database/PostgreSqlSqlHandler.java
M src/org/mmbase/storage/search/implementation/database/SqlHandler.java
M src/org/mmbase/storage/search/implementation/database/informix/excalibur/EtxSqlHandler.java
M src/org/mmbase/storage/search/legacy/ConstraintParser.java M src/org/mmbase/util/NodeWriter.java M src/org/mmbase/util/QueryConvertor.java R src/org/mmbase/util/XMLDatabaseReader.java M src/org/mmbase/util/XMLEntityResolver.java R src/org/mmbase/util/xml/DatabaseReader.java
-- Pierre van Rooden Mediapark, C 107 tel. +31 (0)35 6772815 "Anything worth doing is worth overdoing."
_______________________________________________ Developers mailing list [email protected] http://lists.mmbase.org/mailman/listinfo/developers
