Skip to content

Conversation

@BrunoSMonteiro74
Copy link
Contributor

This is an initial increment still with some parts incomplete.

Added TODO for moving utility method decodeBlobHexFromBytesToText() to read hex value from blob field. It would make more sense as decodeBlobHexFromBytesToBytes and then we would read raw data - or not.
- Use try with resources.
- Remove extra comment lines which raises warnings.
- Added missing jdoc.
Added TODO for moving utility method decodeBlobHexFromBytesToText() to read hex value from blob field. It would make more sense as decodeBlobHexFromBytesToBytes and then we would read raw data - or not.
…\x%s' using instead the plain blobLiteral, which expects an hexadecimal encoded value with morf blobLiteral(value).
…stgreSQLMetaDataProvider, to read the actual partitioned tables as regular ones. Those tables are represented in a distinct form when Postgres.
Changed comments on PostgreSQLDialect
…l binary fields with byte values ranging from 0x00 to 0xFF.
Propagate the impact of that change to other classes.
TestDatabaseMetaDataProvider - testTableWithPartition(): extend test to validate that table and column metadata for partition table is still read.
Unit test for create with hash rule passes.
@BrunoSMonteiro74 BrunoSMonteiro74 changed the title Fix syntax for "PARTITION BY RANGE" clause Add support for table partitions with morf Sep 18, 2025
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
28.8% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

…for Upgrade step that has a PRF index matching in columns but not in uniqueness. Oracle throws an error.
…sn't being closed.

Fix bugs when reading partitions of tables.
Complete code to create range and hash partitions on PostgreSQLDialect when read from cryo SPs.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
28.6% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants