Yes, this is a limitation of sys.fn_validate_plan_guide that can cause a false negative (not a false positive).
The server attempts to compile just the statement in the plan guide, not the whole batch (stored procedure in this case). The request fails because the temporary table definition is not part of the compilation.
I could not reproduce the same issue with a table variable on:
Microsoft SQL Server 2012 (SP3-CU2) (KB3137746) - 11.0.6523.0 (X64)
Mar 2 2016 21:29:16
Copyright (c) Microsoft Corporation
Developer Edition (64-bit) on Windows NT 6.3 (Build 10586: )
The plan validation and guide application worked correctly.
Workarounds
Requesting an estimated plan for a call to the procedure succeeds, with the resulting batch plan showing the guided plan, with the properties of the statement root node showing the correct PlanGuideDB and PlanGuideName properties in SSMS:
An alternative is to create the temporary table (copied from the procedure definition) on the session that calls sys.fn_validate_plan_guide.
CREATE TABLE #test
(
ID INT
);
-- Succeeds
SELECT
PG.plan_guide_id,
PG.name,
PG.scope_type_desc,
PG.hints,
FVPG.msgnum,
FVPG.severity,
FVPG.[state],
FVPG.[message]
FROM sys.plan_guides AS PG
CROSS APPLY sys.fn_validate_plan_guide(PG.plan_guide_id) AS FVPG;
DROP TABLE #test;
Either of these can be used to validate the negative result from sys.fn_validate_plan_guide when the error refers to a missing #object.
I'm going to go out on a bit of a limb here and guess what you are trying to achieve.
I suspect that you want to find the nearest neighbour from within the DS1 table for all the rows in DS1.
For a test set I created the following randomly filled table.
-- Test table
CREATE TABLE DS1 (
MSPID INTEGER IDENTITY(1,1) NOT NULL PRIMARY KEY,
Location Geometry NOT NULL
);
-- 1 million random points
INSERT INTO DS1 (Location)
SELECT TOP 1000000
Geometry::Point(
RAND(CAST(NEWID() AS VARBINARY))*100000,
RAND(CAST(NEWID() AS VARBINARY))*100000,
0) Location
FROM TALLY;
CREATE SPATIAL INDEX DS1_SIDX ON DS1 (Location)
USING GEOMETRY_AUTO_GRID
WITH (BOUNDING_BOX =(-15000, -15000, 2015000, 2015000));
I a version of your initial query for a quick test, but unfortunately on my desktop the optimiser decide that parallel was better and ignored the spatial index resulting in a 3 second query. Restricting it to a single core caused it to use the index and returned in milliseconds.
-- Nearest neighbour test
DECLARE @Location Geometry = Geometry::Point(50000,50000,0);
/* Ignored index
SQL Server Execution Times:
CPU time = 21781 ms, elapsed time = 3805 ms.
*/
SELECT TOP 1
MSPID,
Location.STDistance(@location) Distance
FROM DS1
WHERE Location.STDistance(@location) IS NOT NULL
ORDER BY Location.STDistance(@location)
/* Used index
SQL Server Execution Times:
CPU time = 0 ms, elapsed time = 10 ms.
*/
SELECT TOP 1
MSPID,
Location.STDistance(@location) Distance
FROM DS1
WHERE Location.STDistance(@location) IS NOT NULL
ORDER BY Location.STDistance(@location)
OPTION (MAXDOP 1);
To achieve what you want for a single row from DS1 and assuming that you want the nearest row from DS1 to the one you've picked, the following uses the index for me and again works in milliseconds.
-- For a specific row using cross apply
DECLARE @MSPID INTEGER = 500000;
SELECT
a.MSPID FromMSPID,
b.MSPID ToMSPID,
b.Distance
FROM DS1 a
CROSS APPLY (
SELECT TOP 1
MSPID,
c.Location.STDistance(a.Location) Distance
FROM DS1 c
WHERE c.Location.STDistance(a.Location) IS NOT NULL AND
c.MSPID <> a.MSPID
ORDER BY c.Location.STDistance(a.Location)
) b
WHERE a.MSPID = @MSPID
OPTION (MAXDOP 1);
This can be made to go through the entire table, but it will take some time to complete as it needs to find the nearest neighbour for each one. It took ~1 minute for 10,000, so for the full million that would take almost 2 hours. I have found that restricting this query to a single core isn't required and allowing parallism will improve performance. For me it halved the time. I would also recommend that you have a look at this question and it's answer for other performance considerations.
/* For all (well 10,000) using cross apply
SQL Server Execution Times:
CPU time = 60641 ms, elapsed time = 64545 ms.
*/
SELECT TOP 10000
a.MSPID FromMSPID,
b.MSPID ToMSPID,
b.Distance
FROM DS1 a
CROSS APPLY (
SELECT TOP 1
MSPID,
c.Location.STDistance(a.Location) Distance
FROM DS1 c
WHERE c.Location.STDistance(a.Location) IS NOT NULL AND
c.MSPID <> a.MSPID
ORDER BY c.Location.STDistance(a.Location)
) b
OPTION (MAXDOP 1);
Best Answer
The index use seems irrelevant to the concatenation.
The other 2 solutions are awkward.
Demo