Github user HeartSaVioR commented on the pull request:
https://github.com/apache/storm/pull/612#issuecomment-118058205
@ziyunhx
Agreed to @lazyval.
- Project setting files should be removed from repo, which includes PDB,
etc. (Maybe csproj too?)
- If we can make DLLs from source code, DLLs should be removed from repo.
Btw, I wonder Storm community can maintain non-JVM .net codes (which is
actually a project) within JVM project. It also adds/changes 10000+ lines to
storm-core, which is big compared to origin storm-core.
Please note that multilang implementations in storm-core are simple enough
to represent just one file.
They don't require additional libraries installed, too.
There're some advanced Storm multilang projects, for example,
StreamParse(https://github.com/Parsely/streamparse).
.Net multilang adapter could be better to be maintained like StreamParse, I
mean, outer Storm project.
It seems to be more natural.
How about posting new feature to user / dev mailing list and gather a
consensus that we would be better to include this?
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
