andygrove commented on issue #5091: URL: https://github.com/apache/datafusion-comet/issues/5091#issuecomment-5870681924
#5193 and #5367 replaced five of the seven loops here: decimal to boolean, the array_insert validity masks, pow, make_decimal and the columnar_to_row decimal reinterpretation. The other two were tried in #5367 and reverted because they regressed (to_naive_date_opt rebuilds the epoch per row in days_to_date, and the covariance filter allocates two arrays per batch). Closing this; the zip-based cleanup of the covariance null sync can go in its own PR if anyone wants it. -- 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]
