Skip to content

Conversation

khdang
Copy link
Member

@khdang khdang commented Sep 13, 2019

Description

  • Enable LINQ ThenBy operator after OrderBy to use composite indexing
  • Refactor LINQ tests

E.g

LINQ with ThenBy

query.OrderBy(f => f.FamilyId).ThenBy(f => f.Int).ThenByDescending(f => f.IsRegistered)

SQL

SELECT VALUE root 
FROM root 
ORDER BY root["FamilyId"] ASC, root["Int"] ASC, root["IsRegistered"] DESC

Type of change

  • New feature (non-breaking change which adds functionality)

Closing issues

@khdang khdang added the LINQ label Sep 13, 2019
@khdang khdang self-assigned this Sep 13, 2019
@khdang khdang requested a review from bchong95 September 13, 2019 00:32
@khdang khdang changed the title Enable LINQ ThenBy operator after OrderBy and add type check functions Enable LINQ ThenBy operator after OrderBy Sep 18, 2019
@kirankumarkolli
Copy link
Member

@khdang can you please take a look at the test failures?

@khdang
Copy link
Member Author

khdang commented Oct 3, 2019

@khdang can you please take a look at the test failures?

I've updated the tests.

@khdang khdang closed this Oct 3, 2019
@khdang khdang reopened this Oct 3, 2019
@khdang khdang requested a review from kirankumarkolli October 3, 2019 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants