Skip to content

Conversation

@no23reason
Copy link
Contributor

Users can now provide a callback that will be called with the Execution object as soon as it is available. This is useful for getting some of the information available in the Execution object before the actual data is loaded. The Execution object can also be used to cancel the execution result operation.

Also unify Execution types in data_access.py: no need to use the ExecutionResponse alias anymore.

JIRA: CQ-1387
risk: low

Users can now provide a callback that will be called with the Execution
object as soon as it is available. This is useful for getting some of
the information available in the Execution object before the actual
data is loaded. The Execution object can also be used to cancel the
execution result operation.

JIRA: CQ-1387
risk: low
The ExecutionResponse is just an alias so unify the types in the file
to keep things simpler.

JIRA: CQ-1387
risk: low
@no23reason no23reason marked this pull request as ready for review June 12, 2025 11:38
Martozar
Martozar previously approved these changes Jun 12, 2025
lupko
lupko previously approved these changes Jun 12, 2025
Copy link
Contributor

@lupko lupko left a comment

Choose a reason for hiding this comment

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

LGTM

Now that the type is Execution, the corresponding parameters should
be called the same.

JIRA: CQ-1387
risk: low
@no23reason no23reason dismissed stale reviews from lupko and Martozar via a5eef85 June 12, 2025 12:37
@hkad98 hkad98 enabled auto-merge June 12, 2025 12:39
@hkad98 hkad98 merged commit e49ec29 into gooddata:master Jun 12, 2025
9 checks passed
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.

4 participants