Solved

Group By and Function

Posted on 2011-02-17
5
204 Views
Last Modified: 2012-05-11
This is SQL 2000....

 I have the SQl below. I replaced that SUM statement with a function. This gets a summary. Since I added the function call, I need
to add "group by" at the end of the SQL. But that will cause it to give lots of of rows..because of that PO_Date. How can I change this OR change
 the function to get me the same rows/results as that original SUM?
---***** Function ************----
ALTER FUNCTION [dbo].[fnCalculateCutOffSum]
(@orderDate datetime, @warehouse varchar(2),@ShipMethodAbbrev varchar(10))
RETURNS int
AS
BEGIN
-- *** Used in WHT_OrderSummary
declare @Today int
  -- minus 45 mins..deduct that from @orderDate 
  select @Today = sum(case 
                    when datepart(hour,DateAdd(minute,-45, @orderDate)) <= Datepart(Hour,cutoffHour) 
                        Then 1 else 0
                    end)  --compare hour portion of dates
   from nbCutOff
     where --Priority = @priority 
        warehouse = @warehouse
        and ShipMethodAbbrev = @ShipMethodAbbrev  

return @Today
end

------------ SQL ------------------
ALTER PROCEDURE [dbo].[OrderSummaryTest]
AS 
BEGIN
	SELECT CASE o.Customer_ID
            WHEN 100 THEN 'X'
            WHEN 200 THEN 'Y'
            WHEN 400 THEN 'W'
            WHEN 500 THEN 'P'
            ELSE 'B'
       END AS Company,
       sum(...) AS UPS,
       sum(...) AS AIR,
       sum(..) AS USPS,
       sum(..) AS MAN,
     
    
       dbo.fnCalculateCutOffSum (o.PO_Date,'FN','UPSA') as Today,--added this but then i need to add "group by" for o.PO_Date
       /*sum(CASE 
                WHEN o.PO_Date < dateadd(hour,19,CONVERT(varchar,getdate(),101)) AND c.Abbrev_Name = 'UPSA' THEN 1
                WHEN o.PO_Date < dateadd(hour,14,CONVERT(varchar,getdate(),101)) AND c.Abbrev_Name = 'USPS' THEN 1
                WHEN o.PO_Date < dateadd(hour,15,convert(varchar,getdate(),101)) THEN 1
       ELSE 0
           END) AS Today,*/
      
       count(*) AS Total,
      
FROM   Orders o
.......
 
GROUP BY
       CASE o.Customer_ID
            WHEN 100 THEN 'X'
            WHEN 200 THEN 'Y'
            WHEN 400 THEN 'W'
            WHEN 500 THEN 'P'
            ELSE 'B'
       END 
       
ORDER BY
       1 desc

END

Open in new window

0
Comment
Question by:Camillia
  • 3
  • 2
5 Comments
 
LVL 51

Expert Comment

by:HainKurt
Comment Utility
looks like this is correct

       /*sum(CASE
                WHEN o.PO_Date < dateadd(hour,19,CONVERT(varchar,getdate(),101)) AND c.Abbrev_Name = 'UPSA' THEN 1
                WHEN o.PO_Date < dateadd(hour,14,CONVERT(varchar,getdate(),101)) AND c.Abbrev_Name = 'USPS' THEN 1
                WHEN o.PO_Date < dateadd(hour,15,convert(varchar,getdate(),101)) THEN 1
       ELSE 0
           END) AS Today,*/

you cannot use the function here, that function will be called once, but above statement will run for each row in the group... whats the problem with the code above?
0
 
LVL 51

Accepted Solution

by:
HainKurt earned 500 total points
Comment Utility
maybe you can use

sum(dbo.fnCalculateCutOffSum (o.PO_Date,'FN','UPSA')) as Today

0
 
LVL 7

Author Comment

by:Camillia
Comment Utility
>>but above statement will run for each row in the group...
I want to run function to run for each row...basically, it should be the same results..

Let me try your second suggestion...the thing is...the function is already summing up...so maybe remove the sum from the function?
0
 
LVL 51

Expert Comment

by:HainKurt
Comment Utility
the function will be called for each row in the group... so, yes, you should return something for each row, not for whole group...
0
 
LVL 7

Author Comment

by:Camillia
Comment Utility
I will close it but dont quite get it. Will open another one if i dont figure it out.
0

Featured Post

How your wiki can always stay up-to-date

Quip doubles as a “living” wiki and a project management tool that evolves with your organization. As you finish projects in Quip, the work remains, easily accessible to all team members, new and old.
- Increase transparency
- Onboard new hires faster
- Access from mobile/offline

Join & Write a Comment

Suggested Solutions

Title # Comments Views Activity
Restore Procedure question 4 28
SQL 2012 and SQL 2014 always on 9 23
Calculating Business Hours 18 58
Report Builder 9 22
Introduction SQL Server Integration Services can read XML files, that’s known by every BI developer.  (If you didn’t, don’t worry, I’m aiming this article at newcomers as well.) But how far can you go?  When does the XML Source component become …
Ever wondered why sometimes your SQL Server is slow or unresponsive with connections spiking up but by the time you go in, all is well? The following article will show you how to install and configure a SQL job that will send you email alerts includ…
This video shows, step by step, how to configure Oracle Heterogeneous Services via the Generic Gateway Agent in order to make a connection from an Oracle session and access a remote SQL Server database table.
Using examples as well as descriptions, and references to Books Online, show the different Recovery Models available in SQL Server and explain, as well as show how full, differential and transaction log backups are performed

728 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

Need Help in Real-Time?

Connect with top rated Experts

10 Experts available now in Live!

Get 1:1 Help Now