I have incorporated the review comments on https://github.com/apache/incubator-apex-malhar/pull/273
[This PR fixes APEXMALHAR-2077 and also includes minor enhancement suggested in APEXMALHAR-2009.] P.S: I am not suggesting to block the release for this ticket. But, it would be nice if reviewers can have a look and merge if it looks OK. ~ Yogi On 16 May 2016 at 23:22, Thomas Weise <[email protected]> wrote: > APEXMALHAR-2009 is a new operator, unless it can be completed right away I > suggest to move it out of the release. > > APEXMALHAR-2077 should not block the release either. > > Thanks, > Thomas > > > On Fri, May 13, 2016 at 1:41 AM, Yogi Devendra < > [email protected] > > wrote: > > > https://github.com/apache/incubator-apex-malhar/pull/273 > > > > This PR fixes APEXMALHAR-2077 and also includes minor enhancement > suggested > > in APEXMALHAR-2009. > > > > @Chandni > > Could you please review and merge this? > > > > > > ~ Yogi > > > > On 13 May 2016 at 05:06, Chandni Singh <[email protected]> wrote: > > > > > APEXMALHAR-2078 > > > APEXMALHAR-2009 > > > > > > Thanks, > > > Chandni > > > > > > On Thu, May 12, 2016 at 4:17 PM, Thomas Weise <[email protected]> > > > wrote: > > > > > > > As discussed, due to the backward incompatible web socket change in > > 3.4.0 > > > > we are going to put out a release candidate soon. > > > > > > > > Issues: > > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/issues/?jql=fixVersion%20%3D%203.4.0%20AND%20project%20%3D%20APEXMALHAR%20ORDER%20BY%20status%20ASC > > > > > > > > There is only one open issue: APEXMALHAR-2081 > > > > > > > > Is there anything else that should make it into the release? > > > > > > > > Thanks, > > > > Thomas > > > > > > > > > >
