Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add FeedParser to archetype topology? #551

Closed
jnioche opened this issue Mar 23, 2018 · 1 comment
Closed

Add FeedParser to archetype topology? #551

jnioche opened this issue Mar 23, 2018 · 1 comment
Labels
archetype enhancement fetcher good first issue parser question
Milestone
1.9

Comments

@jnioche
Copy link
Contributor

jnioche commented Mar 23, 2018

The topology in the archetype uses the SitemapParser, why not add the FeedParser as well?
@jnioche
Copy link
Contributor Author

jnioche commented May 22, 2018 •
edited

One side effect of not doing it is that feeds end up being indexed, which is not really useful. Removing the parser for cases where we want to index the feeds, is trivial.

@jnioche jnioche added this to the 1.9 milestone May 22, 2018
jnioche added a commit that referenced this issue Jun 11, 2018
@jnioche
bug: java topology in archetype does not use FeedParserBolt, fixes #551
7bf4a48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
archetype enhancement fetcher good first issue parser question
Projects
None yet
Milestone
1.9
Development

No branches or pull requests
1 participant
@jnioche