+1 to have it as a separate class. On Tue, Mar 22, 2016 at 11:35 AM, Chaitanya Chebolu < [email protected]> wrote:
> +1. Move it out from AbstractFileInputOperator. > > On Tue, Mar 22, 2016 at 11:31 AM, Pradeep A. Dalvi <[email protected]> > wrote: > > > +1 Should be moved outside. > > > > On Tue, Mar 22, 2016 at 11:29 AM, Sandeep Deshmukh < > > [email protected]> > > wrote: > > > > > +1. Used in many cases. We should move it into separate class. > > > > > > Regards, > > > Sandeep > > > > > > On Tue, Mar 22, 2016 at 11:17 AM, Devendra Tagare < > > > [email protected] > > > > wrote: > > > > > > > +1.Using it was a bit of a search earlier.Moving it would defiantly > > ease > > > up > > > > usage. > > > > > > > > Thanks, > > > > Dev > > > > On Mar 21, 2016 10:46 PM, "Tushar Gosavi" <[email protected]> > > > wrote: > > > > > > > > > +1 on moving it out as a separate class. > > > > > > > > > > On Tue, Mar 22, 2016 at 8:21 AM, Sandesh Hegde < > > > [email protected]> > > > > > wrote: > > > > > > > > > > > +1, I was wondering why it was hidden. Used it during one of the > > > pocs. > > > > > > > > > > > > On Mon, Mar 21, 2016, 5:51 PM Pramod Immaneni < > > > [email protected]> > > > > > > wrote: > > > > > > > > > > > > > Yes +1. It is an often requested operator. > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > > > On Mon, Mar 21, 2016 at 5:49 PM, Chandni Singh < > > > > > [email protected]> > > > > > > > wrote: > > > > > > > > > > > > > > > Hi, > > > > > > > > > > > > > > > > The FileLineInputOperator is hidden under > > > > AbstractFileInputOperator. > > > > > > > > Shall we move it outside so that it is easily discovered. > > > > > > > > > > > > > > > > Thanks, > > > > > > > > Chandni > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
