Solved

Partition results from a function variable

Posted on 2010-09-20
10
255 Views
Last Modified: 2012-05-10
Hi,

I have this function that returns data from a table. I use the main function to retrieve all the data I want then I try to break it down from a second function cause I have different needs in the reports that I use.

This is the function:

-----------------------
declare @d datetime
set @d = getdate() + {?Days}
select  * from dbo.ServiceViewBatch( @d )
WHERE serviceType = 'VMF'
----------------------

I need to partition the result for this to be the same as another function I used to use to get the same results, this was the old function. Now, I've tried adding the partition bit to the function above but it keeps saying the BY bit is wrong. Any ideas how to integrate the partition bit in the the function above? It's ONLY the partition bit I need to integrate, same BY statements as all of those are in the table I get the data from.

--------------------
  RETURN ( SELECT * FROM (
SELECT countryCode, cityCode, itemCode, storeType, vendorCode, gridCodeX, gridCodeY, facings, brand, rc, storeCode, storeName, ARABIC, ItemName, upcCode, TREF,
ROW_NUMBER() OVER ( PARTITION BY countryCode, cityCode, storeCode, gridCodeX, vendorCode ORDER BY vendorCode DESC) rnum
FROM dbo.ScheduleView
WHERE (CASE DATEPART(weekday, @date )
                      WHEN 1 THEN [SunVMF] WHEN 2 THEN [MonVMF] WHEN 3 THEN [TueVMF] WHEN 4 THEN [WedVMF] WHEN 5 THEN [ThuVMF] WHEN 6 THEN [FriVMF] WHEN 7 THEN
                       [SatVMF] END = 1)) temp
WHERE rnum = 1
   )
----------------------------
0
Comment
Question by:kenuk110
  • 6
  • 4
10 Comments
 
LVL 142

Expert Comment

by:Guy Hengel [angelIII / a3]
ID: 33714524
>Now, I've tried adding the partition bit to the function above but it keeps saying the BY bit is wrong.
can you please clarify what exctly you are adding when it fails?
I mean, the code, as posted, seems fine ...
0
 

Author Comment

by:kenuk110
ID: 33714570
Sorry, yeah I see what you mean.

This is what I run:

declare @d datetime
set @d = getdate() + {?Days}
select  * from dbo.ServiceViewBatch( @d )
WHERE serviceType = 'VMF'

I'd like to have it like this

declare @d datetime
set @d = getdate() + {?Days}
select  *, ROW_NUMBER() OVER ( PARTITION BY countryCode, cityCode, storeCode, gridCodeX, vendorCode ORDER BY vendorCode DESC) from dbo.ServiceViewBatch( @d )
WHERE serviceType = 'VMF'

When I put this in it says it there is an problem with the PARTITION BY bit. It's obviously wrong but I really don't have the skill to figure out which bit is wrong!!

Any help would be really appreciated.

Regards,

Ken
0
 
LVL 142

Expert Comment

by:Guy Hengel [angelIII / a3]
ID: 33714668
please try this:

declare @d datetime
set @d = getdate() + {?Days}
select v.*
, ROW_NUMBER() OVER ( PARTITION BY countryCode, cityCode, storeCode, gridCodeX, vendorCode ORDER BY vendorCode DESC) 
from dbo.ServiceViewBatch( @d ) v
WHERE v.serviceType = 'VMF'

Open in new window

0
PRTG Network Monitor: Intuitive Network Monitoring

Network Monitoring is essential to ensure that computer systems and network devices are running. Use PRTG to monitor LANs, servers, websites, applications and devices, bandwidth, virtual environments, remote systems, IoT, and many more. PRTG is easy to set up & use.

 

Author Comment

by:kenuk110
ID: 33714720
Hi,

It still brings back back all the records and not just a single record, as in I should only see one record for each gridcodex for that store in that city and in that country.

Let me see if I can figure it out or give you some more info.

0
 

Author Comment

by:kenuk110
ID: 33714879
Hi again,

I'm stuck, I have tried 'partitioning' with just the gridcodex, the storecode, most of them but the result still returns all the records for a gridcodex.

This is the function that I'm using to get the main data:

ALTER function [dbo].[ServiceViewBatch](@date date)
returns table
as

RETURN WITH cte AS (
  SELECT [ServiceValue]
       , [ItemName]
       , [countryCode]
       , [storeCode]
       , [cityCode]
       , [itemCode]
       , [storeType]
       , [vendorCode]
       , [upcCode]
       , [rc]
       , [gridCodeX]
       , [gridCodeY]
       , [facings]
       , [brand]
       , [serviceDay]
       , [serviceType]
       , [ARABIC]
       , [TREF]
       , ROW_NUMBER()
            OVER (ORDER BY CountryCode, CityCode, StoreCode, RC, servicetype, gridcodeX, itemCode) rnum
  FROM [T1].[dbo].[ServiceSchedule]
  WHERE serviceday = DATENAME(WEEKDAY, @date)
)
-- final selection
SELECT a.*
     , rt.svSum as RunTotal
     , convert(int, ceiling(rt.svSum/750)) as BatchGroup
