Dandandan commented on PR #2315: URL: https://github.com/apache/datafusion-ballista/pull/2315#issuecomment-5307471763
> thanks @Dandandan for addressing AQE issues, great to see that AQE work starts to pay off. > > If you're happy with AQE to be on by default i would not object, but lets wait for @andygrove and @avantgardnerio opinion I'll wait some days for @andygrove and @avantgardnerio I also found some easy ~17% improvement by enabling: * packing tasks by default (I believe @avantgardnerio added it, but it was disabled by default) * increase the broadcasting limit to 128MiB https://github.com/apache/datafusion-ballista/pull/2315/changes/8e8fd7779f66f888d45cd8f2f4273cdae869e8c4 -- 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]
