feat(graphql): Add sorting option to GraphQL search

[nmbryant]

Checklist

• [x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
• [ ] Links to related issues (if applicable)
• [ ] Tests for the changes have been added/updated (if applicable)
• [ ] Docs related to the changes have been added/updated (if applicable)

[nmbryant]

If there are any tests related to this, please let me know. I took a look and couldn't find tests related to the GraphQL queries. Also, please let me know if there are any particular docs that need to be updated

[github-actions[bot]]

Unit Test Results

     37 files       37 suites   26m 53s :stopwatch:    589 tests    537 :heavy_check_mark: 52 :zzz: 0 :x: 1 324 runs  1 256 :heavy_check_mark: 68 :zzz: 0 :x:

Results for commit be1790eb.

[jjoyce0510]

Thanks for the PR @nmbryant! Will be taking a look this week.

[nmbryant]

Our use case with DataHub plans on using a couple of single-entity searches so we wanted to add sorting on the GraphQL side rather than do the sorting on the UI side. I do need to take a further look at searchAcrossEntities, but I think the single entity search is what we will want to use.

[nmbryant]

@jjoyce0510 Is this something that you want merged upstream? If it is, I can resolve the merge conflicts and make changes based on any feedback you may have. Otherwise, I will just copy the changes into our fork and close this PR.

[jjoyce0510]

@nmbryant We'd definitely be interested in taking this upstream, yes! If you're able to clear the conflicts, we can work to quickly get it in

[nmbryant]

Awesome, I have the merge conflicts resolved locally just need to do some testing tomorrow before it's ready to go.

[nmbryant]

@jjoyce0510 Fixed the merge conflicts and did some testing, it's ready for review

[github-actions[bot]]

Unit Test Results (build & test)

  8 files    8 suites   16s :stopwatch: 73 tests 73 :heavy_check_mark: 0 :zzz: 0 :x:

Results for commit 72d6e139.

:recycle: This comment has been updated with latest results.

[jjoyce0510]

@nmbryant Hi!

I know there's been some time since there was last activity on this PR!

If you're still interested and willing to make these changes, we'll need to update to get the more recent changes. There should be some improvements that will make this PR much easier.

Otherwise, we will look at taking this over. Apologies for the delay on our side.

John

[aditya-radhakrishnan]

Closing this PR due to inactivity. @nmbryant if you'd like to open this up again, please let us know!