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

Provide DoobieMultipleResultFunctionWithStatus #118

Closed
lsulak opened this issue Mar 21, 2024 · 0 comments · Fixed by #120
Closed

Provide DoobieMultipleResultFunctionWithStatus #118

lsulak opened this issue Mar 21, 2024 · 0 comments · Fixed by #120
Assignees
@lsulak
Labels
bug Something isn't working enhancement New feature or request

Comments

@lsulak
Copy link
Collaborator

lsulak commented Mar 21, 2024

Background

See the description of this feature in AbsaOSS/atum-service#169 where the idea emerged.
@lsulak lsulak added the enhancement New feature or request label Mar 21, 2024
@lsulak lsulak self-assigned this Mar 21, 2024
lsulak added a commit that referenced this issue Mar 25, 2024
@lsulak
#118: DB cleanup and IT tests addition to CI
0b33532
@lsulak lsulak mentioned this issue Mar 25, 2024
Merged
lsulak added a commit that referenced this issue Mar 25, 2024
@lsulak
#118: further cleanup
f25f70f
lsulak added a commit that referenced this issue Mar 25, 2024
@lsulak
#118: adding support for MultipleResultFunction with status for Doobie
05f4a21
@lsulak lsulak mentioned this issue Mar 25, 2024
Merged
lsulak added a commit that referenced this issue Mar 25, 2024
@lsulak
#118: first set of changes - need to redesign / slightly change the t…
2d1c9ba 
…ypes on many places
lsulak added a commit that referenced this issue Mar 28, 2024
@lsulak
#118: refactoring and changes as per the consultation
eae017a
lsulak added a commit that referenced this issue Mar 28, 2024
@lsulak
#118: DB cleanup and IT tests addition to CI
57aa01b
lsulak added a commit that referenced this issue Mar 28, 2024
@lsulak
#118: test fix
bed8a55
lsulak added a commit that referenced this issue Mar 28, 2024
@lsulak
#118: adding status info into Right - always retrievable
a577c3d
lsulak added a commit that referenced this issue Mar 28, 2024
@lsulak
#118: removing redundant comments and tiny refactoring
34b1287
lsulak added a commit that referenced this issue Mar 28, 2024
@lsulak
#118: fixing integration test due to new design
bdcdce5
@benedeki benedeki added the bug Something isn't working label Apr 2, 2024
lsulak added a commit that referenced this issue Apr 3, 2024
@lsulak
#118: aggregation of errors into a single one, basic three implementa…
3455339 
…tions, unit tests and small refactoring
lsulak added a commit that referenced this issue Apr 3, 2024
@lsulak
#118: forgotten docs update
4d727e8
lsulak added a commit that referenced this issue Apr 3, 2024
@lsulak
#118: better naming
1fdeabb
lsulak added a commit that referenced this issue Apr 3, 2024
@lsulak
#118: better naming
7844278
lsulak added a commit that referenced this issue Apr 3, 2024
@lsulak
#118: majority Win algorithm improved, now it is certain that the fir…
c674be9 
…st one will be returned if multiple errors were present with the same max count
lsulak added a commit that referenced this issue Apr 3, 2024
@lsulak
#118: fixing IT tests, caused by recent auto-rename/refactoring error
fef374d
@benedeki benedeki mentioned this issue Apr 4, 2024
Closed
lsulak added a commit that referenced this issue Apr 4, 2024
@lsulak
#118: post-review improvements
3bba29b
lsulak added a commit that referenced this issue Apr 24, 2024
@lsulak
#118: moving database -> demo_database folder, and moving examples
1650f30
lsulak added a commit that referenced this issue Apr 24, 2024
@lsulak
#118: removing the code in formerly known 'examples/' module - this i…
950947b 
…s a subset of what we have anyway, and it's completely ignored
lsulak added a commit that referenced this issue Apr 26, 2024
@lsulak
Merge pull request #119 from AbsaOSS/feature/118-db-function-cleanup-…
c06a209 
…pre-118

#118: DB cleanup and IT tests addition to CI
lsulak added a commit that referenced this issue Apr 26, 2024
@lsulak
#118: slightly improving type names
60d25d9
lsulak added a commit that referenced this issue Jun 25, 2024
@lsulak
#118: adding support for MultipleResultFunction with status for Doobie (
2e0631a 
#120)

* #118: adding support for MultipleResultFunction with status for Doobie
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lsulak lsulak

Labels
bug Something isn't working enhancement New feature or request
Projects
CPS small repos project
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#118: adding support for MultipleResultFunction with status for Doobie AbsaOSS/fa-db
2 participants
@lsulak @benedeki