apex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Priyanka Gugale <priya...@datatorrent.com>
Subject Re: [APEX-3/APEX-104] Module Separator.
Date Tue, 01 Dec 2015 09:01:32 GMT
+1 for $.

-Priyanka

On Tue, Dec 1, 2015 at 2:26 PM, Chandni Singh <chandni@datatorrent.com>
wrote:

> I will also consider tilde  as a separator. Module1~Operator1.
>
> Less conflict and more readable.
>
> On Tue, Dec 1, 2015 at 12:22 AM, Shubham Pathak <shubham@datatorrent.com>
> wrote:
>
> > +1 for $ or (::) double colon
> >
> > Thanks,
> > Shubham
> >
> > On Tue, Dec 1, 2015 at 11:11 AM, Chinmay Kolhatkar <
> > chinmay@datatorrent.com>
> > wrote:
> >
> > > Hi,
> > >
> > > I want to propose 2 more options for this:
> > > 1) $ (Dollar delimited)
> > > Dollar sign is usually to indicate inner classes in Java. Considering
> > that
> > > I believe this can work in REST APIs and CLI as well.
> > >
> > > Hence the operator inside module can look like:
> > > Module1*$*Module2*$*Operator1
> > >
> > > 2) | (Pipe Delimited)
> > > Operator name would look like:
> > > Module1*|*Module2*|*Operator1
> > >
> > > 3) / (Unix File Seperator)
> > > This seperate is common and used in most of the places. file seperator
> is
> > > also used to indicate the content inside something.
> > > Operator name would look like:
> > > Module1*/*Module2*/*Operator1
> > >
> > > If there are technical challenges in above, I would vote for (::)
> double
> > > colon.
> > >
> > > -Chinmay.
> > >
> > >
> > > ~ Chinmay.
> > >
> > > On Tue, Dec 1, 2015 at 11:01 AM, Tushar Gosavi <tushar@datatorrent.com
> >
> > > wrote:
> > >
> > > > dot can not be used in dt-site.xml, as '.' is used as separator for
> > > parsing
> > > > various component from configuration file.
> > > >
> > > > - Tushar.
> > > >
> > > > On Tue, Dec 1, 2015 at 10:48 AM, Yogi Devendra <
> > yogidevendra@apache.org>
> > > > wrote:
> > > >
> > > > > I would prefer '.' (dot).
> > > > > If there are some technical challenges for dot then I vote for :
> > > (colon)
> > > > or
> > > > > :: (double colon) in that sequence.
> > > > >
> > > > > ~ Yogi
> > > > >
> > > > > On 1 December 2015 at 10:17, Gaurav Gupta <gaurav@datatorrent.com>
> > > > wrote:
> > > > >
> > > > > > What about using “#”? We use this for Unifiers.
> > > > > >
> > > > > > Thanks
> > > > > > - Gaurav
> > > > > >
> > > > > > > On Nov 30, 2015, at 8:08 PM, Tushar Gosavi <
> > tushar@datatorrent.com
> > > >
> > > > > > wrote:
> > > > > > >
> > > > > > > Hi All,
> > > > > > >
> > > > > > > As part of module support, we will need to generate names
for
> > > > operators
> > > > > > > internal to the module before adding it to the original
DAG to
> > > avoid
> > > > > > > conflict of names. For example if a developer adds a operator
> > with
> > > > name
> > > > > > "A"
> > > > > > > and module is name "M" in the DAG. Developer of 'M' is
adding a
> > > > > operator
> > > > > > > 'A' into the module DAG. To avoid name conflict
> > > > > > > with the operator in the main DAG we will generate name
for
> > > internal
> > > > > > > operators by concatenation of module name and operator
name.
> The
> > > 'A'
> > > > > > within
> > > > > > > module 'M' will be added to main DAG as 'M_A'.  This works
for
> > > > > recursive
> > > > > > > modules too. (i.e module containing module).
> > > > > > >
> > > > > > > Which separator should we use for module namespace? The
> > requirement
> > > > are
> > > > > > > - Less chances of collision. (User can add operator with
name
> > 'M_A'
> > > > > too).
> > > > > > > - Compatible with json format. (Should not have a problem
while
> > > > parsing
> > > > > > > json response from rest api)
> > > > > > > - Should be able to specify property in dt-site.xml.
> > > > > > >
> > > > > > > While '_' works, In the pull request it has been pointed
out
> that
> > > it
> > > > > has
> > > > > > > higher chance of collision. Few choices are
> > > > > > >
> > > > > > > 1) _ (Underscore)
> > > > > > > 2) __ (double underscore)
> > > > > > > 3) : (colon, I will check compatibility with rest API,
as it is
> > > used
> > > > to
> > > > > > > separate key and value in json)
> > > > > > > 4) :: (Double colon).
> > > > > > > 5) Let developer choose the separator by specifying dag
> property,
> > > but
> > > > > > > default is still needed.
> > > > > > > 6) Any other option?
> > > > > > >
> > > > > > > Please suggest which approach to take?
> > > > > > >
> > > > > > >
> > > > > > > Regards,
> > > > > > > -Tushar.
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>

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