Skip to content

Conversation

@natalya-aksman
Copy link
Member

@natalya-aksman natalya-aksman commented Oct 27, 2025

Fix a typo in the unit test (meant to be reverse order but non-reverse order was actually used) plus added more test cases.

Disable-check: approval-count

Can wait until the next release as no new issues with direct compress insert were uncovered.

@github-actions
Copy link

@erimatnor, @akuzm: please review this pull request.

Powered by pull-review

@natalya-aksman natalya-aksman requested review from Poroma-Banerjee and svenklemm and removed request for akuzm and erimatnor October 27, 2025 15:28
@natalya-aksman natalya-aksman added this to the v2.24.0 milestone Oct 27, 2025
@codecov
Copy link

codecov bot commented Oct 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.54%. Comparing base (904c928) to head (f9592c0).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8843      +/-   ##
==========================================
+ Coverage   82.46%   82.54%   +0.07%     
==========================================
  Files         248      248              
  Lines       47946    47920      -26     
  Branches    12224    12220       -4     
==========================================
+ Hits        39539    39555      +16     
- Misses       3503     3516      +13     
+ Partials     4904     4849      -55     

☔ 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.

@natalya-aksman natalya-aksman force-pushed the add_direct_compress_insert_unit_tests branch from 3a9a79b to f9592c0 Compare October 31, 2025 15:00
@natalya-aksman natalya-aksman merged commit 03bba13 into timescale:main Oct 31, 2025
50 of 52 checks 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