Skip to content

Conversation

ptiurin
Copy link
Contributor

@ptiurin ptiurin commented May 8, 2025

Initially proposed in #430

Exposing get_async_query_info method that allows users to easily access additional information about their async query: status, error message, query id, etc.

This includes refactoring to future-proof this method as multiple queries might correspond to a single async token.

@ptiurin ptiurin marked this pull request as ready for review May 8, 2025 17:01
@ptiurin ptiurin requested a review from a team as a code owner May 8, 2025 17:01
Copy link

@goprean goprean left a comment

Choose a reason for hiding this comment

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

lgtm.
Can you add a comment on the PR that was issued with just exposing the get_async_query_data so the developer knows what we are fixing his PR but in a different way.

Copy link

@ptiurin ptiurin merged commit 1eef56a into main May 12, 2025
8 checks passed
@ptiurin ptiurin deleted the feat-async-query-info branch May 12, 2025 11:08
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