vibhatha commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2060572959
Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
lidavidm commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2060570597
sure, there's no rush
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific
vibhatha commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2060528094
@lidavidm I followed the thread. I think I should be able to, but probably
next week?
--
This is an automated message from the Apache Git Service.
To respond to the message,
lidavidm commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2060405495
Ah sorry, thanks for the reminder
@vibhatha do you think you could put in the fix sometime?
--
This is an automated message from the Apache Git Service.
To respond to the
donraab commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2053712197
@martin-traverse @lidavidm Here's a
[link](https://github.com/apache/arrow/commit/09d6ca747cb9f247a15268cfd04fa6aeb89c3f12)
to the change that I explained in a previous comment may
martin-traverse commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2053610032
@donraab apologies you are correct, EDL is available as an option and
classed as category A by FINOS, we have just not configured it in our license
scanning setup. I had not
jbonofre commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2049158035
@donraab yeah, agree some scanners are not smart enough to deal with dual
licenses
--
This is an automated message from the Apache Git Service.
To respond to the message,
donraab commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2049147285
@jbonofre Thanks! Eclipse Collections is dual-licensed with EDL for this
reason so folks shouldn't have issues including it as a dependency. I am a bit
concerned now if automated
jbonofre commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2049128235
@donraab Hi. The EPL license is Cat B at ASF, meaning that it could be
possible as binary, but avoided in source
(https://www.apache.org/legal/resolved.html). As EDL is a kind of
donraab commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2049114181
@lidavidm You're welcome. I was curious to understand why Apache Arrow had a
dependency on Eclipse Collections which is how I found this issue, albeit a bit
too late unfortunately
lidavidm commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2048875585
Aha, good to know. Thanks for chiming in - I saw the report and just wanted
to make sure we avoided any potential problems even if it turned out to be a
false alarm. (In the end,
donraab commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2048872157
@martin-traverse FYI, Eclipse Collections is dual licensed under EPL and
EDL. One of the biggest users of Eclipse Collections is the Legend project
which is also a FINOS project.
alexanderankin commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2035528131
You can remove the generated annotation now
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
martin-traverse commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2031235200
Thank you so much for the quick response - I look forward to getting the fix
in 16.0!
--
This is an automated message from the Apache Git Service.
To respond to the
kou commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2030936960
Issue resolved by pull request 40904
https://github.com/apache/arrow/pull/40904
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
jbonofre commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2028821829
> I explicitly added the blocker label because I'm not sure if our scripts
treat critical fix as a blocker. I think that label is more for call outs in
the release notes cc @raulcd
assignUser commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2028812716
I explicitly added the blocker label because I'm not sure if our scripts
treat critical fix as a blocker. I think that label is more for call outs in
the release.notra cc @raulcd
jbonofre commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2027931977
Thanks for the report. We have several Maven plugins that can check the
license (rat doesn't check the license by its own). I will take a look to
detect this (we should accept Cat
lidavidm commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2027693200
I'm going to try to get this in ASAP. Actually, I just labeled this as a
critical fix. I put together a basic Java-only license scan already. My
schedule is about to go pear-shaped
martin-traverse commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2027628874
Thanks for the quick feedback - it is actually good from our point of view
to hear that this is not intentional! Is there any chance the fix will be in
version 16 or is it
lidavidm commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2027573940
We also need to set up similar CI to prevent issues like this in the future.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
lidavidm commented on issue #40896:
URL: https://github.com/apache/arrow/issues/40896#issuecomment-2027573546
https://github.com/apache/arrow/commit/09d6ca747cb9f247a15268cfd04fa6aeb89c3f12
Sorry about this. We were trying to eliminate a dependency on Netty from the
core. Looking
22 matches
Mail list logo