Skip to content

Conversation

gribnoysup
Copy link
Collaborator

Following up on a recent discussion, we want to make some changes to the default sort logic:

  • sort should never apply to views because even with _ids it's not taking index into account making queries extremely slow
  • $natural gets pushed to the very bottom of the list to de-prioritize it, we also add a note that it's not a safe sort order to use in production
image

@gribnoysup gribnoysup requested a review from a team as a code owner August 28, 2025 16:05
@github-actions github-actions bot added the feat label Aug 28, 2025
Copy link
Collaborator

@paula-stacho paula-stacho left a comment

Choose a reason for hiding this comment

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

Maybe the Not available for views and timeseries. could be made more prominent (bold or so)?

@gribnoysup gribnoysup merged commit f8370d2 into main Sep 1, 2025
56 of 57 checks passed
@gribnoysup gribnoysup deleted the COMPASS-9749 branch September 1, 2025 13:26
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