Skip to content

Fix InitializeCompute flow#493

Merged
alexcos20 merged 9 commits intomainfrom
bug/debug_provider_reuseOrder
Jun 23, 2022
Merged

Fix InitializeCompute flow#493
alexcos20 merged 9 commits intomainfrom
bug/debug_provider_reuseOrder

Conversation

@alexcos20
Copy link
Copy Markdown
Member

@alexcos20 alexcos20 commented Jun 22, 2022

Changes proposed in this PR:

  • make sure we always return correct startOrder tx_id, even if caller sends a reuseOrder tx
  • add extra debug logging

Copy link
Copy Markdown
Contributor

@calina-c calina-c left a comment

Choose a reason for hiding this comment

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

Should we keep all the logs?

@alexcos20
Copy link
Copy Markdown
Member Author

Should we keep all the logs?

I would keep them for a while, until C2d is stable

@alexcos20 alexcos20 marked this pull request as ready for review June 23, 2022 03:39
@alexcos20 alexcos20 changed the title add some logs Fix InitializeCompute flow Jun 23, 2022
@alexcos20 alexcos20 requested a review from calina-c June 23, 2022 06:17
@alexcos20 alexcos20 merged commit 5f078f0 into main Jun 23, 2022
@alexcos20 alexcos20 deleted the bug/debug_provider_reuseOrder branch June 23, 2022 07:29
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.

2 participants