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

Implement SQL sinks #903

Open
wants to merge 22 commits into
base: dev
Choose a base branch
from
Open

Implement SQL sinks #903

wants to merge 22 commits into from

Conversation

jo-bao
Copy link
Contributor

@jo-bao jo-bao commented Oct 12, 2023 •
edited
Loading

Resolves #762

@jo-bao jo-bao closed this Oct 12, 2023
@jo-bao jo-bao deleted the jb/#762-sql-grid-datasources branch October 12, 2023 13:39
@jo-bao jo-bao restored the jb/#762-sql-grid-datasources branch October 12, 2023 13:39
@jo-bao jo-bao reopened this Oct 12, 2023
@jo-bao jo-bao marked this pull request as draft October 12, 2023 13:42
t-ober
t-ober requested changes Oct 16, 2023
View reviewed changes
Copy link
Contributor

@t-ober t-ober left a comment

Choose a reason for hiding this comment

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

That looks already very good. Very nice work @jo-bao 🚀

src/main/java/edu/ie3/datamodel/io/sink/SqlSink.java Outdated Show resolved Hide resolved
src/main/java/edu/ie3/datamodel/io/sink/SqlSink.java Outdated Show resolved Hide resolved
src/main/java/edu/ie3/datamodel/io/DatabaseIdentifier.java Outdated Show resolved Hide resolved
src/main/java/edu/ie3/datamodel/io/DatabaseIdentifier.java Outdated Show resolved Hide resolved
src/main/java/edu/ie3/datamodel/io/DatabaseIdentifier.java Outdated Show resolved Hide resolved
src/main/java/edu/ie3/datamodel/io/sink/SqlSink.java Outdated Show resolved Hide resolved
src/main/java/edu/ie3/datamodel/io/sink/SqlSink.java Outdated Show resolved Hide resolved
src/main/java/edu/ie3/datamodel/io/sink/SqlSink.java Outdated Show resolved Hide resolved
src/test/groovy/edu/ie3/datamodel/io/sink/SqlSinkTest.groovy Outdated Show resolved Hide resolved
src/test/groovy/edu/ie3/datamodel/io/sink/SqlSinkTest.groovy Outdated Show resolved Hide resolved
@jo-bao jo-bao marked this pull request as ready for review November 22, 2023 08:37
@sebastian-peter
Copy link
Member

I guess this was blocked by the PSU not being released, but can now be unblocked by #1006

jo-bao and others added 5 commits February 26, 2024 09:34
@jo-bao
Merge remote-tracking branch 'origin/dev' into jb/#762-sql-grid-datas…
31da1fd 
…ources

# Conflicts:
#	src/test/groovy/edu/ie3/test/common/TimeSeriesTestData.groovy
@jo-bao
Merge branch 'dev' into jb/#762-sql-grid-datasources
4243e57 
# Conflicts:
#	src/main/java/edu/ie3/datamodel/io/naming/DatabaseNamingStrategy.java
fixing tests
d85e234
@jo-bao
replace UniqueEntity
15d97cc
@jo-bao
Main Method
d77e478
@jo-bao jo-bao requested review from sebastian-peter and removed request for sebastian-peter July 2, 2024 11:44
@jo-bao jo-bao added the io Issues relating to input/output label Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t-ober t-ober t-ober requested changes

@sebastian-peter sebastian-peter Awaiting requested review from sebastian-peter

Requested changes must be addressed to merge this pull request.

Assignees

@jo-bao jo-bao

Labels
io Issues relating to input/output
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Adding SQL sink
3 participants
@jo-bao @sebastian-peter @t-ober