apex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chandni Singh <chan...@datatorrent.com>
Subject Re: Why are parser and formatter operator hidden in Malhar/contrib/schema
Date Tue, 15 Dec 2015 21:29:40 GMT
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