I think your comment was misleading. It sounded like you added the actual dependency to the POM and not just to the LICENSE file.
> On 02 Nov 2015, at 16:26, Aljoscha Krettek <aljos...@apache.org> wrote: > > This is the PR: https://github.com/apache/flink/pull/1316 if anyone is > interested or knows something about how we have to declare licenses. >> On 02 Nov 2015, at 16:10, Aljoscha Krettek <aljos...@apache.org> wrote: >> >> flink-storm has a dependency on storm-core. Therefore I though it should be >> added to the LICENSE file (not the one in root but the one we ship with the >> binary distribution). >>> On 02 Nov 2015, at 16:00, Matthias J. Sax <mj...@apache.org> wrote: >>> >>> You added Storm dependency? Why? Which version? How does it affect >>> storm-compatibility? >>> >>> We decided to not include Storm as dependency to Flink when I started >>> with it. Thus, fat-jars must include Storm classes if storm-compat is use. >>> >>> On 11/02/2015 03:07 PM, Maximilian Michels wrote: >>>> Alright. Good finds. I'll cancel the release candidate and will create >>>> a new one once the fixes are in. >>>> >>>> On Mon, Nov 2, 2015 at 3:02 PM, Aljoscha Krettek <aljos...@apache.org> >>>> wrote: >>>>> I also have an open PR that adds some new dependencies, most notably >>>>> Nifi, Storm, betty-router and Elasticsearch, and removes AWS. >>>>>> On 02 Nov 2015, at 14:39, Aljoscha Krettek <aljos...@apache.org> wrote: >>>>>> >>>>>> -1 >>>>>> We include openjdk JMH and this is GPL v2 license. This could be a >>>>>> problem. >>>>>>> On 02 Nov 2015, at 14:27, Fabian Hueske <fhue...@gmail.com> wrote: >>>>>>> >>>>>>> -1 >>>>>>> A user reported a bug in the Scala DataSet API: FLINK-2953 >>>>>>> >>>>>>> Should be easy to solve. I will provide a fix soon. >>>>>>> >>>>>>> 2015-10-30 15:51 GMT+01:00 Maximilian Michels <m...@apache.org>: >>>>>>> >>>>>>>> We can continue testing now: >>>>>>>> >>>>>>>> https://docs.google.com/document/d/1keGYj2zj_AOOKH1bC43Xc4MDz0eLhTErIoxevuRtcus/edit >>>>>>>> >>>>>>>> On Fri, Oct 30, 2015 at 3:49 PM, Maximilian Michels <m...@apache.org> >>>>>>>> wrote: >>>>>>>> >>>>>>>>> Please vote on releasing the following candidate as Apache Flink >>>>>>>>> version >>>>>>>>> 0.10.0: >>>>>>>>> >>>>>>>>> The commit to be voted on: >>>>>>>>> 6044b7f0366deec547022e4bc40c49e1b1c83f28 >>>>>>>>> >>>>>>>>> Branch: >>>>>>>>> release-0.10.0-rc4 (see >>>>>>>>> https://git1-us-west.apache.org/repos/asf/flink/?p=flink.git) >>>>>>>>> >>>>>>>>> The release artifacts to be voted on can be found at: >>>>>>>>> http://people.apache.org/~mxm/flink-0.10.0-rc4/ >>>>>>>>> >>>>>>>>> The release artifacts are signed with the key with fingerprint >>>>>>>>> C2909CBF: >>>>>>>>> http://www.apache.org/dist/flink/KEYS >>>>>>>>> >>>>>>>>> The staging repository for this release can be found at: >>>>>>>>> https://repository.apache.org/content/repositories/orgapacheflink-1051 >>>>>>>>> >>>>>>>>> ------------------------------------------------------------- >>>>>>>>> >>>>>>>>> The vote is open for the next 72 hours and passes if a majority of at >>>>>>>>> least three +1 PMC votes are cast. >>>>>>>>> >>>>>>>>> The vote ends on Monday November 2, 2015. >>>>>>>>> >>>>>>>>> [ ] +1 Release this package as Apache Flink 0.10.0 >>>>>>>>> [ ] -1 Do not release this package because ... >>>>>>>>> >>>>>>>>> =================================== >>>>>>>>> >>>>>>>>> The following commits have been added on top of release-0.10.0-rc3: >>>>>>>>> >>>>>>>>> 698fbc3 [release][scripts] shade away curator correctly with different >>>>>>>>> Scala versions >>>>>>>>> >>>>>>>> >>>>>> >>>>> >>> >> >
