Skip to content

fix: correct namespaceId jdbcType bindings(#6718) - #7003

Open
juicewcode wants to merge 1 commit into
apache:masterfrom
juicewcode:fix/6718-namespace-id-jdbctype-varchar
Open

fix: correct namespaceId jdbcType bindings(#6718)#7003
juicewcode wants to merge 1 commit into
apache:masterfrom
juicewcode:fix/6718-namespace-id-jdbctype-varchar

Conversation

@juicewcode

Copy link
Copy Markdown
Contributor

Fixes #6718

Summary

  • Changed the namespaceId JDBC type in the meta_data insert statement from TINYINT to VARCHAR
  • Changed the namespaceId JDBC type in the namespace_plugin_rel query condition from TINYINT to VARCHAR
  • Added mapper tests for string namespace IDs

Test

  • Tested inserting and reading back a non-numeric string namespace ID in meta_data
  • Tested filtering namespace_plugin_rel records by a string namespace ID

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

  - Change meta_data insert namespaceId binding from TINYINT to VARCHAR
  - Change namespace_plugin_rel query namespaceId binding from TINYINT to VARCHAR
  - Add mapper tests for string namespace IDs
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.

[BUG] meta_data insert binds namespaceId with wrong jdbcType (TINYINT for a VARCHAR column)

1 participant