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.

Sign up for GitHub

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

Jump to bottom

CORE-17431 utxo_transaction_metadata table and related migrations #1286

Merged
merged 5 commits into from
Oct 13, 2023
Merged

CORE-17431 utxo_transaction_metadata table and related migrations #1286

merged 5 commits into from
Oct 13, 2023

Conversation

vlajos
Copy link
Contributor

@vlajos vlajos commented Oct 10, 2023 •
edited
Loading

CORE-17431 Create utxo_transaction_metadata for storing metadata since it is relatively large but does not change frequently. Also, two new fields for accessing signed group parameters and CPI file checksum easier.

Tests:

  • Running the 5.1 migration script on top of a 5.0 combined worker with some vnodes and transactions. After running the migration, the new table and fields appeared with the expected content and the metadata components were deleted from the components table.
  • Running utxo smoke tests on the 5.1 combined worker executed the migrations on a clean database.

Runtime-os: corda/corda-runtime-os#4845

@vlajos vlajos force-pushed the vlajos/CORE-17431-utxo_transaction_metadata branch 5 times, most recently from 4b80d54 to e1124a7 Compare October 12, 2023 09:19
@corda-jenkins-ci02
Copy link
Contributor

corda-jenkins-ci02 bot commented Oct 12, 2023 •
edited
Loading

Jenkins build for PR 1286 build 10

Build Successful:
Jar artifact version produced by this PR: 5.1.0.33-alpha-1697124844578

@vlajos vlajos mentioned this pull request Oct 12, 2023
Merged
@vlajos vlajos marked this pull request as ready for review October 12, 2023 10:26
@vlajos vlajos requested a review from a team as a code owner October 12, 2023 10:26
@vlajos vlajos requested a review from a team October 12, 2023 10:26
@vlajos vlajos force-pushed the vlajos/CORE-17431-utxo_transaction_metadata branch from e1124a7 to 838c5c7 Compare October 12, 2023 14:13
@vlajos vlajos force-pushed the vlajos/CORE-17431-utxo_transaction_metadata branch from 6fb66f5 to 0f8baa4 Compare October 12, 2023 15:33
blsemo
blsemo approved these changes Oct 13, 2023
View reviewed changes
Copy link
Contributor

@blsemo blsemo left a comment

Choose a reason for hiding this comment

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

LGTM

@vlajos vlajos merged commit 086845e into release/os/5.1 Oct 13, 2023
4 checks passed
@vlajos vlajos deleted the vlajos/CORE-17431-utxo_transaction_metadata branch October 13, 2023 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blsemo blsemo blsemo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vlajos @blsemo