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

Mssing step in open a database connection? #434

Open
stelar7 opened this issue Dec 13, 2024 · 1 comment
Open

Mssing step in open a database connection? #434

stelar7 opened this issue Dec 13, 2024 · 1 comment
Labels
Milestone

Comments

@stelar7
Copy link

stelar7 commented Dec 13, 2024

In open a database connection, there doesnt seem to be anything that sets the requests processed and done flags..

Step 10.6 sets it via upgrade a database, but in the case that we dont enter step 10, the request is left in the unprocessed state.

This will block some steps like deleting a database step 3, that says Wait until all previous requests in queue have been processed

Am I missing a spot where this would be set? or is it a spec issue?

@inexorabletash
Copy link
Member

The open() steps (5.2.1.3 and 5.2.2.2) set the done flag, but I think you're correct and the processed flag isn't set. So: spec issue, new in "v3"

I suspect this was introduced back in 671f315 with the processed flag itself.

PRs welcome!

@inexorabletash inexorabletash added this to the V3 milestone Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants