zeroshade commented on code in PR #13768:
URL: https://github.com/apache/arrow/pull/13768#discussion_r936711343


##########
go/arrow/datatype_nested.go:
##########
@@ -329,6 +333,208 @@ func (t *MapType) Layout() DataTypeLayout {
        return t.value.Layout()
 }
 
+type (
+       UnionTypeCode = int8
+       UnionMode     int8
+)
+
+const (
+       MaxUnionTypeCode    UnionTypeCode = 127
+       InvalidUnionChildID int           = -1
+
+       SparseMode UnionMode = iota
+       DenseMode
+)
+
+type UnionType interface {
+       NestedType
+       Mode() UnionMode
+       ChildIDs() []int
+       TypeCodes() []UnionTypeCode

Review Comment:
   Thanks, I intended to do so and completely forgot as I got wrapped up in 
making the IPC stuff work for Unions (the next change after this). I'll go back 
and get these all documented



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to