Skip to content

feat:(Console): add search feature#187

Merged
jbonofre merged 4 commits intoapache:mainfrom
vignesh-manel:console_search
Mar 20, 2026
Merged

feat:(Console): add search feature#187
jbonofre merged 4 commits intoapache:mainfrom
vignesh-manel:console_search

Conversation

@vignesh-manel
Copy link
Copy Markdown
Contributor

@vignesh-manel vignesh-manel commented Mar 12, 2026

Adds a search dialog to the console to allowing users to quickly navigate to catalogs, namespaces, tables, views, and principals

Features:

  • Search across catalogs, namespaces, tables, views, and principals in one place
  • Word-prefix matching
  • Recently viewed history (up to 8 items) persisted across sessions via localStorage

Limitations:

  • Fetches all data eagerly on dialog open
Screen.Recording.2026-03-12.at.8.11.58.PM.mov

@vignesh-manel vignesh-manel marked this pull request as ready for review March 12, 2026 14:54
@jbonofre jbonofre self-requested a review March 12, 2026 18:44
Copy link
Copy Markdown
Member

@jbonofre jbonofre left a comment

Choose a reason for hiding this comment

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

I love it !

Thanks !

Comment thread console/src/hooks/useSearchData.ts
* "a" → does NOT match "online_store" ✗ (no word starts with "a")
* "a" → matches "accounting" ✓ (word "accounting" starts with "a")
*/
function matchesQuery(text: string, query: string): boolean {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This function only matches single-word queries: it takes the query as a single token.

For instance a namespace like "my namespace" won't match anything.

Maybe we can consider splitting the query on whitespace and requiring each word to match at least one segment.

That's not a blocker but worth to consider.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated the logic to split on whitespace and match segment

@jbonofre jbonofre merged commit 3ae2f7f into apache:main Mar 20, 2026
2 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