@Liang: yes, your understanding of my proposal is correct. Why remove empty sort_columns? if user specifies empty sort columns, I should throw an exception saying sort_columns specified not present? I feel no need to remove empty sort columns, by default we set sort_columns as empty sort_columns internally.
@xuchuanyin: yes, that's all. But I also want to change CarbonCommonConstants.LOAD_SORT_SCOPE_DEFAULT, because if some place if sort_scope is displayed or addressed without referring sort_columns. I want to show default as NO_SORT @david: I will check about this use case and development scope of this version. If required, I will do it in a separate PR. Thanks, Ajantha On Mon, Dec 17, 2018 at 7:17 AM David CaiQiang <david.c...@gmail.com> wrote: > Better to support alter 'sort_columns' and 'sort_scope' also. > > After the table creation and data loading, the user can adjust > 'sort_columns' and 'sort_scope'. > > > > > > > ----- > Best Regards > David Cai > -- > Sent from: > http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/ >
