October 24, 2012 at 7:24 pm
Hi All,
I need to calculate running total (totalInterestedPaid) for the following temp table (#tmpInterestedPaid)
tran_dateCalc_interest_accruedInterest_paidTotalInterestPaid
2012090854.06 NULL 0
2012091054.06 NULL 0
2012091124.04 112.35 0
2012091323.67 20.12 0
2012091423.67 NULL 0
The totalInterestedPaid should be additional the previous date Calc_interest_accrued with today Calc_interest_accrued but whenever there is interest_paid is not equal to null the Running total will be reset to zero.
The expected result is as follow
tran_dateCalc_interest_accruedInterest_paidTotalInterestPaid
2012090854.06 NULL 54.06
2012091054.06 NULL 108.12
2012091124.04 112.35 132.16
2012091323.67 20.12 23.67
2012091423.67 NULL 47.34
I believe recursive CTE will work but just not sure how to code it.
Thanks,
Derek
October 28, 2012 at 7:54 pm
I'm thinking that you may have your interest accured and interest paid terminology a little mixed up here.
Take a look at the following Quirky Update (QU) method for calculating what I think you need.
CREATE TABLE #Pmts
(tran_date DATETIME PRIMARY KEY CLUSTERED
,Calc_interest_accrued MONEY, Interest_paid MONEY
,TotalInterestPaid MONEY, TotalInterestAccrued MONEY
,TotalInterestDue AS (TotalInterestAccrued - TotalInterestPaid))
INSERT INTO #Pmts (tran_date, Calc_interest_accrued, Interest_paid
,TotalInterestPaid, TotalInterestAccrued)
SELECT '2012-09-08',54.06,NULL,0,0
UNION ALL SELECT '2012-09-10',54.06,NULL,0,0
UNION ALL SELECT '2012-09-11',24.04,112.35,0,0
UNION ALL SELECT '2012-09-13',23.67,20.12,0,0
UNION ALL SELECT '2012-09-14',23.67,NULL,0,0
DECLARE @InterestPaid MONEY, @InterestAccrued MONEY
SELECT @InterestPaid = 0, @InterestAccrued = 0
UPDATE #Pmts
SET TotalInterestPaid = ISNULL(@InterestPaid, 0) + TotalInterestPaid
,TotalInterestAccrued = TotalInterestAccrued + @InterestAccrued
,@InterestPaid = ISNULL(Interest_paid, 0) + @InterestPaid
,@InterestAccrued = @InterestAccrued + Calc_interest_accrued
SELECT * FROM #Pmts
DROP TABLE #Pmts
Note that QU requires the CLUSTERED index on your date column and it will be much faster than any recursive approach to solving this problem.
Even though it doesn't produce exactly the results you need, I'm hoping it puts you on the right track.
My thought question: Have you ever been told that your query runs too fast?
My advice:
INDEXing a poor-performing query is like putting sugar on cat food. Yeah, it probably tastes better but are you sure you want to eat it?
The path of least resistance can be a slippery slope. Take care that fixing your fixes of fixes doesn't snowball and end up costing you more than fixing the root cause would have in the first place.
Need to UNPIVOT? Why not CROSS APPLY VALUES instead?[/url]
Since random numbers are too important to be left to chance, let's generate some![/url]
Learn to understand recursive CTEs by example.[/url]
[url url=http://www.sqlservercentral.com/articles/St
October 28, 2012 at 9:16 pm
dwain.c (10/28/2012)
I'm thinking that you may have your interest accured and interest paid terminology a little mixed up here.Take a look at the following Quirky Update (QU) method for calculating what I think you need.
CREATE TABLE #Pmts
(tran_date DATETIME PRIMARY KEY CLUSTERED
,Calc_interest_accrued MONEY, Interest_paid MONEY
,TotalInterestPaid MONEY, TotalInterestAccrued MONEY
,TotalInterestDue AS (TotalInterestAccrued - TotalInterestPaid))
INSERT INTO #Pmts (tran_date, Calc_interest_accrued, Interest_paid
,TotalInterestPaid, TotalInterestAccrued)
SELECT '2012-09-08',54.06,NULL,0,0
UNION ALL SELECT '2012-09-10',54.06,NULL,0,0
UNION ALL SELECT '2012-09-11',24.04,112.35,0,0
UNION ALL SELECT '2012-09-13',23.67,20.12,0,0
UNION ALL SELECT '2012-09-14',23.67,NULL,0,0
DECLARE @InterestPaid MONEY, @InterestAccrued MONEY
SELECT @InterestPaid = 0, @InterestAccrued = 0
UPDATE #Pmts
SET TotalInterestPaid = ISNULL(@InterestPaid, 0) + TotalInterestPaid
,TotalInterestAccrued = TotalInterestAccrued + @InterestAccrued
,@InterestPaid = ISNULL(Interest_paid, 0) + @InterestPaid
,@InterestAccrued = @InterestAccrued + Calc_interest_accrued
SELECT * FROM #Pmts
DROP TABLE #Pmts
Note that QU requires the CLUSTERED index on your date column and it will be much faster than any recursive approach to solving this problem.
Even though it doesn't produce exactly the results you need, I'm hoping it puts you on the right track.
Not quite right. You need to modify the update to have a FROM clause from the target table so that you can add the MAXDOP option to prevent parallelism. Like this...
UPDATE tgt
SET TotalInterestPaid = ISNULL(@InterestPaid, 0) + TotalInterestPaid
,TotalInterestAccrued = TotalInterestAccrued + @InterestAccrued
,@InterestPaid = ISNULL(Interest_paid, 0) + @InterestPaid
,@InterestAccrued = @InterestAccrued + Calc_interest_accrued
FROM #Pmts tgt WITH(TABLOCKX)
OPTION (MAXDOP 1)
;
The TablockX isn't required on a Temp Table but it'll bypass all sorts of row escalation to make leaner (memory-wise) and faster code.
I haven't checked the rest of the code nor have my changes made it solve the OP's precise problem.
--Jeff Moden
Change is inevitable... Change for the better is not.
October 28, 2012 at 9:25 pm
Jeff,
Thanks for keeping me honest. I must be rusty from my holiday away to have forgotten the MAXDOP and TABLOCK.
This should also work, no?
UPDATE #Pmts WITH(TABLOCK)
SET TotalInterestPaid = ISNULL(@InterestPaid, 0) + TotalInterestPaid
,TotalInterestAccrued = TotalInterestAccrued + @InterestAccrued
,@InterestPaid = ISNULL(Interest_paid, 0) + @InterestPaid
,@InterestAccrued = @InterestAccrued + Calc_interest_accrued
OPTION (MAXDOP 1)
My thought question: Have you ever been told that your query runs too fast?
My advice:
INDEXing a poor-performing query is like putting sugar on cat food. Yeah, it probably tastes better but are you sure you want to eat it?
The path of least resistance can be a slippery slope. Take care that fixing your fixes of fixes doesn't snowball and end up costing you more than fixing the root cause would have in the first place.
Need to UNPIVOT? Why not CROSS APPLY VALUES instead?[/url]
Since random numbers are too important to be left to chance, let's generate some![/url]
Learn to understand recursive CTEs by example.[/url]
[url url=http://www.sqlservercentral.com/articles/St
October 28, 2012 at 9:58 pm
Don't know. I've never tried it that way. I know the way I demonstrated works. Can't vouch for the way you showed at all.
--Jeff Moden
Change is inevitable... Change for the better is not.
Viewing 5 posts - 1 through 4 (of 4 total)
You must be logged in to reply to this topic. Login to reply