peter-toth opened a new pull request, #60206:
URL: https://github.com/apache/spark/pull/60206

   ### What changes were proposed in this pull request?
   
   This fixes two tests of #59916 that fail on master. Both check with DPP off 
that master still adds the bloom filter from a non-deterministic creation side. 
SPARK-60808 (#60176), merged before #59916, keeps that bloom filter out now, so 
the tests now expect none:
   - "SPARK-60702: do not add bloom filter if the creation side that the 
subquery reads is not deterministic";
   - "SPARK-60702: do not add bloom filter whose creation side is a 
non-deterministic pruned side".
   
   ### Why are the changes needed?
   
   `InjectRuntimeFilterSuite` fails on master. #59916 was tested on a base 
without SPARK-60808.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. Test only.
   
   ### How was this patch tested?
   
   `InjectRuntimeFilterSuite` passes, 62 of 62. On master `da165c1c658` the two 
tests fail with `List() had size 0 instead of expected size 1`.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code with Claude Opus 5.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