Skip to content

Enhance query handling for nested columns and date filters#66

Merged
gutzbenj merged 1 commit intomainfrom
gutzbenj/imrpove
Feb 3, 2026
Merged

Enhance query handling for nested columns and date filters#66
gutzbenj merged 1 commit intomainfrom
gutzbenj/imrpove

Conversation

@gutzbenj
Copy link
Collaborator

@gutzbenj gutzbenj commented Feb 3, 2026

  • Improve nested/dotted column handling: alias nested columns with underscores for consistent querying when data is loaded via database accessors.
  • Consolidate date-type filters to be ORed together when fetching data into memory, allowing multiple date conditions to apply correctly.
  • Add validation to ensure filters with values specify corresponding columns.

- Improve nested/dotted column handling: alias nested columns with underscores for consistent querying when data is loaded via database accessors.
- Consolidate date-type filters to be ORed together when fetching data into memory, allowing multiple date conditions to apply correctly.
- Add validation to ensure filters with values specify corresponding columns.
@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

❌ Patch coverage is 77.77778% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.68%. Comparing base (d383522) to head (4038846).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/koality/executor.py 36.36% 7 Missing ⚠️
src/koality/models.py 72.72% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
+ Coverage   79.47%   84.68%   +5.21%     
==========================================
  Files           7        7              
  Lines        1179     1208      +29     
==========================================
+ Hits          937     1023      +86     
+ Misses        242      185      -57     
Flag Coverage Δ
unittests 84.68% <77.77%> (+5.21%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gutzbenj gutzbenj merged commit ae6c893 into main Feb 3, 2026
13 checks passed
@gutzbenj gutzbenj deleted the gutzbenj/imrpove branch February 3, 2026 21:19
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.

1 participant