Bradley Dice

Results 480 comments of 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!

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

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