Skip to content

Added missing stream() overrides to DatasetGraphWithGraphTransform. - #4256

Merged
afs merged 1 commit into
apache:mainfrom
Aklakan:20260923-fix-DatasetGraphWithGraphTransform
Sep 23, 2026
Merged

afs merged 1 commit into
apache:mainfrom
Aklakan:20260923-fix-DatasetGraphWithGraphTransform

Conversation

@Aklakan

@Aklakan Aklakan commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

GitHub issue resolved #

Pull request Description: Fixes a regression observed in DatasetGraphRDFS due to the introduction of DatasetGraphWrapper.stream with #3970. This PR adds missing overrides to the base class DatasetGraphWithGraphTransform.stream.

Blame DatasetGraphWrapper.java#L215-L222


  • Tests are included.
    - [ ] Documentation change and updates are provided for the Apache Jena website
  • Commits have been squashed to remove intermediate development commit messages.
    - [ ] Key commit messages start with the issue number (GH-xxxx)

By submitting this pull request, I acknowledge that I am making a contribution to the Apache Software Foundation under the terms and conditions of the Contributor's Agreement.


See the Apache Jena "Contributing" guide.

@Aklakan
Aklakan force-pushed the 20260923-fix-DatasetGraphWithGraphTransform branch from 677d24f to 499832d Compare September 23, 2026 13:12

@afs afs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved but please remove the unused import.

@@ -35,6 +35,7 @@
import java.util.ArrayList;
import java.util.Iterator;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This can be removed - it is an unused import.

Suggested change
import java.util.Iterator;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

whoops, removed.

@Aklakan
Aklakan force-pushed the 20260923-fix-DatasetGraphWithGraphTransform branch from 499832d to 0a20cd1 Compare September 23, 2026 13:50
@afs
afs merged commit ad65246 into apache:main Sep 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants