Task
|
Status
|
UnitTest
|
shared Logic with PostgreSQLSchemaManager
- move logic to PostgreSQLSchemaManager
- on instantiation create reference to PostgreSQLSchemaManager
- reference schema manager for shared logic
|
DONE
|
|
variables for reflection
- change names to full method/variable name
- remove integer declaration for write modes
- get methods from interfaces and not implementations
- remove unnecessary class declarations
|
DONE
|
|
BINB_BLOB
- move check for PGXAConnection to first step and let super handle
|
DONE
|
|
s_bindFactoryArray
- change static initialization to first copy array and then overwrite with custom binds
|
DONE
|
|
appendMatchExpression
- MUTT just do append OR, look at matchOperatorTest for how to extract arguments from Pair object
|
|
|
appendSuffixHint
- change logic to use one buffer
|
DONE
|
|
appendTypeConversion
- check bool <-> string
- check for new log for if(op instead d) in MySQLAdapter
|
|
|
isUnicode
- remove table alias from query
- modify logic to check column type and if character then check unicode
|
DONE
|
|
caseInsensitive
- change from LOWER to UPPER
|
DONE
|
|
indexNameMatches
- check if toDatabaseCase is required
|
DONE
|
|
executeQuery
- if cannot get QueryExecutor instead synchronize on connection
|
DONE
|
|
BIND_BLOB
- track down why connection used for unittests has autoCommit enabled
|
DONE
|
|
unwrap
- Override unwrap and move logic in getPostgreSQLConnection to this method
|
DONE
|
|
Task
|
Status
|
UnitTest
|
column iterator
- instead use table.columnCount
|
|
|
Comparing Primitive
- no need to use getOrdinal
|
|
|
isLOB
- use max_field_size for both
|
|
|
getOwner
- update all code that requires owner, since default owner now exists
|
|
|
getDependentViews
- check for similarities with original implementation
|
|
|
appendColumnType
- Primitive.String use isCLOB, remove check from nPrecision
|
|
|
renameTable / renameColumn
- recreate the tiggers/procedures instead of dropping and recreating the helper column
|
|
|
appendColumnAlteration
- ensure BLOB data is not being lost
|
|
|
appendIndexSuffix
- remove indexfill limits check
- change to first add tablespace then index fill
- for index fill less than 10, set to 10
|
|
|
isValidColumnName
|
|
|
toMetaDatacase
- check if value returned by super implementation works
|
|
|
toMetaDatacase / isCaseInsensitiveColumn
- Move logic to strip enclosing quotes from returned values in to metaDatacase
|
|
|
getCaseSensitiveName
- check if it is needed if code is implemented that strips off quotations from metadata
|
|
|
isImplicitConversion
- start off by returning true, run unit test and for those that fail, return false
|
|
|
unlinkLargeObjects
- change to 2 implementations that take a table and a column
|
|
|
alterColumn
- When BLOB column is created, create triggers for lo_manage
- After case when columns do not match, return
|
|
|
dropFulltextProcedure
- remove string concatenation
|
|
|
createFulltextProcedure / dropFulltextProcedure
|
|
|
dropTable
- move drop procedure before super call
- rename unneeded buf.setLength
|
|
|
dummyviews
- use comment to store old select statement
|
|
|
getTableName
- implement where returns quoted tableName without schema
|
|
|
renameTable
- merge dropFulltextTrigger and Procedure
|
|
|
getIndexName
|
|
|
appendTSIncrement
- quarter use interval'3 months'
|
|
|
makeViewDummy
- Rename Method name
- Add comment to dummy view with original statement
|
|
|