-
Notifications
You must be signed in to change notification settings - Fork 207
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
PubSub: Synchronous pull raises 504 after a few seconds if no messages available #9
Comments
For the reference, I tried running the script both with PubSub version |
@plamut I have tried running the script with many versions and master branch it only passes with |
@HemangChothani Thanks for the research! (cc: @hongalex ). Will add this to the agenda of the next PubSub meeting. |
Hello, I have been running into an issue on my project when using this pub sub library. I am seeing the same error I am using: The only difference with me is that I got it after calling: I would appreciate some help, I am not sure if my issue is related to this one, but the error messages do match up which led me to believe this might be the cause. |
Ack. Will be reverting |
@cesarmarroquin Listing topics is a different method on the backend, and it's likely not related to the synchronous pull. The Does the error occur often, or just every now and then when using the list topics method? Does your application otherwise work, generally speaking? Any additional info would be appreciated, thanks! |
This should have been fixed here by reverting the timeout to the previous value, closing the issue. |
Forked from googleapis/google-cloud-python#7709 (comment).
Environment details
Tested with the following:
Steps to reproduce
Actual result:
After a few seconds a 504 DeadlineExceeded exception is raised.
Expected result:
Assuming that the network functions normally, there should be no error, and the
subscriber.pull()
should simply return an empty list of messages.Mind that there is no error if the
return_immediately=True
flag is used, but the latter is set to be deprecated and should thus not be used.Code sample
The following script might be useful:
reproduce.py
The text was updated successfully, but these errors were encountered: