Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide HDTManager::generateHDT consuming an InputStream #86

Closed
jmkeil opened this issue Jan 16, 2019 · 3 comments
Closed

Provide HDTManager::generateHDT consuming an InputStream #86

jmkeil opened this issue Jan 16, 2019 · 3 comments
Milestone

Comments

@jmkeil
Copy link

jmkeil commented Jan 16, 2019

It would be very helpful to have a method HDTManager::generateHDT(InputStream, String, RDFNotation, HDTOptions, ProgressListener), which consumes a InputStream instead of a file path.

In case of TempHDTImporterOnePass it is quiet straightforward to do that. In case of TempHDTImporterTwoPass the things are more complicated, as the InputStream would be needed to consume twice.

@mielvds
Copy link
Member

mielvds commented Apr 16, 2021

@jmkeil good suggestion! it would be too complicated with TempHDTImporterTwoPass though. always welcome to submit a PR

@jmkeil
Copy link
Author

jmkeil commented Apr 27, 2021

@jmkeil […] always welcome to submit a PR

As I decided against using HDT in my project, I don't have time to do that. Sorry.

@ate47
Copy link
Contributor

ate47 commented Feb 1, 2023

It was implemented with #162 at this line:

public static HDT generateHDT(InputStream fileStream, String baseURI, RDFNotation rdfNotation, CompressionType compressionType, HDTOptions hdtFormat, ProgressListener listener) throws IOException, ParserException {

So we can close this issue :)

@D063520 D063520 closed this as completed Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants