nielspardon commented on issue #12597:
URL: https://github.com/apache/gluten/issues/12597#issuecomment-5126993781

   **Update — `Type.Nothing` direction changed (Increment 2)**
   
   The Increment 2 checklist item previously read "drop `Type.Nothing`". That's 
been corrected to **relocate** (kind `33`→`40`, kept).
   
   Rationale: in the Substrait community sync on 2026-07-29 the PMCs decided 
upstream should add a dedicated core `null` type rather than have producers 
avoid a bottom type. That proposal is up as 
[substrait-io/substrait#1147](https://github.com/substrait-io/substrait/pull/1147)
 (`Type.Null` at `kind` field `40`). Increment 2 deliberately relocates 
Gluten's `Nothing` fork to field 40 to match, so once #1147 is merged and 
released upstream the fork converges by a **pure message rename** 
`Nothing`→`Null` — no field renumbering and no changes to the Velox/ClickHouse 
consumer mappings. Gluten keeps `Type.Nothing` as-is until then.
   
   Net effect: Increment 2 is semantically neutral for `Nothing` (relocate 
only), and the former "drop" step becomes a future rename gated on the upstream 
`Type.Null` release.
   


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