I believe I have addressed the issue in https://issues.apache.org/jira/browse/ARROW-10499 / https://github.com/apache/arrow/pull/8595 but it looks like there are other unrelated CI errors happening right now 😕.
On 2020/11/04 17:17:14, Terence Honles <terence.hon...@gmail.com> wrote: > Hi Bryan, > > I started looking into the issue myself. I bisected one of the issues to a > change in apache/arrow#8533 > <https://github.com/apache/arrow/pull/8533> and asked > the author > <https://github.com/apache/arrow/pull/8533#issuecomment-721815501> about > the check that seemed to be causing the issue. Benjamin replied back as you > can see in the thread and it looks like the `DCHECK_NE` line can be > removed. Locally, I'm also seeing another issue I'm starting to debug, but > the following diff "fixes" the builder for me locally: > > diff --git a/cpp/src/arrow/io/memory.cc b/cpp/src/arrow/io/memory.cc > index 1cde5e64e..88f006fe0 100644 > --- a/cpp/src/arrow/io/memory.cc > +++ b/cpp/src/arrow/io/memory.cc > @@ -54,7 +54,7 @@ BufferOutputStream::BufferOutputStream(const > std::shared_ptr<ResizableBuffer>& b > Result<std::shared_ptr<BufferOutputStream>> BufferOutputStream::Create( > int64_t initial_capacity, MemoryPool* pool) { > // ctor is private, so cannot use make_shared > - DCHECK_NE(pool, nullptr); > auto ptr = std::shared_ptr<BufferOutputStream>(new BufferOutputStream); > RETURN_NOT_OK(ptr->Reset(initial_capacity, pool)); > return ptr; > diff --git > a/java/flight/flight-core/src/test/java/org/apache/arrow/flight/TestDoExchange.java > b/java/flight/flight-core/src/test/java/org/apache/arrow/flight/TestDoExchange.java > index b7e7a20be..04383c476 100644 > --- > a/java/flight/flight-core/src/test/java/org/apache/arrow/flight/TestDoExchange.java > +++ > b/java/flight/flight-core/src/test/java/org/apache/arrow/flight/TestDoExchange.java > @@ -303,7 +303,7 @@ public class TestDoExchange { > final FlightStream reader = stream.getReader(); > reader.cancel("", null); > // Cancel should be idempotent > - reader.cancel("", null); > } > } > > The second change is due to the test suite throwing an exception about the > test having a memory leak, and could also be related to apache/arrow#8533 > <https://github.com/apache/arrow/pull/8533> since that updates memory > allocation. > > -Terence > > On 2020/11/03 20:53:59, Bryan Cutler <c...@gmail.com> wrote: > > There seems to be a Github check error with the Java JNI tests for the > ORC> > > adapter that is affecting a lot of recent PRs, see> > > https://github.com/apache/arrow/pull/8577/checks?check_run_id=1346780145.> > > > From the log, it looks like some env setting, but I can't tell what's> > > wrong. Anyone else know?> > > > > Thanks,> > > Bryan> > > >
