MarquezProject / marquez

Collect, aggregate, and visualize a data ecosystem's metadata
https://marquezproject.ai
Apache License 2.0
1.78k stars 320 forks source link

Remove limit from assertion evaluation. #2844

Closed phixMe closed 4 months ago

phixMe commented 4 months ago

Problem

There is a limit on the number of assertions we would process for some reason, giving us strange results that were not reflective of the full data set.

Closes: #2839

One-line summary: Fixes bug where our status indicator would be the wrong color.

Checklist

netlify[bot] commented 4 months ago

Deploy Preview for peppy-sprite-186812 ready!

Name Link
Latest commit 19d36e08ca6331b636bce3191e9fb72fdb20390f
Latest deploy log https://app.netlify.com/sites/peppy-sprite-186812/deploys/667ee1cb6dcd210008dc202d
Deploy Preview https://deploy-preview-2844--peppy-sprite-186812.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.56%. Comparing base (b5bcfbf) to head (20e3f50).

:exclamation: Current head 20e3f50 differs from pull request most recent head 19d36e0

Please upload reports for the commit 19d36e0 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2844 +/- ## ============================================ - Coverage 84.74% 84.56% -0.19% + Complexity 1456 1441 -15 ============================================ Files 253 251 -2 Lines 6562 6504 -58 Branches 305 303 -2 ============================================ - Hits 5561 5500 -61 - Misses 850 851 +1 - Partials 151 153 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.