Skip to content

Conversation

@wenjin272
Copy link
Collaborator

@wenjin272 wenjin272 commented Jan 15, 2026

Linked issue: #381

Purpose of change

Python built-in actions support async execution.

Tests

e2e

API

Documentation

  • doc-needed
  • doc-not-needed
  • doc-included

@github-actions github-actions bot added priority/major Default priority of the PR or issue. fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. labels Jan 15, 2026
@github-actions
Copy link

@wenjin272 Please select only one documentation label in your PR description.

@wenjin272
Copy link
Collaborator Author

wenjin272 commented Jan 15, 2026

This pr is based on the pr #400 of @Shekharrajak. Since the 0.2 version is planned code freeze on 1.19, I have to push for the PR to be finalized. I add @Shekharrajak as the co-author, thanks for your contribution.

There are three issues about async execution for built-in actions.

  1. Due to Pemja logging throws exception in new native python thread if it is not initlized in main thread alibaba/pemja#88, we should add a log in main thread.
  2. Due to the same root cause of 1, cross language resource can't be used in async execution.
  3. The token metric will cause the jvm crash, so there are still test case failed. But I think we should fix it in another pr.

@wenjin272 wenjin272 marked this pull request as draft January 16, 2026 06:03
@wenjin272 wenjin272 marked this pull request as ready for review January 16, 2026 08:14
@wenjin272
Copy link
Collaborator Author

@xintongsong, @Sxnan, Could you help review this pr. I also refactor the token usages metric report logic.

@xintongsong xintongsong merged commit 004cb46 into apache:main Jan 16, 2026
35 of 36 checks passed
@Shekharrajak
Copy link

It would have been better if the commits are cherry-picked. No worries!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-needed Your PR changes impact docs. fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants