namrathamyske opened a new pull request, #8042:
URL: https://github.com/apache/iceberg/pull/8042

   For spark 3.4, https://github.com/apache/iceberg/pull/7637, 
https://github.com/apache/iceberg/pull/7520 have been added for enabling AQE to 
solve for small files problem and taking care of skews. 
   
   For spark 3.4, `RequiresDistributionAndOrdering` has 
`distributionStrictlyRequired()` which has been to false by iceberg 
https://github.com/apache/iceberg/blame/37f53518a09803e4ef6b4669f58fbcc960ea5994/spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/source/SparkWrite.java#L139.
 
   
   To enable same in 3.3 without presence of `distributionStrictlyRequired()` 
in `RequiresDistributionAndOrdering`, we can add a table level property which 
can be checked which determining Repartition vs Rebalance operator. Idea taken 
from 
https://github.com/apache/spark/blob/37aa62f629e652ed70505620473530cd9611018e/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DistributionAndOrderingUtils.scala#L66.
   
   Extending this PR on https://github.com/apache/iceberg/pull/7932. 
@bowenliang123 
   
   @RussellSpitzer @rdblue @aokolnychyi  could you take a look at this PR?
   
   


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