Skip to content

Conversation

@royischoss
Copy link
Contributor

@royischoss royischoss commented Dec 16, 2025

📝 Description

Optimize model monitoring stream pod, avoiding working on duplicate events in TSDB steps


🛠️ Changes Made

This change includes:

  1. Splitting the graph earlier between parquet to tsdb branches, avoiding flatten for events that target to the TSDB
  2. Small change in the handling of NOP event
  3. Mapping the first event of batch to the TSDB while this event hold all necessary data
  4. v3io - include events table, added a MapFeatureNamesTSDB for some fields need to be covered

TDEngine and TimeScaleDB:
image

V3io:
image


✅ Checklist

  • I updated the documentation (if applicable)
  • I have tested the changes in this PR
  • I confirmed whether my changes are covered by system tests
    • If yes, I ran all relevant system tests and ensured they passed before submitting this PR
    • I updated existing system tests and/or added new ones if needed to cover my changes
  • If I introduced a deprecation:

🧪 Testing

Manual for v3io and TimeScaleDB, test_app_flow with ModelRunnerStep using both v3io and TimeScaleDB TSDB


🔗 References


🚨 Breaking Changes?

  • Yes (explain below)
  • No

🔍️ Additional Notes

@royischoss royischoss marked this pull request as ready for review December 16, 2025 09:55
@royischoss royischoss requested a review from a team as a code owner December 16, 2025 09:55
Copy link
Member

@Eyal-Danieli Eyal-Danieli left a comment

Choose a reason for hiding this comment

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

nice, a small comment

@assaf758 assaf758 merged commit e4e8718 into mlrun:development Dec 16, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants