-
Notifications
You must be signed in to change notification settings - Fork 65
apollo_gateway: create large storage writer trait #11361
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
Conversation
noaov1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@noaov1 reviewed 3 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @einat-starkware).
noaov1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@noaov1 made 3 comments.
Reviewable status: all files reviewed, 3 unresolved discussions (waiting on @einat-starkware).
crates/apollo_gateway/src/large_storage_writer.rs line 13 at r1 (raw file):
pub trait LargeStorageWriterTrait: Send + Sync { async fn set_proof(&self, facts_hash: Felt, proof: Proof) -> Result<(), LargeStorageError>; }
Suggestion:
pub trait ProofArchiveWriterTrait: Send + Sync {
async fn set_proof(&self, facts_hash: Felt, proof: Proof) -> Result<(), LargeStorageError>;
}crates/apollo_gateway/src/large_storage_writer.rs line 16 at r1 (raw file):
#[derive(Debug, Error)] pub enum LargeStorageError {
Suggestion:
pub enum ProofArchiveError {crates/apollo_gateway/src/large_storage_writer.rs line 23 at r1 (raw file):
#[derive(Clone, Default)] // TODO(Einat): Add GCS related fields. pub struct LargeStorageWriter;
Suggestion:
pub struct GcsProofArchiveWriter;4587f49 to
6dd2a16
Compare
c422d30 to
ec258b1
Compare
e216f8c to
740cc86
Compare
3134825 to
78b4f58
Compare
207e94e to
eb2cb58
Compare
78b4f58 to
7f3a6cb
Compare
7f3a6cb to
75fb099
Compare
eb2cb58 to
6869945
Compare
75fb099 to
d52fc85
Compare
6869945 to
878fadd
Compare
noaov1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@noaov1 partially reviewed 2 files and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @avivg-starkware).
878fadd to
7e76d0d
Compare
d52fc85 to
7255e2b
Compare

No description provided.