Skip to content

Conversation

@pulpdrew
Copy link
Contributor

Closes HDX-2641

Summary

With this change, HyperDX will now display pinned filter values as soon as the search page loads, without waiting for the filter values to be queried from ClickHouse. This enables users to quickly apply relevant filters, before the (sometimes very slow) filter values query completes.

Demo

For this demo, I added an artificial delay to the filter query to simulate an environment where filter queries are slow

Screen.Recording.2025-10-28.at.11.44.24.AM.mov

@vercel
Copy link

vercel bot commented Oct 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
hyperdx-v2-oss-app Ready Ready Preview Comment Oct 28, 2025 4:46pm

@changeset-bot
Copy link

changeset-bot bot commented Oct 28, 2025

🦋 Changeset detected

Latest commit: e7d7c81

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@hyperdx/app Patch
@hyperdx/api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@claude
Copy link

claude bot commented Oct 28, 2025

Code Review

No critical issues found.

The implementation looks solid:

  • ✅ Type-safe merge of pinned and queried filter values using Sets
  • ✅ Properly memoized with correct dependencies
  • ✅ Maintains existing behavior while adding new functionality
  • ✅ Clean separation of concerns (pinned filters available immediately, queried filters loaded async)

Minor suggestions (non-blocking):

  • The comment on line 1010 is helpful for future maintainers
  • Consider if sorting logic in shownFacets (lines 879-883) should prioritize pinned filters even when facets are still loading

Good work on improving the UX for slow ClickHouse environments!

@pulpdrew pulpdrew force-pushed the drew/show-pinned-filters-while-loading branch from 1cc0808 to 8a8fe18 Compare October 28, 2025 15:53
@github-actions
Copy link
Contributor

github-actions bot commented Oct 28, 2025

E2E Test Results

All tests passed • 26 passed • 3 skipped • 208s

Status Count
✅ Passed 26
❌ Failed 0
⚠️ Flaky 0
⏭️ Skipped 3

View full report →

@pulpdrew pulpdrew requested review from a team and brandon-pereira and removed request for a team October 28, 2025 16:02
Copy link
Contributor

@knudtty knudtty left a comment

Choose a reason for hiding this comment

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

Fantastic. LGTM

@kodiakhq kodiakhq bot merged commit 3ee93ae into main Oct 28, 2025
9 checks passed
@kodiakhq kodiakhq bot deleted the drew/show-pinned-filters-while-loading branch October 28, 2025 16:47
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.

3 participants