Open main menu

CDOT Wiki β

Changes

PostgreSQL Adapter Project - Code Review 2 Changes

598 bytes added, 22:46, 26 January 2014
no edit summary
{{Admon/obsolete}}
 
== General ==
* Run server with connections file that has no port
|-
|Why does DB add quotes to column names that where not original quoted e.g. if a column name contains $ ?
|?
|-
|Check meta data that jdbc driver returns if column names are quoted e.g. name, value, cache
|'name','value' they are quoted
|-
|Check what happens if table with column sizes that exceeds page size for a row. If there is limit, then move virtual columns to tables instead
|The default for most TOAST-able data types is that Compression will be attempted first, then out-of-line storage if the row is still too big.http://www.postgresql.org/docs/8.4/interactive/storage-toast.html
|-
|Are postgresql functions atomictransaction safe?|'''YES'''
|-
|Check if size of column storage for characters can be specified
|
|}
 
== Classes ==
|getOwner
* update all code that requires owner, since default owner now exists
|'''DONE'''
|
|-
|getDependentViews
* check for similarities with original implementation
|'''DONE''' no similarities
|
|-
|renameTable / renameColumn
* recreate the tiggers/procedures instead of dropping and recreating the helper column
|'''DONE''' **
|
|-
* change to first add tablespace then index fill
* for index fill less than 10, set to 10
|'''DONE'''
|
|-
|isValidColumnName
* use && instead
|'''DONE'''
|
|-
|toMetaDatacase
* check if value returned by super implementation works
|'''DONE'''
|
|-
|toMetaDatacase / isCaseInsensitiveColumn
* Move logic to strip enclosing quotes from returned values in to metaDatacase
|'''DONE'''
|
|-
|getCaseSensitiveName
* check if it is needed if code is implemented that strips off quotations from metadata
|'''DONE'''
|
|-
|isImplicitConversion
* start off by returning true, run unit test and for those that fail, return false
|'''DONE'''
|
|-
* When BLOB column is created, create triggers for lo_manage
* After case when columns do not match, return
|'''DONE'''
|
|-
|dropFulltextProcedure
* remove string concatenation
|'''DONE'''
|
|-
|createFulltextProcedure / dropFulltextProcedure
* make inline
|'''DONE'''
|
|-
* move drop procedure before super call
* rename unneeded buf.setLength
|'''DONE'''
|
|-
|dummyviews
* use comment to store old select statement
|'''DONE'''
|
|-
|getTableName
* implement where returns quoted tableName without schema
|'''DONE'''
|
|-
|renameTable
* merge dropFulltextTrigger and Procedure
|'''DONE'''
|
|-
|getIndexName
* use index.getName(Table)
|'''DONE'''
|
|-
* Rename Method name
* Add comment to dummy view with original statement
|'''DONE'''
|
|}
* remove appendSeparator
* add a flag (bSep) where to add separator or not
|'''DONE'''
|
|}
 
=== PostgreSQLAdapterTest===
|isComptatibleVersion
* just return true
|'''DONE'''
|
|-
|getUnicodeCheckGuard
* use simpler logic
|'''DONE'''
|
|}
 
=== sql.AllTests===
* just pass in property file
* change logic to not use loop
|'''DONE'''
|
|-
|setMetadataDatabaseProperties
* move statement for owner property to place in alphabetical order
|'''DONE'''
|
|}
 
=== SQLCursor ===
|installLoginConfig
* use sqlAdapter.appendLiteral
|'''DONE'''
|
|}
|encoding and collation
* get values from custom properties
|'''DONE'''
|
|-
|add comment that locktimeout exception is not support on postgresql
|'''DONE'''
|
|-
|add procedure to check that all contrib modules have been installed
|
|
|-
|set default owner to nexjsa
|'''DONE'''
|
|}