LinqToQuerystring icon indicating copy to clipboard operation
LinqToQuerystring copied to clipboard

$inlinecount in EntityFramework doesn't generate a sql count query

Open diegomesata opened this issue 11 years ago • 4 comments

Hello, LinqToQuerystring is a great tool. I tried to use microsoft webapi odata in my web api controllers but $inlinecount just doesn't work out of the box, you have to do a lot of hacks in order to make it works, but LinqToQuerystring solved my problems!

I was debugging the entity framework's sql querys and I found that $inlinecount doesn't generate a sql count query. I think all data are being loaded in memory and after that are being counted (too much memory!).

[6880] SELECT
[6880] [Extent1].[LogId] AS [LogId],
[6880] [Extent1].[Time] AS [Time],
[6880] [Extent1].[Event] AS [Event],
[6880] [Extent1].[UserId] AS [UserId],
[6880] [Extent1].[Message] AS [Message],
[6880] [Extent1].[IP] AS [IP] [6880] FROM [dbo].[Log] AS [Extent1] [6880] ORDER BY [Extent1].[LogId] DESC

I think the call should be:

SELECT
[GroupBy1].[A1] AS [C1] FROM ( SELECT
COUNT(1) AS [A1] FROM [dbo].[Log] AS [Extent1] ORDER BY [Extent1].[LogId] DESC ) AS [GroupBy1]

Could you check it?

diegomesata avatar Jun 06 '14 18:06 diegomesata

Thanks @thehockeygod. Some of the properties you mention are already supported by schema.org. MSRP can be expressed by http://schema.org/priceType and horsepower by http://auto.schema.org/enginePower. http://auto.schema.org/modelDate describes the release date of a vehicle model. Trim levels cannot be easily captured in schema.org (of course you can play with http://schema.org/vehicleInteriorType, http://schema.org/color etc). Towing capacity and other properties you mention should be added either to auto.schema.org hosted extension or to auto external extension we are working on now (see https://www.w3.org/community/gao/).

trypuz avatar May 16 '17 13:05 trypuz

This issue is being tagged as Stale due to inactivity.

github-actions[bot] avatar Jul 26 '20 01:07 github-actions[bot]

See issue #7 for the context of the move from the main Schema.org issue tracker to this repository.

RichardWallis avatar Sep 29 '20 16:09 RichardWallis