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

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

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

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/
Reply | Threaded
Open this post in threaded view
|

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

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/
Best Regards
David Cai
Reply | Threaded
Open this post in threaded view
|

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

Jacky Li
+1.

I think this change is fine.

Regards,
Jacky

> 在 2018年6月8日,下午3:10,David CaiQiang <[hidden email]> 写道:
>
> +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/
>