[GitHub] samza pull request #728: User provided configuration should have higher prec...

2018-10-15 Thread prateekm
GitHub user prateekm opened a pull request: https://github.com/apache/samza/pull/728 User provided configuration should have higher precedence than Samza generated configuration. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] samza pull request #727: SAMZA-1953: Remove TableDescriptorsProvider interfa...

2018-10-15 Thread atoomula
GitHub user atoomula opened a pull request: https://github.com/apache/samza/pull/727 SAMZA-1953: Remove TableDescriptorsProvider interface Also cleaned up few Samza-sql warnings. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] samza pull request #714: Samza 1930: Blog for SAMZA 1.0 (Release Notes/Meetu...

2018-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/714 ---

[GitHub] samza pull request #728: SAMZA-1954: User provided configuration should have...

2018-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/728 ---

[GitHub] samza pull request #729: Fix case-studies for LinkedIn, Optimizely, Tripadvi...

2018-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/729 ---

[GitHub] samza pull request #729: Fix case-studies for LinkedIn, Optimizely, Tripadvi...

2018-10-15 Thread vjagadish1989
GitHub user vjagadish1989 opened a pull request: https://github.com/apache/samza/pull/729 Fix case-studies for LinkedIn, Optimizely, Tripadvisor, Slack. Rewrite some of them. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] samza pull request #730: Samza 1950: Fixing Style for Meetups

2018-10-15 Thread Sanil15
GitHub user Sanil15 opened a pull request: https://github.com/apache/samza/pull/730 Samza 1950: Fixing Style for Meetups You can merge this pull request into a Git repository by running: $ git pull https://github.com/Sanil15/samza SAMZA-1950 Alternatively you can review and

[GitHub] samza pull request #731: Removing regex-input static-input exclusivity check

2018-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/731 ---

[GitHub] samza pull request #731: Removing regex-input static-input exclusivity check

2018-10-15 Thread rmatharu
GitHub user rmatharu opened a pull request: https://github.com/apache/samza/pull/731 Removing regex-input static-input exclusivity check This PR converts a hard-check that fails in case of regex-defined input if there is an overlap between regex-input and task-input, to a

Re: [VOTE] SEP-15: New Runtime Context API

2018-10-15 Thread Prateek Maheshwari
+1 (non-binding) for these changes. - Prateek > On Oct 12, 2018, at 3:27 PM, Cameron Lee wrote: > > Hi all, > > SEP-15 has been updated now that SAMZA-1714 has been reviewed and implemented. > Please vote on whether there are further breaking changes needed in the API > or we can accept this

Re: [VOTE] SEP-13: unified ApplicationDescriptor and ApplicationRunner APIs for high and low- level APIs in YARN and standalone deployment

2018-10-15 Thread Xinyu Liu
+1 (binding). Thanks for the effort to making our API consistent. Thanks, Xinyu On Fri, Oct 12, 2018 at 12:32 PM Prateek Maheshwari wrote: > +1 (non-binding) from me. Thanks for making the changes and updating the > SEP! > > - Prateek > > On Fri, Oct 12, 2018 at 12:15 PM Yi Pan wrote: > > > >

[GitHub] samza pull request #726: SAMZA-1952: StreamPartitionCountMonitor for standal...

2018-10-15 Thread shanthoosh
GitHub user shanthoosh opened a pull request: https://github.com/apache/samza/pull/726 SAMZA-1952: StreamPartitionCountMonitor for standalone. You can merge this pull request into a Git repository by running: $ git pull https://github.com/shanthoosh/samza

Re: [VOTE] SEP-13: unified ApplicationDescriptor and ApplicationRunner APIs for high and low- level APIs in YARN and standalone deployment

2018-10-15 Thread Jagadish Venkatraman
+1(binding) from my side. This is a great overhaul of our APIs. Thanks On Monday, October 15, 2018, Xinyu Liu wrote: > +1 (binding). Thanks for the effort to making our API consistent. > > Thanks, > Xinyu > > On Fri, Oct 12, 2018 at 12:32 PM Prateek Maheshwari > wrote: > > > +1 (non-binding)

Re: [VOTE] SEP-14: System and Stream Descriptors

2018-10-15 Thread Xinyu Liu
+1 (binding). Great API for IO! Thanks, Xinyu On Fri, Oct 12, 2018 at 5:36 PM Yi Pan wrote: > +1 (binding) from me. Thanks! > > On Fri, Oct 12, 2018 at 12:30 PM Prateek Maheshwari > wrote: > > > Hi folks, > > > > Now that SAMZA-1804 has been implemented and reviewed, we've updated > > SEP-14

[GitHub] samza pull request #726: SAMZA-1952: StreamPartitionCountMonitor for standal...

2018-10-15 Thread shanthoosh
Github user shanthoosh closed the pull request at: https://github.com/apache/samza/pull/726 ---

[GitHub] samza pull request #726: SAMZA-1952: StreamPartitionCountMonitor for standal...

2018-10-15 Thread shanthoosh
GitHub user shanthoosh reopened a pull request: https://github.com/apache/samza/pull/726 SAMZA-1952: StreamPartitionCountMonitor for standalone. This patch adds the capability to detect the partition change of the input streams of a stateless standalone jobs and trigger a

Re: [VOTE] SEP-14: System and Stream Descriptors

2018-10-15 Thread Jagadish Venkatraman
+1 binding from my side, thanks! This is a great addition to Samza-1.0 On Fri, Oct 12, 2018 at 12:30 PM Prateek Maheshwari wrote: > Hi folks, > > Now that SAMZA-1804 has been implemented and reviewed, we've updated > SEP-14 with the latest APIs and design decisions. > > Please vote for

Re: [VOTE] SEP-15: New Runtime Context API

2018-10-15 Thread Prateek Maheshwari
+1 (non-binding) for these changes. (Resending from a non-LI email due to email delivery issues) - Prateek On Fri, Oct 12, 2018 at 3:28 PM Cameron Lee wrote: > > Hi all, > > SEP-15 has been updated now that SAMZA-1714 has been reviewed and implemented. > Please vote on whether there are further

Re: [VOTE] SEP-15: New Runtime Context API

2018-10-15 Thread Xinyu Liu
+1 (binding). @Cameron: your first voting email seems not showing up for me. For future communications, please use your personal email instead. Thanks, Xinyu On Mon, Oct 15, 2018 at 10:44 AM Prateek Maheshwari wrote: > +1 (non-binding) for these changes. > > (Resending from a non-LI email due

Re: [VOTE] SEP-15: New Runtime Context API

2018-10-15 Thread Jagadish Venkatraman
+1 (binding) from my side. LGTM On Mon, Oct 15, 2018 at 10:44 AM Prateek Maheshwari wrote: > +1 (non-binding) for these changes. > > (Resending from a non-LI email due to email delivery issues) > > - Prateek > On Fri, Oct 12, 2018 at 3:28 PM Cameron Lee wrote: > > > > Hi all, > > > > SEP-15

[GitHub] samza pull request #730: Samza-1950: Fixing Style for Meetups

2018-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/730 ---