Re: [DISCUSS][Java] Automatic Modules with JPMS

2024-06-13 Thread Laurent Goujon
I don't even think there's even a wide view across the Java ecosystem regarding JPMS. So far all usages of Apache Arrow I've seen do not require JPMS, and amongst the sister projects I checked (Parquet, Iceberg), none of them have added support for JPMS. I don't know how many dependencies used by

Re: [DISCUSS] Drop Java 8 support

2024-05-24 Thread Laurent Goujon
g list. > > Do we think a vote is needed before officially moving forward with Java 8 > deprecation? > > On Mon, May 20, 2024 at 12:50 PM Laurent Goujon > > wrote: > > > I also mentioned Apache Parquet and haven't seen someone mentioned > if/when > > Apache Parquet

Re: [DISCUSS] Drop Java 8 support

2024-05-20 Thread Laurent Goujon
I also mentioned Apache Parquet and haven't seen someone mentioned if/when Apache Parquet would transition. On Fri, May 17, 2024 at 9:07 AM Dane Pitkin wrote: > Fokko, thank you for these datapoints! It's great to see how other low > level Java OSS projects are approaching this. > > JB, I

Re: Upgrading Java version in build toolchain

2024-05-06 Thread Laurent Goujon
need to find a way to extend the image or create a new one. If people have other options in mind, I'm also all ear. Laurent On Wed, Apr 10, 2024 at 9:07 AM Laurent Goujon wrote: > I can give it a try for sure > > On Fri, Apr 5, 2024 at 10:26 AM Dane Pitkin > wrote: > >> I

Re: [DISCUSS] Drop Java 8 support

2024-05-03 Thread Laurent Goujon
Arrow is used by a couple of projects like Apache Parquet and Apache Iceberg which are still compiled with Java 8. Not a fully OSS project, but Dremio is built with Java 11 and I don't see us migrating to Java 17 until next year. I'm still working on building Arrow with a more recent version of

Re: Upgrading Java version in build toolchain

2024-04-10 Thread Laurent Goujon
decide on a major Arrow version upgrading the target language version. > > That's what I meant by "consensus". > > > > Regards > > JB > > > > On Tue, Apr 2, 2024 at 5:55 PM Laurent Goujon > > wrote: > > > > > > At code level we

Re: Upgrading Java version in build toolchain

