Bradley Dice
Bradley Dice
~Great. I edited the task list to move out of the `experimental` namespace in 24.06.~ Apparently this was already done. Task removed.
I merged the upstream and resolved a merge conflict from #15619. I also pinged ops for a review, after which this should be ready to merge.
Thanks @NvTimLiu!
/merge
@miscco I went ahead and applied my two (small) suggestions, so we can get CI run and merge this sooner. These aren't controversial changes so we should be able to...
/ok to test
@wence- Is this the kind of change you had in mind? I looked over the list of files you put in #14527 and made changes for most of them. I'd...
@davidwendt I think the current implementation should ignore the offset children of strings columns, which will provide the behavior you want. @wence- I haven't gotten to explore the inductive tests...
@wence- I applied most of your suggested changes. Feel free to have a final look, if you'd like. I'm aiming to merge in the next 2 days to avoid conflicts...