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

Add Noop store #408

Merged
merged 1 commit into from Nov 19, 2023
Merged

Add Noop store #408

merged 1 commit into from Nov 19, 2023

Conversation

allada
Copy link
Collaborator

@allada allada commented Nov 18, 2023

Adds a Noop Store that sends data to the void and always returns 404 (NotFound).

closes #385


This change is Reviewable

Copy link
Collaborator Author

@allada allada left a comment

Choose a reason for hiding this comment

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

+@aaronmondal

Reviewable status: 0 of 9 files reviewed, all discussions resolved (waiting on @aaronmondal)

Copy link
Contributor

@aaronmondal aaronmondal left a comment

Choose a reason for hiding this comment

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

Reviewed 9 of 9 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @allada)


config/examples/basic_cas.json line 18 at r1 (raw file):

          "filesystem": {
            "content_path": "/tmp/turbo_cache/data-worker-test/content_path-cas",
            "temp_path": "/tmp/turbo_cache/data-worker-test/tmp_path-cas",

nit: native_link


config/examples/basic_cas.json line 70 at r1 (raw file):

        "ac_store": "AC_MAIN_STORE",
      },
      "work_directory": "/tmp/turbo_cache/work",

nit: native_link

@allada allada force-pushed the add-noop-store branch 2 times, most recently from 447bc11 to d09ec6e Compare November 18, 2023 22:49
Copy link
Collaborator Author

@allada allada left a comment

Choose a reason for hiding this comment

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

Reviewable status: 7 of 9 files reviewed, all discussions resolved (waiting on @aaronmondal)


config/examples/basic_cas.json line 18 at r1 (raw file):

Previously, aaronmondal (Aaron Siddhartha Mondal) wrote…

nit: native_link

Done.


config/examples/basic_cas.json line 70 at r1 (raw file):

Previously, aaronmondal (Aaron Siddhartha Mondal) wrote…

nit: native_link

Done.

Adds a Noop Store that sends data to the void and always returns
404 (NotFound).

closes #385
Copy link
Contributor

@aaronmondal aaronmondal left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r2, 4 of 4 files at r3, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @allada)

@allada allada merged commit aea3768 into main Nov 19, 2023
15 checks passed
@allada allada deleted the add-noop-store branch November 19, 2023 06:50
TripleKai pushed a commit to TripleKai/turbo-cache that referenced this pull request Nov 29, 2023
Adds a Noop Store that sends data to the void and always returns
404 (NotFound).

closes TraceMachina#385
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.

Implement Noop store
2 participants