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

Assemble CARs aws lambda function #20

Closed
vasco-santos opened this issue Mar 3, 2022 · 0 comments · May be fixed by nftstorage/assemble-cars-lambda#2
Closed

Assemble CARs aws lambda function #20

vasco-santos opened this issue Mar 3, 2022 · 0 comments · May be fixed by nftstorage/assemble-cars-lambda#2
Assignees
Labels
kind/enhancement A net-new feature or improvement to an existing feature

Comments

@vasco-santos
Copy link
Contributor

vasco-santos commented Mar 3, 2022

We currently write to S3 all CARs we receive under a /raw namespace. Some of them are already complete, while others are partial CARs with a partial DAG of a given upload.

Work triggered in the context of #19 , but also valuable for other directions.

Flow

Lambda function triggers on S3 write assemble-cars-lambda

  • verify if /raw namespace trigger
  • perform CAR completeness optimistic validations
    • is CAR file complete? we can rely on S3 metadata
  • if CAR is complete: write to /complete namespace
  • else: does CAR file have a DagPB root with know acceptable size?
    • get list of CARs in same directory and verify if their total size is bigger than known size
    • Get all CARs in the S3 directory
    • Validate CAR is complete through all the links
    • Join CARs and write to /complete namespace

Metrics

TBD

  • Successful warm cache requests
  • Length distribution of warm cache content

Notes

  • We need to look into limitations of AWS Lambda memory for "acceptable" size config
@vasco-santos vasco-santos added the kind/enhancement A net-new feature or improvement to an existing feature label Mar 3, 2022
@vasco-santos vasco-santos self-assigned this Mar 3, 2022
@vasco-santos vasco-santos transferred this issue from nftstorage/nft.storage Apr 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A net-new feature or improvement to an existing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants