gravitino icon indicating copy to clipboard operation
gravitino copied to clipboard

[Improvement] add validate check for ColumnPosition when updating column positions.

Open FANNG1 opened this issue 1 year ago • 3 comments

What would you like to be improved?

It seems ridiculous to update the column position to default. I think we should add a validate check when constructing UpdateColumnPosition

How should we improve?

No response

FANNG1 avatar Aug 07 '24 04:08 FANNG1

cc @mchades @caican00 , WDYT?

FANNG1 avatar Aug 07 '24 04:08 FANNG1

cc @mchades @caican00 , WDYT?

yeah, if we want to place a column to the end, it seems batter to use after. And default position is better used in AddColumn.

caican00 avatar Aug 07 '24 06:08 caican00

Hi @FANNG1 Please assign to me . Thanks.

LiuQhahah avatar Aug 08 '24 14:08 LiuQhahah