Re: cvs commit: db-torque/src/generator/src/java/org/apache/torque/engine/platform PlatformPostgresqlImpl.java PlatformHypersonicImpl.java PlatformFactory.java PlatformMysqlImpl.java PlatformAxionImpl.java PlatformMssqlImpl.java PlatformDb2Impl.java PlatformDefaultImpl.java Platform.java PlatformSapdbImpl.java PlatformInterbaseImpl.java PlatformOracleImpl.java PlatformCloudscapeImpl.java package.html PlatformMsaccessImpl.java PlatformSybaseImpl.java PlatformDb2400Impl.java

[Martin Poeschl]

maybe this will work .. but i'mn not sure if we will need some business 
logic within the platform classes for sql generation .. so i'll finnish 
the sql generation and we can do some refactoring later ..

martin

Daniel L. Rall wrote:

Most of these properties seem better stored in a .properties which is 
built into the JAR.  How about a base class (PlatformDefaultImpl) the 
common set of accessors along side PlatformFoo.properties files?  The 
base class could take the database platform name as a new additional 
parameter to its constructor, so that information need not be lost 
while removing all the unnecessary sub-classes.

[EMAIL PROTECTED] wrote:

mpoeschl    2003/09/27 04:10:16

  Added:       src/generator/src/java/org/apache/torque/engine/platform
                        PlatformPostgresqlImpl.java
                        PlatformHypersonicImpl.java PlatformFactory.java
                        PlatformMysqlImpl.java PlatformAxionImpl.java
                        PlatformMssqlImpl.java PlatformDb2Impl.java
                        PlatformDefaultImpl.java Platform.java
                        PlatformSapdbImpl.java 
PlatformInterbaseImpl.java
                        PlatformOracleImpl.java 
PlatformCloudscapeImpl.java
                        package.html PlatformMsaccessImpl.java
                        PlatformSybaseImpl.java PlatformDb2400Impl.java
  Log:
  add platform package
  this packages replaces the db.props files and will be used to 
generate the sql in the near future
    Revision  Changes    Path 



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]