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

DAML-LF: split com.daml.lf.types.Ledger #6264

Merged
merged 2 commits into from
Jun 8, 2020

Conversation

remyhaemmerle-da
Copy link
Collaborator

@remyhaemmerle-da remyhaemmerle-da commented Jun 8, 2020

We split the object com.daml.lf.types.ledger.Ledger in three:

  • one part in com.daml.lf.ledger. (in //daml-lf/transaction) for the code relative to EventId shared between scenario service and sandbox
  • one part in com.daml.lf.ledger. (in //daml-lf/interpreter) for the code relative to EnrichedTransaction shared between scenario service and blinding logic
  • one part in com.daml.lf.scenario. (in //daml-lf/interpreter) for the code part specific to scenario service

fixes #6260

CHANGELOG_BEGIN
CHANGELOG_END

Pull Request Checklist

  • Read and understand the contribution guidelines
  • Include appropriate tests
  • Set a descriptive title and thorough description
  • Add a reference to the issue this PR will solve, if appropriate
  • Include changelog additions in one or more commit message bodies between the CHANGELOG_BEGIN and CHANGELOG_END tags
  • Normal production system change, include purpose of change in description

NOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with /AzurePipelines run to
trigger the build.

We split the object  com.daml.lf.types.ledger in three:
- one part in `com.daml.lf.ledger.` (in //daml-lf/transaction) for the part relative to EventId (shares between scenario service and sandbox)
- one part in `com.daml.lf.ledger.` (in //daml-lf/interpreter) for the part common to Blinding and Scenario
- one part in `com.daml.lf.scenario.` (in //daml-lf/interpreter) for the part specific to Scenario

fixes #6260

CHANGELOG_BEGIN
CHANGELOG_END
@remyhaemmerle-da remyhaemmerle-da added the component/daml-engine DAML-LF Engine & Interpreter label Jun 8, 2020
@remyhaemmerle-da remyhaemmerle-da added this to the Maintenance milestone Jun 8, 2020
@remyhaemmerle-da remyhaemmerle-da changed the title DAML-LF: split com.daml.lf.types.Ledger DAML-LF: split com.daml.lf.types.Ledger Jun 8, 2020
@remyhaemmerle-da remyhaemmerle-da merged commit 00972f3 into master Jun 8, 2020
@remyhaemmerle-da remyhaemmerle-da deleted the remy-drop-scenarion-node-id branch June 8, 2020 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/daml-engine DAML-LF Engine & Interpreter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove dependency of Sandbox to //daml-lf/interpreter
2 participants