You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@carbondata.apache.org by xm_zzc <44...@qq.com> on 2018/06/07 03:16:34 UTC

Use RowStreamParserImp as default value of config 'carbon.stream.parser'

Hi dev:
  Currently the default value of 'carbon.stream.parser' is
CSVStreamParserImp, it transforms InternalRow(0) to Array[Object],
InternalRow(0) represents a string value of one line. But generally user
will analyse data and save them as a Row(Int, String, String, Double...),
not a Row(String), I think parser 'RowStreamParserImpl' is used more often
for real scene. 
So do we use 'RowStreamParserImp' as default value? Welcome to give some
feedback.



--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/

Re: Use RowStreamParserImp as default value of config 'carbon.stream.parser'

Posted by Jacky Li <ja...@qq.com>.

+1.

I think this change is fine.

Regards,
Jacky

> 在 2018年6月8日，下午3:10，David CaiQiang <da...@gmail.com> 写道：
> 
> +1, I agree with using RowStreamParserImpl by default.
> 
> 
> -----
> Best Regards
> David Cai
> --
> Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/
>

Re: Use RowStreamParserImp as default value of config 'carbon.stream.parser'

Posted by David CaiQiang <da...@gmail.com>.

+1, I agree with using RowStreamParserImpl by default.



-----
Best Regards
David Cai
--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/