Szehon Ho
Szehon Ho
Makes sense, i think adding a count sounds fine to me.
Hi @dongjoon-hyun sorry for the back and forth here. As in the description, @aokolnychyi explained he preferred to make the behavior choose UPDATE SET * to refer to nested fields,...
Btw, The code is not new code, its the same code in https://github.com/apache/spark/pull/53149 which was removed, it is just brought back.
Hi, @dongjoon-hyun , @aokolnychyi mentioned it would be good to get into 4.1, because we are still releasing the feature of 'struct coercion' , albeit with a flag. So he...
> Does this change anything when MERGE_INTO_NESTED_TYPE_COERCION_ENABLED is false? Yea, it should not, that should be the guard for the whole feature (nested type coercion)
hi, @nk1506 thanks for ping, ill look at this, but just FYI, when I look a few times back this patch seems a bit far, it adds things like ViewAwareTableBuilder...
yes it is for 4.2, geo type seems disabled in 4.1
@mkaravel @uros-db can you help review?
😄 , yea it sounds like it will be the Spark side. If so, I think @aokolnychyi did mention to me. He is off and ill be off as well...
hm not opposed to change, but curious, would it be better to also print the projection? Do you have a particular projection in mind you want toString for?