FROM cte a
CROSS APPLY(
  SELECT SUM(serviceValue) svSum
  FROM cte b
  WHERE b.rnum <= a.rnum
) rt
------------------------------

The original code is passes the WHERE statement correctly but when it runs the updated code it brings back all the records.

declare @d datetime
set @d = getdate() + {?Days}
select  * from dbo.ServiceViewBatch( @d )
WHERE serviceType = 'VMF'

Really stuck now as to how to select the data so it'd distinct by countryCode, cityCode, storeCode, gridCodeX.
0
 
LVL 142

Expert Comment

by:Guy Hengel [angelIII / a3]
ID: 33714995
note sure, but what about:
ALTER function [dbo].[ServiceViewBatch](@date date)
returns table
as

RETURN WITH cte AS (
  SELECT [ServiceValue]
       , [ItemName]
       , [countryCode]
       , [storeCode]
       , [cityCode]
       , [itemCode]
       , [storeType]
       , [vendorCode]
       , [upcCode]
       , [rc]
       , [gridCodeX]
       , [gridCodeY]
       , [facings]
       , [brand]
       , [serviceDay]
       , [serviceType]
       , [ARABIC]
       , [TREF]
       , ROW_NUMBER()
            OVER (ORDER BY CountryCode, CityCode, StoreCode, RC, servicetype, gridcodeX, itemCode) rnum
  FROM [T1].[dbo].[ServiceSchedule]
  WHERE serviceday = DATENAME(WEEKDAY, @date)
)
-- final selection
SELECT a.*
     , rt.svSum as RunTotal
     , convert(int, ceiling(rt.svSum/750)) as BatchGroup
FROM cte a
CROSS APPLY(
  SELECT SUM(serviceValue) svSum
  FROM cte b
  WHERE b.rnum <= a.rnum
) rt
WHERE a.rnum = 1

Open in new window

0
 

Author Comment

by:kenuk110
ID: 33715243
Hi again,

I have to apologize, I think I've mis comminicated the solution. The function you have sent back needs to be just how it was, it's the main function I use to get back all the data I need. From there I call the function but put in where statements like WHERE serviceType = 'VMF' etc. It just breaks down the full view to just the type of records I need.

The one I'm having trouble with is the small query that I pass to the function, this script:

declare @d datetime
set @d = getdate() + {?Days}
select v.*
, ROW_NUMBER() OVER ( PARTITION BY countryCode, cityCode, storeCode, gridCodeX, vendorCode ORDER BY vendorCode DESC)
from dbo.ServiceViewBatch( @d ) v
WHERE v.serviceType = 'VMF'

I have added this to get what I hoped was distinct per gridcodeX per store per city per country but it brings back all the records attached to that gridcodex. I only actually need one record of any that has this gridcodex assigned to it, it's just the way I need to present the data for this report.

0
 
LVL 142

Accepted Solution

by:
Guy Hengel [angelIII / a3] earned 500 total points
ID: 33715317
hmm...
declare @d datetime
set @d = getdate() + {?Days}
;with data as ( select v.*
, ROW_NUMBER() OVER ( PARTITION BY countryCode, cityCode, storeCode, gridCodeX, vendorCode ORDER BY vendorCode DESC)rn
from dbo.ServiceViewBatch( @d ) v
WHERE v.serviceType = 'VMF'
)
select * from data
where rn = 1

Open in new window

0
 

Author Comment

by:kenuk110
ID: 33715389
PERFECT!!!

Brilliant, thank you so much for this, really.

Regards,

Ken
0
 

Author Closing Comment

by:kenuk110
ID: 33715396
Perfect solution!
0

Featured Post

PRTG Network Monitor: Intuitive Network Monitoring

Network Monitoring is essential to ensure that computer systems and network devices are running. Use PRTG to monitor LANs, servers, websites, applications and devices, bandwidth, virtual environments, remote systems, IoT, and many more. PRTG is easy to set up & use.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Long way back, we had to take help from third party tools in order to encrypt and decrypt data.  Gradually Microsoft understood the need for this feature and started to implement it by building functionality into SQL Server. Finally, with SQL 2008, …
Occasionally there is a need to clean table columns, especially if you have inherited legacy data. There are obviously many ways to accomplish that, including elaborate UPDATE queries with anywhere from one to numerous REPLACE functions (even within…
Two types of users will appreciate AOMEI Backupper Pro: 1 - Those with PCIe drives (and haven't found cloning software that works on them). 2 - Those who want a fast clone of their boot drive (no re-boots needed) and it can clone your drive wh…
Email security requires an ever evolving service that stays up to date with counter-evolving threats. The Email Laundry perform Research and Development to ensure their email security service evolves faster than cyber criminals. We apply our Threat…

832 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question