[GitHub] flink pull request: [FLINK-1655] Renamed StreamInvokable to Stream...

2015-04-15 Thread akshaydixi
Github user akshaydixi closed the pull request at:

https://github.com/apache/flink/pull/508


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request: fixed a few typos in internal_general_arch.md

2015-04-15 Thread akshaydixi
GitHub user akshaydixi opened a pull request:

https://github.com/apache/flink/pull/602

fixed a few typos in internal_general_arch.md



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/akshaydixi/flink typo

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/flink/pull/602.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #602


commit b6491c33d3d4dccd8759839f6e9479f97a74b2ce
Author: Akshay Dixit akshayd...@gmail.com
Date:   2015-04-15T13:26:55Z

fixed a few typos in internal_general_arch.md




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request: [FLINK-1655] Renamed StreamInvokable to Stream...

2015-04-14 Thread akshaydixi
Github user akshaydixi commented on the pull request:

https://github.com/apache/flink/pull/508#issuecomment-92691766
  
Ok. So should I delete this branch then?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request: [FLINK-1655] Renamed StreamInvokable to Stream...

2015-03-21 Thread akshaydixi
GitHub user akshaydixi opened a pull request:

https://github.com/apache/flink/pull/508

[FLINK-1655] Renamed StreamInvokable to StreamOperator and all its 
subclasses and tests in a similar fashion.

Changes made are:

* StreamInvokable - StreamOperator
* ChainableInvokable - ChainableStreamOperator
* SinkInvokable - SinkStreamOperator
* SourceInvokable - SourceStreamOperator

* CoFlatMapInvokable - CoFlatMapStreamOperator
* CoGroupedReduceInvokable - CoGroupedReduceStreamOperator 
* CoInvokable - CoStreamOperator
* CoMapInvokable - CoMapStreamOperator
* CoReduceInvokable - CoReduceStreamOperator
* CoWindowInvokable - CoWindowStreamOperator

* GroupedWindowBufferInvokable - GroupedWindowBufferStreamOperator
* WindowBufferInvokable - WindowBufferStreamOperator

* CounterInvokable - CounterStreamOperator
* FilterInvokable - FilterStreamOperator
* FlatMapInvokable - FlatMapStreamOperator
* GroupedFoldInvokable - GroupedFoldStreamOperator
* GroupedReduceInvokable - GroupedReduceStreamOperator
* MapInvokable - MapStreamOperator
* ProjectInvokable - ProjectStreamOperator 
* StreamFoldInvokable - FoldStreamOperator
* StreamReduceInvokable - ReduceStreamOperator

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/akshaydixi/flink FLINK-1655

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/flink/pull/508.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #508


commit 18ba6e31b7cd0eb0eac857595ba8ad304d81202a
Author: Akshay Dixit akshayd...@gmail.com
Date:   2015-03-21T11:28:53Z

[FLINK-1655] Renamed StreamInvokable to StreamOperator and all its 
subclasses and tests in a similar fashion.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request: [FLINK-1450] Added fold operator for the Strea...

2015-03-15 Thread akshaydixi
Github user akshaydixi commented on the pull request:

https://github.com/apache/flink/pull/481#issuecomment-80946579
  
Thanks @gyfora. I'll try to not mess up commit history in future 
contributions! 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request: [FLINK-1450] Added fold operator for the Strea...

2015-03-14 Thread akshaydixi
Github user akshaydixi commented on the pull request:

https://github.com/apache/flink/pull/481#issuecomment-80363751
  
Ok @gyfora, thanks for reviewing! 
I picked this issue since it seemed a good way to get a handle of 
flink-streaming as I'm interested in pursuing FLINK-1534 and FLINK-1617. But 
I'll be glad to work more on this and try to implement more of the features 
you've described. They do seem challenging so I'll count on your help in the 
tougher bits :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request: [FLINK-1450] Added fold operator for the Strea...

2015-03-13 Thread akshaydixi
Github user akshaydixi commented on the pull request:

https://github.com/apache/flink/pull/481#issuecomment-79525790
  
Thanks @gyfora . I've added fold into GroupedDataStream and 
WindowedDataStream in subsequent commits.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request: [FLINK-1450] Added fold operator for the Strea...

2015-03-12 Thread akshaydixi
GitHub user akshaydixi opened a pull request:

https://github.com/apache/flink/pull/481

[FLINK-1450] Added fold operator for the Streaming API

Tried to follow the steps described in the JIRA Issue comments:
- Created a FoldFunction and a RichFoldFunction
- Created a StreamFoldInvokable
- Integrated it to the DataStream for both Java and Scala
- Implemented StreamFoldTest based on StreamReduceTest

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/akshaydixi/flink FLINK-1450

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/flink/pull/481.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #481


commit 9848f23a0a73228fb45346d69aee68091b135d3d
Author: Akshay Dixit akshayd...@gmail.com
Date:   2015-03-12T17:49:31Z

[FLINK-1450] Added fold operator for the Streaming API




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request: Fixed simple typos

2015-03-07 Thread akshaydixi
GitHub user akshaydixi opened a pull request:

https://github.com/apache/flink/pull/463

Fixed simple typos

Fixed a few typos I found while reviewing the programming guide.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/akshaydixi/flink dixit_typo

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/flink/pull/463.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #463


commit d43ddba84b2286351e4475be6d697f4e5d4a5719
Author: Akshay Dixit akshayd...@gmail.com
Date:   2015-03-07T18:03:15Z

Fixed simple typos




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---