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.

Sign up for GitHub

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

Jump to bottom

Use Java streams on SQL fetches effectively #827

Open
sebastian-peter opened this issue Jun 6, 2023 · 0 comments · May be fixed by #1062
Open

Use Java streams on SQL fetches effectively #827

sebastian-peter opened this issue Jun 6, 2023 · 0 comments · May be fixed by #1062
Assignees
@staudtMarius
Labels
io Issues relating to input/output java Pull requests that update Java code

Comments

@sebastian-peter
Copy link
Member

sebastian-peter commented Jun 6, 2023 •
edited

Once #773 is merged:

The call to stream() here does not make much sense if one thinks of why we want to use it in the first place. We fetch all rows for a query first, then apply a stream on it to "lazily" handle the rows one by one.
We definitely should not handle this in this PR though, I think. I found information on how to do it though: https://stackoverflow.com/a/64391247

Originally posted by @sebastian-peter in #773 (comment)
@sebastian-peter sebastian-peter added question Further information is requested java Pull requests that update Java code io Issues relating to input/output and removed question Further information is requested labels Jun 6, 2023
@staudtMarius staudtMarius self-assigned this Apr 3, 2024
@staudtMarius staudtMarius linked a pull request Apr 9, 2024 that will close this issue
Open
staudtMarius added a commit that referenced this issue Apr 9, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
ba15e89 
…ctively
staudtMarius added a commit that referenced this issue Apr 15, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
46fc1b1 
…ctively
staudtMarius added a commit that referenced this issue Apr 23, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
88023cb 
…ctively
danielfeismann added a commit that referenced this issue Apr 24, 2024
@danielfeismann
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
f952cd2 
…ctively
staudtMarius added a commit that referenced this issue Apr 30, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
b3c6db7 
…ctively
staudtMarius added a commit that referenced this issue May 1, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
f01cbbd 
…ctively
staudtMarius added a commit that referenced this issue May 6, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
1194a43 
…ctively
staudtMarius added a commit that referenced this issue May 7, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
fd57b05 
…ctively
danielfeismann added a commit that referenced this issue May 14, 2024
@danielfeismann
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
167f0b7 
…ctively
danielfeismann added a commit that referenced this issue May 16, 2024
@danielfeismann
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
ea320e7 
…ctively
staudtMarius added a commit that referenced this issue May 23, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
ed1fa01 
…ctively
staudtMarius added a commit that referenced this issue Jun 4, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
fa0337e 
…ctively
staudtMarius added a commit that referenced this issue Jun 10, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
abad5ed 
…ctively
sebastian-peter added a commit that referenced this issue Jun 13, 2024
@sebastian-peter
Merge branch 'refs/heads/dev' into ms/#827-use-java-streams-on-sql-fe…
76b1c29 
…tsches-effectively

# Conflicts:
#	CHANGELOG.md
staudtMarius added a commit that referenced this issue Jun 13, 2024
@staudtMarius
Merge branch 'dev' into ms/#827-use-java-streams-on-sql-fetsches-effe…
d8ba515 
…ctively
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@staudtMarius staudtMarius

Labels
io Issues relating to input/output java Pull requests that update Java code
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improving the usage of streams on SQL fetches. ie3-institute/PowerSystemDataModel
2 participants
@sebastian-peter @staudtMarius