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

the implementation of query session #126

Closed
8 tasks done
jingchen2222 opened this issue Nov 6, 2022 · 4 comments
Closed
8 tasks done

the implementation of query session #126

jingchen2222 opened this issue Nov 6, 2022 · 4 comments

Comments

@jingchen2222
Copy link
Collaborator

jingchen2222 commented Nov 6, 2022

Overview

db3-query

Details

image

Status

Alternatives

Additional context

the concept of query session

@jingchen2222 jingchen2222 self-assigned this Nov 6, 2022
@jingchen2222 jingchen2222 added this to the M1 milestone Nov 6, 2022
@jingchen2222 jingchen2222 changed the title Apply query session Apply query session bill Nov 6, 2022
@imotai
Copy link
Contributor

imotai commented Nov 10, 2022

the full flow of query sesson from the SDK to NODE and MemPool
the_query_session

  1. Every request from the SDK must be signed
  2. Every response from the node must be signed
  3. The query session from the node must be signed by both the SDK and the node

@imotai imotai linked a pull request Nov 10, 2022 that will close this issue
4 tasks
@jingchen2222 jingchen2222 reopened this Nov 13, 2022
@jingchen2222
Copy link
Collaborator Author

jingchen2222 commented Nov 13, 2022

@jingchen2222
Copy link
Collaborator Author

image

@imotai imotai linked a pull request Nov 16, 2022 that will close this issue
@imotai imotai changed the title Apply query session bill the implementation of query session Nov 16, 2022
@imotai
Copy link
Contributor

imotai commented Nov 16, 2022

image

5.4 Hash is the ID of the query session, no txid

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants