Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • Files Files
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 402
    • Issues 402
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 60
    • Merge requests 60
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GNOMEGNOME
  • FilesFiles
  • Merge requests
  • !1008

Slow and steady wins the race: lengthen search activation timeout to improve performance

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Jeff Fortin Tam requested to merge jfft/nautilus:issue_1731_delay_search_activation into master Oct 08, 2022
  • Overview 6
  • Commits 1
  • Pipelines 3
  • Changes 1

A more realistic default delay avoids spamming the search backends.

I've tested this change with Nautilus' master branch through GNOME Builder, and it works great for me on my end. I considered a 500ms delay but it was a bit too noticeable for things like remote search, so I compromised to 400ms based on my calculations of what would be the minimum value we could get away with. I do not recommend going lower than 400ms, at least not until everything else in the search infrastructure is absolutely blazing fast.

While it does not solve every perf issue, it's an improvement, and it unclutters our view to see the remaining performance problems.

Fixes issue #1731 (closed)

Edited Oct 09, 2022 by Jeff Fortin Tam
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: issue_1731_delay_search_activation