Mailing-List: contact dev-help@apex.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@apex.incubator.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CAHekGF9L2rvxCax5Ue6=L0wDaRaE++dij5LjW6+OA0pgdJQ=8g@mail.gmail.com>
References: 
 <CAHekGF9L2rvxCax5Ue6=L0wDaRaE++dij5LjW6+OA0pgdJQ=8g@mail.gmail.com>
Date: Thu, 3 Mar 2016 08:46:55 -0800
Message-ID: 
 <CAKJfLDPVSB-cvRJ9aLftm+PvNbxDpbOo64gM-6=eijbw_ONJYA@mail.gmail.com>
Subject: Re: nomenclature change for APPLICATION_WINDOW_COUNT
From: Thomas Weise <thomas@datatorrent.com>
To: dev@apex.incubator.apache.org
Content-Type: multipart/alternative; boundary=001a113ce5680b458a052d27bf45

--001a113ce5680b458a052d27bf45
Content-Type: text/plain; charset=UTF-8

Yogi,

This is a good suggestion. We should change it in the next major release.

Perhaps we can open a JIRA that we can use to list this and similar items
so they are not forgotten.

With respect to windowing in general we should look into expanding from the
current processing time based approach. We support event time in several
operators. We can look at Beam and Flink to broaden this.

Thomas


On Thu, Mar 3, 2016 at 4:32 AM, Yogi Devendra <yogidevendra@apache.org>
wrote:

> Hi,
>
> I studied source code, docs related to windowing for my presentation on
> "Windowing in Apex".
>
> After going through the phase of learning, teaching apex windowing concepts
> to others; I realized that attribute APPLICATION_WINDOW_COUNT from
> OperatorContext should have better name.
>
> Because:
> 1. From the name it looks like a attribute of the APPLICATION; but actually
> it is an Operator attribute.
> 2. It is very confusing to say that APPLICATION_WINDOW_COUNT can be
> different for different Operators within an Application.
>
> Can we have some better name for this attribute?
> Any suggestions for the new name?
> I propose following names:
> 1. OPERATOR_WINDOW_COUNT
> 2. OPERATOR_WINDOW_SPAN
>
> I understand that, introducing new name to the existing attribute might
> break backward compatibility. But, we can deal with that by deprecating the
> older name and keeping it alive till next major release.
>
> What is your opinion on this?
>
> ~ Yogi
>

--001a113ce5680b458a052d27bf45--