apex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Isha Arkatkar <i...@datatorrent.com>
Subject Re: Why are parser and formatter operator hidden in Malhar/contrib/schema
Date Thu, 17 Dec 2015 00:09:58 GMT
Hi Chandni,

   I have moved that as well to lib, as the parsers depended on that.

Thanks,
Isha

On Wed, Dec 16, 2015 at 1:01 PM, Chandni Singh <chandni@datatorrent.com>
wrote:

> There is a converter package under com.datatorrent.contrib which has a
> Converter API. This belongs in library as well.
>
> Thanks,
> Chandni
>
> On Tue, Dec 15, 2015 at 1:29 PM, Chandni Singh <chandni@datatorrent.com>
> wrote:
>
> > Isha,
> >
> > Thanks for moving this. When you move these files, please place then
> under
> > a package which reflects its functionality. I don't see the need for
> > package called schema.
> >
> > Thanks,
> > Chandni
> >
> > On Tue, Dec 15, 2015 at 12:31 PM, Isha Arkatkar <isha@datatorrent.com>
> > wrote:
> >
> >> Hi,
> >>
> >>   For csv parser there is an additional dependency. So, I'll move only
> >> json
> >> and xml to new location.
> >>
> >> Thanks,
> >> Isha
> >>
> >> On Tue, Dec 15, 2015 at 11:42 AM, Thomas Weise <thomas@datatorrent.com>
> >> wrote:
> >>
> >> > As long as the operators don't introduce additional dependencies they
> >> > should be in lib.
> >> >
> >> >
> >> > On Tue, Dec 15, 2015 at 9:34 AM, Shubham Pathak <
> >> shubham@datatorrent.com>
> >> > wrote:
> >> >
> >> > > Hi Chandni,
> >> > >
> >> > > I had written those operators.
> >> > > Here is the jira for that
> >> https://malhar.atlassian.net/browse/MLHR-1838
> >> > > You would find the entire discussion there.
> >> > >
> >> > > Why are all these operator under Malhar/contrib and not Malhar/lib
> >> > > When i was writing the code i saw AbstractCsvParser in contriib and
> >> hence
> >> > > added there.
> >> > >
> >> > > Recently i got to know which operators must go in contrib and what
> >> must
> >> > go
> >> > > in lib.
> >> > > By that definition, these operators must belong to lib.
> >> > >
> >> > > Thanks,
> >> > > Shubham
> >> > >
> >> > >
> >> > > On Tue, Dec 15, 2015 at 1:32 PM, Chandni Singh <
> >> chandni@datatorrent.com>
> >> > > wrote:
> >> > >
> >> > > > Hi,
> >> > > >
> >> > > > I just came across couple of formatter and parser operators which
> >> are
> >> > > under
> >> > > > Malhar/contrib/schema.
> >> > > >
> >> > > > I have couple of questions:
> >> > > > 1. What does schema denote here?
> >> > > > 2. Why formatter/parser which are functions are placed under
> schema
> >> > > > package?
> >> > > > 2. Why are all these operator under Malhar/contrib and not
> >> Malhar/lib
> >> > > >
> >> > > > Thanks,
> >> > > > Chandni
> >> > > >
> >> > >
> >> >
> >>
> >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message