2024-04-02 Thread Laurent Goujon
some features > from Java 17+ (I'm thinking about record, memory API, etc). > I would be more than happy to help on this as soon as we have a consensus. > > Thanks, > Regards > JB > > On Mon, Apr 1, 2024 at 7:48 PM Laurent Goujon > wrote: > > > > Hello Arrow Jav

Upgrading Java version in build toolchain

2024-04-01 Thread Laurent Goujon
Hello Arrow Java developers, I would wonder if the community would be okay to change the minimum Java version used by the build toolchain to at least Java 17 or 21 (or even 22). This is different from changing the minimum Java version used at runtime which would still be 8 (following the vote

Re: [Discussion][C++][FlightRPC] What stage to submit a PR for Flight SQL ODBC driver

2023-12-14 Thread Laurent Goujon
s.html#clas > > On Wed, Dec 13, 2023, at 17:56, David Li wrote: > > Yes, I can handle this process on the PMC side. Thanks for the help. > > > > On Wed, Dec 13, 2023, at 16:55, Laurent Goujon wrote: > >> David, would you be the PMC contact for the flightsql-odbc driver?

Re: [Discussion][C++][FlightRPC] What stage to submit a PR for Flight SQL ODBC driver

2023-12-13 Thread Laurent Goujon
help. Please do let us know if there's anything we can do to help as > well. > > [1]: https://github.com/apache/ignite/tree/master/modules/platforms/cpp > ________ > From: Laurent Goujon > Sent: Friday, December 8, 2023 11:01 AM > To: dev@arrow.apache.org

Re: [Discussion][C++][FlightRPC] What stage to submit a PR for Flight SQL ODBC driver

2023-12-08 Thread Laurent Goujon
Am I reading the ticket correctly that this is also about importing some of the Dremio code into Arrow project (namely https://github.com/dremio/flightsql-odbc/). If it is the case, let me check how my company can provide the documentation for the project? On Fri, Dec 8, 2023 at 8:41 AM David Li

Re: [DISCUSS] Flight SQL as experimental

2023-12-08 Thread Laurent Goujon
hu, Dec 7, 2023, at 17:14, Joel Lubinitsky wrote: > > > The message types defined in FlightSql.proto are all marked > experimental > > as > > > well. Would this include changes to any of those? > > > > > > On Thu, Dec 7, 2023 at 16:43 Laurent Goujon &

Re: [DISCUSS] Flight SQL as experimental

2023-12-07 Thread Laurent Goujon
we have been using it with Dremio for a while now, and we consider it stable +1 (not binding) Laurent On Wed, Dec 6, 2023 at 4:52 PM Matt Topol wrote: > +1, I agree with everyone else > > On Wed, Dec 6, 2023 at 7:49 PM James Duong > wrote: > > > +1 from me. It's used in a good number of

Java code review request

2022-01-06 Thread Laurent Goujon
Hi, I made a small change to the Java Arrow library to support newer versions of Jackson, as more projects use Jackson 2.12 or higher but Arrow still uses Jackson 2.11. Jira ticket link is https://issues.apache.org/jira/browse/ARROW-15192 Pull request link is

Re: Re: Re: [DISCUSS][Java] Adding GC-Based reference management strategy for buffers

2021-10-07 Thread Laurent Goujon
> > BufferAllocator root = new RootAllocator( > > > > > > BaseAllocator.configBuilder() > > > > > > .maxAllocation(MAX_ALLOCATION) > > > > > > .memoryChunkManagerFactory(MemoryChunkCleaner.newFactory()) > > // &

Re: Re: [DISCUSS][Java] Adding GC-Based reference management strategy for buffers

2021-10-06 Thread Laurent Goujon
t, or I can open a ticket asap? > > > > > Thanks, > > Hongze > > > > > At 2021-10-05 13:39:55, "Laurent Goujon" wrote: > >Hi, > > > >I gave a quick look at your patches but to be honest, it's not easy to > >figure out the problem y

Re: [DISCUSS][Java] Adding GC-Based reference management strategy for buffers

2021-10-04 Thread Laurent Goujon
Hi, I gave a quick look at your patches but to be honest, it's not easy to figure out the problem you're trying to solve in the first place. Or is it simply to discard the use of ref counting to track buffer usages and relying on Java references to keep track of buffers at the expense of creating

[jira] [Created] (ARROW-9000) Java build crashes with JDK14

2020-06-01 Thread Laurent Goujon (Jira)
Laurent Goujon created ARROW-9000: - Summary: Java build crashes with JDK14 Key: ARROW-9000 URL: https://issues.apache.org/jira/browse/ARROW-9000 Project: Apache Arrow Issue Type: Bug

[jira] [Created] (ARROW-3061) headroom does not take into account reservation

2018-08-15 Thread Laurent Goujon (JIRA)
Laurent Goujon created ARROW-3061: - Summary: headroom does not take into account reservation Key: ARROW-3061 URL: https://issues.apache.org/jira/browse/ARROW-3061 Project: Apache Arrow Issue

Re: JDBC Adapter PR - 1759

2018-05-29 Thread Laurent Goujon
Same here. On Tue, May 29, 2018 at 9:59 AM, Siddharth Teotia wrote: > Hi Atul, > > I will take a look today. > > Thanks, > Sidd > > On Tue, May 29, 2018 at 2:45 AM, Atul Dambalkar < > atul.dambal...@xoriant.com> > wrote: > > > Hi Sid, Laurent, Uwe, > > > > Any idea when can someone take a look

Re: JDBC Adapter for Apache-Arrow

2017-10-31 Thread Laurent Goujon
I really like Julian's idea of unwrapping Arrow objects out of the JDBC ResultSet, but I wonder if the unwrap class has to be specific to the driver and if an interface can be designed to be used by multiple drivers: for drivers based on Arrow, it means you could totally skip the

[jira] [Created] (ARROW-1527) Fix Travis JDK9 build

2017-09-11 Thread Laurent Goujon (JIRA)
Laurent Goujon created ARROW-1527: - Summary: Fix Travis JDK9 build Key: ARROW-1527 URL: https://issues.apache.org/jira/browse/ARROW-1527 Project: Apache Arrow Issue Type: Bug

[jira] [Created] (ARROW-1355) Make arrow buildable with java9

2017-08-15 Thread Laurent Goujon (JIRA)
Laurent Goujon created ARROW-1355: - Summary: Make arrow buildable with java9 Key: ARROW-1355 URL: https://issues.apache.org/jira/browse/ARROW-1355 Project: Apache Arrow Issue Type

[jira] [Created] (ARROW-385) Refactor metrics system

2016-11-21 Thread Laurent Goujon (JIRA)
Laurent Goujon created ARROW-385: Summary: Refactor metrics system Key: ARROW-385 URL: https://issues.apache.org/jira/browse/ARROW-385 Project: Apache Arrow Issue Type: Improvement

Re: Arrow Java Build Failiure

2016-09-01 Thread Laurent Goujon
I just opened a pull request to make flatc optional + allow for a user-provided flatc: https://github.com/apache/arrow/pull/129 On Thu, Sep 1, 2016 at 3:11 PM, Laurent Goujon <laur...@dremio.com> wrote: > I should rephrase: there's no flatc binary artifact published on maven

Re: Arrow Java Build Failiure

2016-09-01 Thread Laurent Goujon
Thu, Sep 1, 2016 at 4:20 PM, Jacques Nadeau <jacq...@apache.org> wrote: > > I wonder if that means we should check in the generated code so people on > > those platforms can still build the java. > > > > On Thu, Sep 1, 2016 at 1:04 PM, Laurent Goujon <laur...@dremio.com>

[jira] [Created] (ARROW-208) Add checkstyle policy to java project

2016-05-25 Thread Laurent Goujon (JIRA)
Laurent Goujon created ARROW-208: Summary: Add checkstyle policy to java project Key: ARROW-208 URL: https://issues.apache.org/jira/browse/ARROW-208 Project: Apache Arrow Issue Type: Bug

[jira] [Commented] (ARROW-102) travis-ci support for java project

2016-05-14 Thread Laurent Goujon (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15283609#comment-15283609 ] Laurent Goujon commented on ARROW-102: -- pull request: https://github.com/apache/arrow/pull/63

[jira] [Commented] (ARROW-198) OutOfMemoryError for vector test case

2016-05-14 Thread Laurent Goujon (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-198?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15283603#comment-15283603 ] Laurent Goujon commented on ARROW-198: -- isn't it ARROW-113? > OutOfMemoryError for vector test c

[jira] [Commented] (ARROW-113) TestValueVector test fails if cannot allocate 2GB of memory

2016-05-14 Thread Laurent Goujon (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15283602#comment-15283602 ] Laurent Goujon commented on ARROW-113: -- pull request: https://github.com/apache/arrow/pull/65

[jira] [Commented] (ARROW-192) some classes cannot be resolved for vector component

2016-05-07 Thread Laurent Goujon (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15275428#comment-15275428 ] Laurent Goujon commented on ARROW-192: -- The templates are under vector/src/main/codegen. Modify

[jira] [Commented] (ARROW-192) some classes cannot be resolved for vector component

2016-05-05 Thread Laurent Goujon (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15272961#comment-15272961 ] Laurent Goujon commented on ARROW-192: -- The maven plugin which generates these classes is not run