Skip to content

feat: Improve data source page UX and add connection testing - #7125

Open
ntkathole wants to merge 1 commit into
feast-dev:masterfrom
ntkathole:dataSource_UX
Open

ntkathole wants to merge 1 commit into
feast-dev:masterfrom
ntkathole:dataSource_UX

Conversation

@ntkathole

Copy link
Copy Markdown
Member

What this PR does / why we need it:

This PR improves the Feast UI Data Sources pages and adds a Test Connection
capability backed by a new registry endpoint.

UI improvements

  • Two views — a card gallery (default) and an enhanced table, each with
    per-type brand logos and a Batch/Streaming/On-Demand category badge.
  • Listing: search + type filter + sort, removable filter chips with
    "Clear all", table pagination + column sorting, relative "Created" times,
    and a distinct "no matches" empty state.
  • Detail page: redesigned with breadcrumbs, a hero header, a full
    connection-configuration panel (per source type), and a metadata sidebar
    (owner, event-timestamp columns, event-time range, created, last updated,
    tags, consuming views). Friendly rendering of well-known tags (e.g. PII, team).
  • Test Connection panel: runs a live check, shows status + discovered
    schema table, and remembers the last result across navigation/reloads.
Screenshot 2026-11-10 at 8 58 42 PM Screenshot 2026-11-10 at 9 06 23 PM

@ntkathole ntkathole self-assigned this Nov 10, 2026
@ntkathole
ntkathole requested a review from a team as a code owner November 10, 2026 15:40
@codecov-commenter

codecov-commenter commented Nov 10, 2026 •

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 62.63736% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.60%. Comparing base (ff8abe0) to head (cb8bd73).
⚠️ Report is 18 commits behind head on master.

Files with missing lines Patch % Lines
sdk/python/feast/registry_server.py 63.41% 25 Missing and 5 partials ⚠️
...k/python/feast/api/registry/rest/saved_datasets.py 20.00% 4 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7125      +/-   ##
==========================================
+ Coverage   52.25%   52.60%   +0.35%     
==========================================
  Files         459      459              
  Lines       58045    58290     +245     
  Branches     8550     8598      +48     
==========================================
+ Hits        30331    30664     +333     
+ Misses      25640    25525     -115     
- Partials     2074     2101      +27     
Flag Coverage Δ
go-feature-server 30.96% <ø> (+0.37%) ⬆️
python-unit 54.12% <62.63%> (+0.34%) ⬆️
Files with missing lines Coverage Δ
sdk/python/feast/api/registry/rest/data_sources.py 64.67% <100.00%> (+0.71%) ⬆️
...k/python/feast/api/registry/rest/saved_datasets.py 48.00% <20.00%> (-0.52%) ⬇️
sdk/python/feast/registry_server.py 54.89% <63.41%> (+1.09%) ⬆️

... and 24 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ff8abe0...cb8bd73. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ntkathole
ntkathole force-pushed the dataSource_UX branch 2 times, most recently from 9b80c51 to 8aa1d43 Compare November 11, 2026 03:16

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants