zhengruifeng opened a new pull request, #58162:
URL: https://github.com/apache/spark/pull/58162

   ### What changes were proposed in this pull request?
   
   This PR retries the `python_worker_logs()` assertion in
   `test_udf_with_pyspark_logger`.
   
   The UDF result assertion still runs once. Only the follow-up read from the 
Python
   worker log TVF is retried, using the existing `eventually` helper.
   
   ### Why are the changes needed?
   
   The log assertion can be flaky when a Python worker log row is not visible at
   the moment the test reads `python_worker_logs()`. Retrying the log-table
   assertion keeps the expected rows unchanged while tolerating delayed log
   visibility.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Not run.
   
   Checked locally:
   
   ```
   git diff --check
   awk 'length>100 && $0 !~ /^[[:space:]]*(import|package) / && $0 !~ 
/https?:\/\// {print FILENAME":"FNR": "length" chars"}' 
python/pyspark/sql/tests/test_udf.py
   ```
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Codex (GPT-5)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to