Posted by
Ajantha Bhat on
Dec 17, 2018; 8:58am
URL: http://apache-carbondata-dev-mailing-list-archive.168.s1.nabble.com/Discussion-Make-no-sort-as-default-sort-scope-and-keep-sort-columns-as-empty-by-default-tp70233p70744.html
@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 <
[hidden email]> wrote: