marquez icon indicating copy to clipboard operation
marquez copied to clipboard

Filtering/Searching for jobs and datasets in the Marquez API

Open phixMe opened this issue 1 year ago • 3 comments

Problem

This is meant to support the efforts of #2610 to add capabilities to our backend to filter both jobs/datasets and updates our api spec. Fixes: #2610 (Partial, only backend)

Solution

Changes queries, updates docs and adds some tests for both jobs and datasets.

One-line summary: Support filtering of jobs and datasets.

Checklist

  • [x] You've signed-off your work
  • [x] Your changes are accompanied by tests (if relevant)
  • [x] Your change contains a small diff and is self-contained
  • [ ] You've updated any relevant documentation (if relevant)
  • [ ] You've included a one-line summary of your change for the CHANGELOG.md (Depending on the change, this may not be necessary).
  • [ ] You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
  • [ ] You've included a header in any source code files (if relevant)

phixMe avatar Sep 20 '23 01:09 phixMe

Codecov Report

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

Project coverage is 83.77%. Comparing base (a515c91) to head (f1efb45). Report is 153 commits behind head on main.

:exclamation: Current head f1efb45 differs from pull request most recent head e01ce0c. Consider uploading reports for the commit e01ce0c to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2617      +/-   ##
============================================
+ Coverage     83.15%   83.77%   +0.62%     
- Complexity     1286     1338      +52     
============================================
  Files           244      247       +3     
  Lines          5942     6115     +173     
  Branches        282      281       -1     
============================================
+ Hits           4941     5123     +182     
+ Misses          854      843      -11     
- Partials        147      149       +2     

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

codecov[bot] avatar Sep 20 '23 01:09 codecov[bot]

Deploy Preview for peppy-sprite-186812 canceled.

Name Link
Latest commit e01ce0c9a96d8a3bf5b68559b0df41d643908cb9
Latest deploy log https://app.netlify.com/sites/peppy-sprite-186812/deploys/65e75a51f4fd470008126a22

netlify[bot] avatar Nov 08 '23 18:11 netlify[bot]

@pawel-big-lebowski I made the changes you requested. I forgot about this PR in the ether.

phixMe avatar Nov 09 '23 22:11 phixMe