Igniters, I would like to resurrect this discussion and will review the change again shortly. If anyone want to join the review - you are welcome!
ср, 22 авг. 2018 г. в 18:49, Denis Magda <dma...@apache.org>: > Hi Dmitry, > > That's a BSD-3-Clause license if to believe this statement > "SPDX-License-Identifier: BSD-3-Clause": > https://github.com/pmem/llpl/blob/master/LICENSE > > This license can be used with ASF software: > https://www.apache.org/legal/resolved.html#category-a > > -- > Denis > > On Wed, Aug 22, 2018 at 9:28 AM Dmitriy Pavlov <dpavlov....@gmail.com> > wrote: > > > Hi Denis, > > > > Could you please double check if we can refer to any library licensed to > > Intel. Can we develop code only version of this support (without shipping > > it in release)? > > > > https://github.com/apache/ignite/pull/4381 is quite huge change, > > including 128 files changed, patch review will require resources from > > community members to review. I would like to be sure we can include this > > patch from the legal point of view. > > > > Sincerely, > > Dmitriy Pavlov > > > > пт, 3 авг. 2018 г. в 19:23, Dmitriy Pavlov <dpavlov....@gmail.com>: > > > >> Hi Mulugeta, > >> > >> I appologise, I've missed that license is already there. So I guess it > is > >> not standard open-source license, it is seems it is not listed in > >> https://www.apache.org/legal/resolved.html#category-a > >> > >> So there can be legal concern related to including this lib as > dependency > >> into Apache product. It should not block review, we can later > >> consult Secretary/Legal to find out how we can correctly include > reference > >> to lib. > >> > >> Sincerely, > >> Dmitriy Pavlov > >> > >> чт, 2 авг. 2018 г. в 0:24, Mammo, Mulugeta <mulugeta.ma...@intel.com>: > >> > >>> Hi Dmitriy, > >>> > >>> Do you mean our LLPL library? It has a license, please look here: > >>> https://github.com/pmem/llpl > >>> > >>> Regarding the changes made to Ignite, you may refer to the pull request > >>> here: https://github.com/apache/ignite/pull/4381 > >>> > >>> Thanks, > >>> Mulugeta > >>> > >>> -----Original Message----- > >>> From: Dmitriy Pavlov [mailto:dpavlov....@gmail.com] > >>> Sent: Wednesday, August 1, 2018 10:49 AM > >>> To: dev@ignite.apache.org > >>> Subject: Re: Adding experimental support for Intel Optane DC Persistent > >>> Memory > >>> > >>> Hi Mulugeta Mammo, > >>> > >>> I've just noticed that repository, what you refer is full fork of > Ignite. > >>> How can I see differences with original Ignite? > >>> > >>> One more thing, library which you're referencing seems to not contain > >>> license, at least github can not parse it. Apache product has > limitations > >>> which libraries may be used (see > >>> https://www.apache.org/legal/resolved.html#category-a and > >>> https://www.apache.org/legal/resolved.html#category-b) > >>> > >>> Could you please comment if there is some legal risk? > >>> > >>> Sincerely, > >>> Dmitriy Pavlov > >>> > >>> ср, 1 авг. 2018 г. в 20:43, Dmitriy Pavlov <dpavlov....@gmail.com>: > >>> > >>> > Hi, > >>> > > >>> > This link works for me > >>> > > >>> > > https://cwiki.apache.org/confluence/display/IGNITE/IEP-26%3A+Adding+Ex > >>> > perimental+Support+for+Intel+Optane+DC+Persistent+Memory > >>> > > >>> > Sincerely, > >>> > Dmitriy Pavlov > >>> > > >>> > чт, 26 июл. 2018 г. в 15:31, Stanislav Lukyanov < > >>> stanlukya...@gmail.com>: > >>> > > >>> >> Ah, ok, it’s just the ‘.’ at the end of the link. Removed it and > it’s > >>> >> fine. > >>> >> > >>> >> From: Stanislav Lukyanov > >>> >> Sent: 26 июля 2018 г. 15:12 > >>> >> To: dev@ignite.apache.org > >>> >> Subject: RE: Adding experimental support for Intel Optane DC > >>> >> Persistent Memory > >>> >> > >>> >> Hi, > >>> >> > >>> >> The link you’ve shared gives me 404. > >>> >> Perhaps you need to add a permission for everyone to access the > page? > >>> >> > >>> >> Thanks, > >>> >> Stan > >>> >> > >>> >> From: Mammo, Mulugeta > >>> >> Sent: 26 июля 2018 г. 2:44 > >>> >> To: dev@ignite.apache.org > >>> >> Subject: Adding experimental support for Intel Optane DC Persistent > >>> >> Memory > >>> >> > >>> >> Hi, > >>> >> > >>> >> I have added a new proposal to support Intel Optane DC Persistent > >>> >> Memory for Ignite here: > >>> >> > https://cwiki.apache.org/confluence/display/IGNITE/Adding+Experimenta > >>> >> l+Support+for+Intel+Optane+DC+Persistent+Memory > >>> >> . > >>> >> > >>> >> I'm looking forward to your feedback and collaboration on this. > >>> >> > >>> >> Thanks, > >>> >> Mulugeta > >>> >> > >>> >> > >>> >> > >>> >> > >>> > >> >