Skip to content

fix(pyamber, v1.2): Invalidate cache while loading executor_def. - #7174

Open
github-actions[bot] wants to merge 1 commit into
release/v1.2from
backport/7173-invalidate-cache-while-loading-executor-v1.2
Open

fix(pyamber, v1.2): Invalidate cache while loading executor_def.#7174
github-actions[bot] wants to merge 1 commit into
release/v1.2from
backport/7173-invalidate-cache-while-loading-executor-v1.2

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

What changes were proposed in this PR?

Automated backport of #7173 to release/v1.2.

Source: 24a20c8 · automation run

Any related issues, documentation, discussions?

Backport of #7173. Originally linked #7126.

How was this PR tested?

Release-branch CI runs on this branch once the build is fixed and this PR is marked ready for review.

Was this PR authored or co-authored using generative AI tooling?

No.

### What changes were proposed in this PR?
This PR calls `importlib.invalidate_caches()` prior to importing newly
written UDF modules in `ExecutorManager.load_executor_definition`.

### Any related issues, documentation, discussions?
Fixes #7126

### How was this PR tested?
Tested with existing Python unit tests:
*
`amber/src/test/python/core/architecture/managers/test_executor_manager.py::TestUpdateExecutor::test_repeated_updates_keep_carrying_the_running_state`

Verified that rapid sequential executor updates cleanly load without
throwing `ModuleNotFoundError`.

### Was this PR authored or co-authored using generative AI tooling?
No.

(backported from commit 24a20c8)
@github-actions

Copy link
Copy Markdown
Contributor Author

The cherry-pick applied cleanly but the backported tree failed its pre-merge build. Fix the build on this branch, then mark this PR ready for review.

@github-actions
github-actions Bot requested a review from xuang7 July 31, 2026 18:53
@Yicong-Huang
Yicong-Huang marked this pull request as ready for review July 31, 2026 19:20
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