Github user gdfm commented on the pull request:
https://github.com/apache/incubator-samoa/pull/11#issuecomment-76979176
Makes sense, as we want to update the Storm dependency anyway.
However, it will for sure happen to have conflicting dependencies on
different platforms, and we don't want to mix them. The tests should ensure
this isolation, i.e., never mix two platforms, and the different profiles aim
at that.
We have a single kryo.version variable to ensure that we don't use
different Kryo versions around the codebase, but in this case it seems to me
that it is necessary.
If Flink depends on 2.23, but we have it set on 2.17, then we should be
able to override it in the samoa-flink module. Do you agree?
---
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.
---
