sadpandajoe commented on PR #40039: URL: https://github.com/apache/superset/pull/40039#issuecomment-4521686901
@rusackas @rebenitez1802 > Concern: sort(sortComparatorForNoSearch) on search results. The server already returned results ranked by relevance. Sorting client-side with sortComparatorForNoSearch could reorder them and push the most relevant match down. Is this intentional? The existing mergeData also sorts, but for search results, server ranking is typically preferred. This deserves a comment or consideration. > > Concern: page > 0 during search (the else branch, line 401+). The "append normally" path for search pagination pages isn't explicitly shown in the diff but falls through to the original mergeData path. Verify that paginating through search results (scrolling the dropdown) still works correctly. Both of these have the same current behavior as master. Unsure if we want in-line comments, but the functionality hasn't changed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
