Skip to content

Conversation

@rickyma
Copy link
Contributor

@rickyma rickyma commented Apr 15, 2024

What changes were proposed in this pull request?

Fix the issue of log variable printing.

Why are the changes needed?

A follow-up PR for: #1461.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Unnecessary.

@github-actions
Copy link

Test Results

 2 363 files  ±0   2 363 suites  ±0   4h 30m 35s ⏱️ +9s
   912 tests ±0     911 ✅ ±0   1 💤 ±0  0 ❌ ±0 
10 585 runs  ±0  10 571 ✅ ±0  14 💤 ±0  0 ❌ ±0 

Results for commit 965ea49. ± Comparison against base commit 6ea4300.

@zuston zuston merged commit 222f5d4 into apache:master Apr 16, 2024
@jerqi
Copy link
Contributor

jerqi commented Apr 30, 2024

@rickyma Could you raise a pull request for branch 0.9? There are some conflicts so I can't cherry-pick this to branch 0.9 directly.

@rickyma
Copy link
Contributor Author

rickyma commented May 5, 2024

@rickyma Could you raise a pull request for branch 0.9? There are some conflicts so I can't cherry-pick this to branch 0.9 directly.

I think we also need this #1643. I've put them together.
Done by #1672.

jerqi pushed a commit that referenced this pull request May 6, 2024
…#1672)

### What changes were proposed in this pull request?

Fix the issue of log variable printing.

### Why are the changes needed?

Cherry pick commits #1643 and #1648 from master to branch-0.9.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Unnecessary.
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.

3 participants