Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Query: string.Concat gives different results than L2O for some types #13104

Closed
maumar opened this issue Aug 24, 2018 · 2 comments
Closed

Query: string.Concat gives different results than L2O for some types #13104

maumar opened this issue Aug 24, 2018 · 2 comments
Assignees

Comments

@maumar
Copy link
Contributor

maumar commented Aug 24, 2018

query:

from g in gs
from m in ms
orderby g.Nickname, m.Id
select new
{
	HasSoulPatch = string.Concat("HasSoulPatch " + g.HasSoulPatch, " HasSoulPatch"),
	Rank = string.Concat("Rank " + g.Rank, " Rank"),
	SquadId = string.Concat("SquadId " + g.SquadId, " SquadId"),
	Rating = string.Concat("Rating " + m.Rating, " Rating"),
	Timeline = string.Concat("Timeline " + m.Timeline, " Timeline")
}

result:

{ HasSoulPatch = HasSoulPatch 1 HasSoulPatch, Rank = Rank 1 Rank, SquadId = SquadId 1 SquadId, Rating = Rating 2.1 Rating, Timeline = Timeline 0002-01-02 10:00:00.0000000 +01:30 Timeline }

l2o:

{ HasSoulPatch = HasSoulPatch True HasSoulPatch, Rank = Rank Corporal Rank, SquadId = SquadId 1 SquadId, Rating = Rating 2.1 Rating, Timeline = Timeline 1/2/0002 10:00:00 AM +01:30 Timeline }
@ajcvickers
Copy link
Contributor

We may need to consider cases like this when working on the 3.0 behavior for funcletization (#12795) but it may also be just a case of translating to SQL Server appropriately using case to go from the store representation to the string we want.

@ajcvickers ajcvickers modified the milestones: 3.0.0, Backlog Jan 25, 2019
@ajcvickers
Copy link
Contributor

Duplicate of #14205

@ajcvickers ajcvickers marked this as a duplicate of #14205 Jan 25, 2019
@ajcvickers ajcvickers removed this from the Backlog milestone Jan 25, 2019
@ajcvickers ajcvickers reopened this Oct 16, 2022
@ajcvickers ajcvickers closed this as not planned Won't fix, can't repro, duplicate, stale Oct 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants