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

Blocking behavior of pymgclient #59

Open
cloudcell opened this issue Sep 4, 2023 · 5 comments
Open

Blocking behavior of pymgclient #59

cloudcell opened this issue Sep 4, 2023 · 5 comments
Labels
bug bug community community Effort - Medium Effort - Medium Frequency - Monthly Frequency - Monthly Priority - Later Priority - Later Reach - Some Reach - Some Severity - S2 Severity - S2

Comments

@cloudcell
Copy link

It appears that pymgclient's behavior is different from neo4j's library in a way that pymgclient blocks all operations of an app until the cypher query is completely performed. Would it be possible to use async withingn pymgclient (if neo4j uses async, as we suspect)?

@antejavor
Copy link

@cloudcell
Copy link
Author

cloudcell commented Sep 4, 2023

Here is some more context. To replicate the issue please create a simple flask app that would run a query and create an endpoint that responds to another query, say, fetching a file from a server. So, the issue appears that when one runs
gunicorn -w 1 -b 127.0.0.1:5432 script:app -- the app does not work properly during query processing
but when one runs
gunicorn -w 2 -b 127.0.0.1:5432 script:app -- the app does run properly (i.e. can run the cypher query and fetch a file from a server simultaneously)

This behavior does not appear when using neo4j, but only when using pymgclient.
What this means is that when one is using pymgclient, there needs to be a second worker to process other requests to the server as pymgclient appears to block the worker.

@katarinasupe katarinasupe added bug bug Severity - S2 Severity - S2 Effort - Unknown Effort - Unknown Importance - I2 Importance - I2 community community labels Dec 29, 2023
@katarinasupe
Copy link

Hi @cloudcell, so sorry we didn't attend to this sooner. Can you tell me if this is a blocking issue for you and how this affects your project currently?

@cloudcell
Copy link
Author

cloudcell commented Dec 29, 2023 via email

@katarinasupe
Copy link

Hi @cloudcell, thank you for the response. I'm glad to hear you found a workaround. If you have another issue, please open it here on GitHub or hop on a call with us to discuss more (maybe we can help).

@DavIvek DavIvek added Effort - Medium Effort - Medium and removed Effort - Unknown Effort - Unknown labels Feb 23, 2024
@hal-eisen-MG hal-eisen-MG added the Priority - Next Priority - Next label Feb 25, 2024
@katarinasupe katarinasupe added Frequency - Monthly Frequency - Monthly Reach - Some Reach - Some and removed Importance - I2 Importance - I2 labels Feb 27, 2024
@hal-eisen-MG hal-eisen-MG added Priority - Later Priority - Later and removed Priority - Next Priority - Next labels Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bug community community Effort - Medium Effort - Medium Frequency - Monthly Frequency - Monthly Priority - Later Priority - Later Reach - Some Reach - Some Severity - S2 Severity - S2
Projects
Development

No branches or pull requests

5 participants