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

2018-06-08 Thread Jacky Li 
+1.

I think this change is fine.

Regards,
Jacky

> 在 2018年6月8日，下午3:10，David CaiQiang  写道：
> 
> +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'

2018-06-08 Thread David CaiQiang 
+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/

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

2018-06-06 Thread xm_zzc 
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/