[SPARK-34343][SQL][TESTS] Add missing test for some non-array types in PostgreSQL#31456
[SPARK-34343][SQL][TESTS] Add missing test for some non-array types in PostgreSQL#31456sarutak wants to merge 3 commits intoapache:masterfrom
Conversation
|
Kubernetes integration test starting |
|
Kubernetes integration test status success |
|
Test build #134835 has finished for PR 31456 at commit
|
|
SPARK-34357 (#31473) changed the type-mapping of JDBC |
|
Kubernetes integration test starting |
|
Kubernetes integration test status failure |
|
Test build #134887 has finished for PR 31456 at commit
|
maropu
left a comment
maropu
left a comment
There was a problem hiding this comment.
okay, I've checked that PostgresIntegrationSuite passes.
|
Thanks! Merged to master. |
…n PostgreSQL ### What changes were proposed in this pull request? This PR added tests for some non-array types in PostgreSQL. PostgreSQL supports wide range of types (https://www.postgresql.org/docs/13/datatype.html) and `PostgresIntegrationSuite` contains tests for some types but ones for the following types are missing. * bit varying * point * line * lseg * box * path * polygon * circle * pg_lsn * macaddr * macaddr8 * numeric * pg_snapshot * real * time * timestamp * tsquery * tsvector * txid_snapshot * xml NOTE: Handling money types can be buggy so this PR doesn't add tests for those types. ### Why are the changes needed? To ensure those types work with Spark well. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Extended `PostgresIntegrationSuite`. Closes #31456 from sarutak/test-for-some-types-postgresql. Authored-by: Kousuke Saruta <sarutak@oss.nttdata.com> Signed-off-by: Takeshi Yamamuro <yamamuro@apache.org>
4 participants
What changes were proposed in this pull request?
This PR added tests for some non-array types in PostgreSQL.
PostgreSQL supports wide range of types (https://www.postgresql.org/docs/13/datatype.html) and
PostgresIntegrationSuitecontains tests for some types but ones for the following types are missing.NOTE: Handling money types can be buggy so this PR doesn't add tests for those types.
Why are the changes needed?
To ensure those types work with Spark well.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Extended
PostgresIntegrationSuite.