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 >> >> >> >> >> >> >> >> >> >
