SQLite: Client-eval OrderBy() on DateTimeOffset, decimal, TimeSpan & ulong

[bricelam]

#11889 updates <, <=, >, >=, MIN & MAX to client-eval on these types. Should we do the same for ORDER BY?

[ajcvickers]

Triage: yes, do it for order-by.

[smitpatel]

Presently SqlTranslatingExpressionVisitor does not have information about it is translating OrderBy terms. It would be breaking change to introduce it.

[smitpatel]

Possible duplicate of #10265

[bricelam]

@smitpatel Are Average, Max, Min, and Sum regressed in 3.0? If not, I think we can punt this.

[smitpatel]

Aggregate are not blocked right now. It can be done once we refactor for GroupBy aggregate support.

[divega]

Clearing up milestone so we can decide what to do in triage.

[Ignatella]

And TimeSpan OrderBy still does not work

[ajcvickers]

@Ignatella Please file a new issue and attach a small, runnable project or post a small, runnable code listing that reproduces what you are seeing so that we can investigate.

[bricelam]

@Ignatella After #18844, you could do .OrderBy(e => e.TimeSpan.TotalDays)