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

feat: support optimistic concurrency control headers #3462

Draft
wants to merge 18 commits into
base: master
Choose a base branch
from
Draft

feat: support optimistic concurrency control headers #3462

wants to merge 18 commits into from

Conversation

Torres-ssf
Copy link
Contributor

@Torres-ssf Torres-ssf commented Dec 9, 2024 •
edited by fuel-service-user
Loading

Release notes

In this release, we:

  • TBD

Summary

flowchart TD
    %% Initialization
    A[Initialize LATEST_BLOCK_HEIGHT = 0] --> B[Set request header required block to be LATEST_BLOCK_HEIGHT]
    
    %% Send Request
    B --> C[Send Request]
    
    %% Response Handling
    C -->|Response: 200 OK| D[Extract CURRENT_BLOCK_HEIGHT from Response]
    C -->|Response: 412 Precondition Failed| E[Increment Retry Count]
    
    %% Update Latest Block Height
    D --> F[Is CURRENT_BLOCK_HEIGHT > LATEST_BLOCK_HEIGHT?]
    F -->|Yes| G[Update LATEST_BLOCK_HEIGHT]
    F -->|No| H[No Change]
    G --> I[Proceed with next operation/request]
    H --> I[Proceed with next operation/request]
    
    %% Handling 412 - Precondition Failed
    E -->|Retry Count < Max Retries| J(Wait and Retry)
    J --> C
    E -->|Retry Count >= Max Retries| K[Abort]
    
    %% End Conditions
    K --> O[End]
    I --> O[End]
Loading

Checklist

  • All changes are covered by tests (or not applicable)
  • All changes are documented (or not applicable)
  • I reviewed the entire PR myself (preferably, on GH UI)
  • I described all Breaking Changes (or there's none)

@vercel Vercel
Copy link

vercel bot commented Dec 9, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fuels-template ❌ Failed (Inspect) Dec 13, 2024 11:38pm
ts-docs ❌ Failed (Inspect) Dec 13, 2024 11:38pm
ts-docs-api ❌ Failed (Inspect) Dec 13, 2024 11:38pm

@Torres-ssf Torres-ssf self-assigned this Dec 9, 2024
@Torres-ssf Torres-ssf added the feat Issue is a feature label Dec 9, 2024
@arboleya arboleya mentioned this pull request Dec 10, 2024
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@digorithm digorithm Awaiting requested review from digorithm digorithm will be requested when the pull request is marked ready for review digorithm is a code owner

@arboleya arboleya Awaiting requested review from arboleya arboleya will be requested when the pull request is marked ready for review arboleya is a code owner

@Dhaiwat10 Dhaiwat10 Awaiting requested review from Dhaiwat10 Dhaiwat10 will be requested when the pull request is marked ready for review Dhaiwat10 is a code owner

@danielbate danielbate Awaiting requested review from danielbate danielbate will be requested when the pull request is marked ready for review danielbate is a code owner

@nedsalk nedsalk Awaiting requested review from nedsalk nedsalk will be requested when the pull request is marked ready for review nedsalk is a code owner

@petertonysmith94 petertonysmith94 Awaiting requested review from petertonysmith94 petertonysmith94 will be requested when the pull request is marked ready for review petertonysmith94 is a code owner

@maschad maschad Awaiting requested review from maschad maschad will be requested when the pull request is marked ready for review maschad is a code owner

At least 3 approving reviews are required to merge this pull request.

Assignees

@Torres-ssf Torres-ssf

Labels
feat Issue is a feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Review and Add Special Headers for Optimistic Concurrency Control
1 participant
@Torres-ssf