Skip to content

Conversation

@birschick-bq
Copy link
Contributor

Checks the return status when polling for response.

  • Must be in the FINISHED_STATE. Other states are invalid after polling.
  • Difficult to write tests for this without "abusing" the server to invoke a server error.

Copy link
Contributor

@CurtHagenlocher CurtHagenlocher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@CurtHagenlocher CurtHagenlocher merged commit 7d2b719 into apache:main Dec 7, 2024
8 checks passed
@birschick-bq birschick-bq deleted the dev/birschick-bq/fix-poll-error branch December 10, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants