-
Notifications
You must be signed in to change notification settings - Fork 9
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
DA5-15 Recurring merge from main #1320
Conversation
…to jacob/ES-1398
* Change ACL mappings to add tokenSelection worker.
Add index on the version field to improve optimistic locking checks.
Update recently introduced index so it uses both the "key" and "version" columns as both are used by the State Manager during update and delete queries.
…ed columns and renaming (#1288) The created and is_ref_input columns are unnecessary to have in the utxo_transaction_sources table. The created column will be the same as in utxo_transaction_component table and it’s a 1:1 mapping. Whether a state is an input or a reference can be found out by using the groupIdx column rather than having an extra is_ref_input column. Also rename the ref_transaction_id and ref_leaf_idx columns as those are quite confusing.
Remove mentioning of attachments from the API - this has never been implemented and is unlikely to make an appearance soon, so having dead API calls is misleading.
Add `findUnconsumedStatesByExactType` to `UtxoLedgerService `, allowing developers to retrieve states by type from the vault in pages. This dissuades developers from retrieving every state from the vault in one go and returning them to a flow. Remove the non-paged version of the API.
ES-1485 PR and Branch housekeeping automation dry run
ES-1485 PR and Branch housekeeping automation (still dry-run)
Set permission for FlowMapper to send messages to Kafka topic "flow.mapper.session.in" Set permission for TokenSelection to send messages to Kafka topic "flow.session"
ES-1398 Pipeline integration with corda-api scanner PR gates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR title failed to match regex -> ^((CORDA|EG|ENT|INFRA|CORE|DOC|ES)-\d+)(.*)
Scanning for breaking API changes introduced by this PR Scan Failed: https://ci02.dev.r3.com/job/Corda5/job/corda-api-compatibility/job/PR-1320/2/ If the breaking changes are intentional, run |
Non-blocking downstream job failed for corda-non-functional-test https://ci02.dev.r3.com/job/Corda5/job/corda-api-compatibility/job/PR-1320/2/ has failed for PR 1320 build 2 Please investigate if your changes may have broken compilation on https://github.com/corda/corda-non-functional-test |
Non-blocking downstream job failed for corda-e2e-test https://ci02.dev.r3.com/job/Corda5/job/corda-api-compatibility/job/PR-1320/2/ has failed for PR 1320 build 2 Please investigate if your changes may have broken compilation on https://github.com/corda/corda-e2e-tests |
Jenkins build for PR 1320 build 3 Build Successful: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR title failed to match regex -> ^((CORDA|EG|ENT|INFRA|CORE|DOC|ES)-\d+)(.*)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for me for Jenkinsfiles and Gradle configuration files
Changes
release/os/5.1
Notes
.github/CODEOWNERS
to pre 5.1 freeze state as the interop feature branch is not frozen and C5 interop related changes will not be integrated into os/5.1.