Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Add a spouts directory and README for spout implementations #3169

Merged
merged 2 commits into from
Feb 26, 2019

Conversation

nwangtw
Copy link
Contributor

@nwangtw nwangtw commented Jan 24, 2019

Heron community has been interested in maintaining spout implementations for users. There have been discussions in slack channel (#general) and dev@ mailing list.

There are multiple options and each has its own pros and cons. This PR is just a start of the effort to unblock contributors. The directory might be migrated to a different place after a final decision is made.

Please feel free to comment or join the discussion in dev@heron.apache.org.

@huijunw
Copy link
Contributor

huijunw commented Jan 28, 2019

there was a contrib dir, which included spouts.
https://github.com/apache/incubator-heron/tree/0.15.0/contrib
i think we may recover the contrib dir, and incubate codes such as spouts from there till they are migrated to other places. thoughts?

@nwangtw
Copy link
Contributor Author

nwangtw commented Jan 29, 2019

there was a contrib dir, which included spouts.
https://github.com/apache/incubator-heron/tree/0.15.0/contrib
i think we may recover the contrib dir, and incubate codes such as spouts from there till they are migrated to other places. thoughts?

Cool. Thanks!

@nwangtw
Copy link
Contributor Author

nwangtw commented Feb 8, 2019

I am thinking maybe we should have both spouts and bolts. Both are for reusable components anyway. For example KafkaSpout reads from Kafka and KafkaBolt could be the writer.

@simingweng
Copy link
Contributor

@nwangtw definitely make sense to have both Spout and Bolt, Apache Storm has the same thing.

@nwangtw
Copy link
Contributor Author

nwangtw commented Feb 19, 2019

@simingweng thx for the input!

@nwangtw nwangtw merged commit f1e6544 into apache:master Feb 26, 2019
thinker0 added a commit to thinker0/heron that referenced this pull request Mar 19, 2019
…catch-exception-ignore

* commit 'bf1a710ba29a52d7d6d902bd52f31ac1969e3357':
  Fix a path in scripts/ci/build_maven_artifacts.sh (apache#3199)
  Add a spouts directory and README for spout implementations (apache#3169)
sreev pushed a commit to sreev/incubator-heron that referenced this pull request Apr 9, 2020
)

* Add a spouts directory and README for spout impementations

* Add bolts dir
nicknezis pushed a commit that referenced this pull request Sep 14, 2020
* Add a spouts directory and README for spout impementations

* Add bolts dir
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants