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

Update streaming-io.md #27

Merged
merged 1 commit into from Nov 3, 2020
Merged

Update streaming-io.md #27

merged 1 commit into from Nov 3, 2020

Conversation

LorenzBuehmann
Copy link
Contributor

Update necessary due to renaming of StreamOps to StreamRDFOps

Update necessary due to renaming of StreamOps to StreamRDFOps
Copy link
Member

@kinow kinow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @LorenzBuehmann ! One part is duplicate in #22, but the other part is not. That PR is pending the release of Jena before the site is updated.

@@ -87,7 +87,7 @@ an `StreamRDF` backed by a stream-based writer
which can be done as:

StreamRDF writer = StreamRDFWriter.getWriterStream(output, lang) ;
StreamOps.graphToStream(writer, model.getGraph()) ;
StreamRDFOps.graphToStream(writer, model.getGraph()) ;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@afs this line is not in #22 , but the other line is in #22.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

StreamRDFOps is the right name so when this is merged, it'll be right.

@afs afs merged commit 24c7c85 into apache:master Nov 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants