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 <[email protected]> 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 <[email protected]> > 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 <[email protected] > > > > 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 < > [email protected]> > > > 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 > > > > > > > > > >
