namanjain24-sudo commented on PR #25527: URL: https://github.com/apache/datafusion/pull/25527#issuecomment-6422650541
One thing worth flagging before you re-run it. The switch removes the function declaration, but the `__grouping_id` values it returns are `UInt8`, and DataFusion writes unsigned literals with `typeVariationReference: 1` (`variation_const.rs`) without ever declaring the variation — #11544, the type-variation twin of #11545. That is not new here: on the previous head the export carried 1 declaration and 2 variation references, and it now carries 0 declarations and the same 2. Main's export of that query has neither, because it has no literal of that type to write. So the URN is gone, but if Isthmus resolves variation anchors as well it may trip on those next. Could you re-run the branch and see which it is? I would rather know now than find out after this merges. -- 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]
