Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fixed unnecessary column type changes in H2DbWireRecordStoreImpl #5157

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

nicolatimeus
Copy link
Contributor

Note: We are using the Conventional Commits convention for our pull request titles. Please take a look at the PR title format document for the supported types and scopes.

Brief description of the PR. [e.g. Added null check on object to avoid NullPointerException]

Fixes issue that causes H2DbWireRecordStore to incorrectly change the column type and drop existing values when a new TypedValue of one of the following types is inserted:

  • FLOAT
  • DOUBLE
  • STRING
  • BYTE_ARRAY

Copy link
Contributor

@marcellorinaldo marcellorinaldo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested as working. Retested also the deprecated wires implementations.

@MMaiero MMaiero merged commit 9d11914 into eclipse-kura:develop Feb 29, 2024
4 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 29, 2024
…#5157)

fix: Fixed unnecessary column type changes in H2DbWireRecordStore

Signed-off-by: Nicola Timeus <[email protected]>
(cherry picked from commit 9d11914)
MMaiero added a commit that referenced this pull request Feb 29, 2024
… [backport release-5.4.0] (#5158)

* fix: Fixed unnecessary column type changes in H2DbWireRecordStoreImpl (#5157)

fix: Fixed unnecessary column type changes in H2DbWireRecordStore

Signed-off-by: Nicola Timeus <[email protected]>
(cherry picked from commit 9d11914)

* ore: Updated distrib reference to new core bundle

Signed-off-by: MMaiero <[email protected]>

---------

Signed-off-by: MMaiero <[email protected]>
Co-authored-by: nicolatimeus <[email protected]>
Co-authored-by: MMaiero <[email protected]>
@nicolatimeus nicolatimeus deleted the fix_h2-column-change branch March 22, 2024 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants