The GitHub Actions job "GraphAr C++ CI Nightly" on incubator-graphar.git/main has failed. Run started by GitHub user acezen (triggered by acezen).
Head commit for run: 277d6669f9c495afe0806324bb1c3d496c724ae5 / alex <[email protected]> feat(java): let the storage API reach the local filesystem (#960) * feat(java): let the storage API reach the local filesystem The storage API merged in #958 has no implementation, so nothing can open a GraphAr file yet. This adds the local filesystem adapter that the same issue asked for, keeping the module free of any GraphAr layout, format, or query concern: it resolves a URI to a path, opens a seekable input over it, and writes through a position-reporting output. Directories are created on demand for an output file, an existing target is refused unless the caller asked to replace it, and a read past the end of a file reports the end of the stream rather than a partial buffer. Closes the adapter half of #953. Not-tested: only the file scheme is exercised; other schemes are the concern of later adapters. * feat(java): reuse one transfer buffer for direct-buffer writes Review feedback on #960: a write from a buffer without a backing array allocated a fresh staging array on every call. The output is single threaded already, because it tracks its own position, so one instance buffer can serve every such write. It is allocated on first use, so an output that only ever sees heap buffers never pays for it. Adds the missing test for a direct buffer larger than one transfer chunk, which the previous test did not reach: with the copy loop reduced to a single pass the new test fails with "expected:<20000> but was:<8192>". Report URL: https://github.com/apache/incubator-graphar/actions/runs/32766692797 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
