Re: [RDF] adding Quads - Bulk

2024-03-10 Thread Peter Hull
5 > I did it right after the mail. > > Thanks Fredy > > On 10.03.24 13:42, Peter Hull wrote: > > On Sat, 9 Mar 2024 at 22:37, Fred Hauschel wrote: > > > >> Is there a reason, why there is no Method like GraphLike#add(List > >> statements); ? > > It wou

Re: [RDF] adding Quads - Bulk

2024-03-10 Thread Peter Hull
On Sat, 9 Mar 2024 at 22:37, Fred Hauschel wrote: > Is there a reason, why there is no Method like GraphLike#add(List > statements); ? It would be possible to add a method with a default implementation to GraphLike - then look to see if this can be more efficient for RDF4J (e.g. avoiding the

Re: [COMPRESS] Help needed to fix COMPRESS-651

2024-03-09 Thread Peter Hull
On Sat, 9 Mar 2024 at 18:19, Peter Hull wrote: > There's another constructor for BZip2CompressorInputStream which > allows for this, it's not the default. Specifically, the patch below makes the test pass. Whether this should be default for the one-arg constructor is a matter for disc

Re: [COMPRESS] Help needed to fix COMPRESS-651

2024-03-09 Thread Peter Hull
On Sat, 9 Mar 2024 at 14:33, Gary Gregory wrote: > > If you want to help in Commons COMPRESS-land, please see > https://issues.apache.org/jira/browse/COMPRESS-651 I swear I looked into this a while ago, and the issue was pbzip2 works by compressing the source in chunks, in parallel, then cat'ing

Re: [COMPRESS] Decompress BZIP2 File Max Output is 900000 chars

2024-01-31 Thread Peter Hull
I can't add to the JIRA bug but I had a quick play on WSL (debian), Java 21, compress 1.25.0 and found: Using dd if=/dev/random I could create a big file, compress it with bzip2 and then decompress it with BZip2CompressorInputStream , no problems Same file compressed with pbzip2 was truncated at

Re: [CRYPTO] Why does Makefile use CXX (C++) for linking?

2023-11-02 Thread Peter Hull
On Thu, 2 Nov 2023 at 10:35, sebb wrote: > On macOS, CC and CXX have the same definition, so it's not surprising > there was no difference in your testing. Face palm. Sorry - To unsubscribe, e-mail:

Re: [CRYPTO] Why does Makefile use CXX (C++) for linking?

2023-11-02 Thread Peter Hull
On Wed, 1 Nov 2023 at 23:54, Alex Remily wrote: > I believe it is for cross compilation, owing to the comments in the > makefile: > # for cross-compilation on Ubuntu, install the g++-mingw-w64-x86-64 package I think you could also argue that the other way around - ie. those are the packages you

Re: [IMAGING] Logging vs Throwing exceptions

2023-05-31 Thread Peter Hull
That whole class looks like it needs a bit of TLC (or Javadoc at least!) On Wed, 31 May 2023 at 06:49, Miguel Muñoz wrote: > > > In addition to logging and swallowing the exception, this method also then > returns null. This is also a bad practice. > > > > The caller has to check for null. One

Re: [CSV] New feature to allow access to leading/trailing comments in CSV files?

2022-09-06 Thread Peter Hull
On Tue, 6 Sept 2022 at 15:56, Gilles Sadowski wrote: > > About your patch: It is preferable to have a separate test method for > each test case. If there is no better description, it is fine to append > a "counter" to the "common" test name. i.e. > > Hi Gilles, I have done this, partly, and

Re: [CSV] New feature to allow access to leading/trailing comments in CSV files?

2022-09-06 Thread Peter Hull
Gary > > On Tue, Sep 6, 2022, 06:05 Peter Hull wrote: > > > Hi Bruno, > > Thanks for the swift reply! I have created CSV-304. I attached a patch to > > the ticket but I don't know how to submit a pull request, please could > you > > advise? > > Peter >

Re: [CSV] New feature to allow access to leading/trailing comments in CSV files?

2022-09-06 Thread Peter Hull
Title..." . This way others > can review your code and comment there. And having the JIRA will help > future users with similar use cases in case it's not maintained, or if > there's some other feature they are missing. > > Thanks > -Bruno > > On Tue, 6 Sept 2022 at 20

[CSV] New feature to allow access to leading/trailing comments in CSV files?

2022-09-06 Thread Peter Hull
Dear all, I have an application where it would be useful to be able to get the leading comments (ie. before the first record) from a CSV file. I asked a question on StackOverflow[1] but I got no replies and as far as I can see it's not possible. I looked into implementing this myself and it