SQL Server errors - sql-server

Code:
Declare #ParmDefinition Nvarchar(1000),
#St Nvarchar(500),
#TTable varchar(30)
Set #TTable='[0Detail]'
Declare #TTempStore Table (
Iden Int,
Row_ Int,
Accs_iden int,
Am_Bed Money,
Am_Bes Money,
Doc_No Decimal(15,0),
Desc_ Nvarchar(500),
Checked bit,
Error_ int)
SET #ParmDefinition = N'#alaki table(Iden Int,
Row_ Int,
Accs_iden int,
Am_Bed Money,
Am_Bes Money,
Doc_No Decimal(15,0),
Desc_ Nvarchar(500),Checked bit,Error_ int) OUTPUT '
Set #St = N' Select * into #alaki from '+#TTable
EXECUTE sp_executesql #St, #ParmDefinition, #alaki = #TTempStore
SELECT * FROM #TTempStore
Errors:
Msg 156, Level 15, State 1, Line 1
Incorrect syntax near the keyword 'table'.
Msg 102, Level 15, State 1, Line 1
Incorrect syntax near '#alaki'.


You should post a minimal working sample for us to be able to help you, remember that next time.
Why not declare the table with #TTempStore ?
Declare #St Nvarchar(500),#TTable varchar(30)
Set #TTable='[0Detail]'
create table #TTempStore (
Iden Int,
Row_ Int,
Accs_iden int,
Am_Bed Money,
Am_Bes Money,
Doc_No Decimal(15,0),
Desc_ Nvarchar(500),
Checked bit,
Error_ int)
Set #St=N' Select * into #TTempStore from '+#TTable
EXECUTE #St
Select * from #TTempStore
drop table #TTempStore --here you can leave the table if you have more things to do
I haven't tested it, but it should point you in the right direction. If you need I can probably make a sql fiddle later.


Could the #TTable value be different every time? If it's always going to be [0Detail] then you don't need dynamic SQL. However, here is a dynamic SQL solution to your problem:
Declare #St Nvarchar(500),#TTable varchar(30)
Set #TTable='[0Detail]'
Declare #TTempStore Table (
Iden Int,
Row_ Int,
Accs_iden int,
Am_Bed Money,
Am_Bes Money,
Doc_No Decimal(15,0),
Desc_ Nvarchar(500),
Checked bit,
Error_ int)
Set #St=N' Select * from '+#TTable
INSERT INTO #TTempStore
EXECUTE (#St)
Select * from #TTempStore

Related

Simple insert stored procedure not working

I am trying to make a simple stored procedure but it's not working. When I execute the procedure with my parameters in it, below is the following:
ALTER proc [dbo].[sp_NewProduct]
#ProductName nvarchar(50),
#ProductNumber nvarchar(25),
#MakeFlag bit,
#FinishedGoodsFlag bit,
#Color nvarchar(15),
#SafetyStockLevel smallint,
#ReorderPoint smallint,
#StandardCost money,
#ListPrice money,
#DaysToManufacture int,
#SellStartDate date,
#rowguid uniqueidentifier,
#ModifiedDate datetime
as
insert dbo.product
(Name,
ProductNumber,
MakeFlag,
FinishedGoodsFlag,
Color,
SafetyStockLevel,
ReorderPoint,
StandardCost,
ListPrice,
DaysToManufacture,
SellStartDate,
rowguid,
ModifiedDate)
values
(#ProductName,
#ProductNumber,
#MakeFlag,
#FinishedGoodsFlag,
#Color, #SafetyStockLevel,
#ReorderPoint,
#StandardCost,
#ListPrice,
#DaysToManufacture,
#SellStartDate,
#rowguid,
#ModifiedDate)
Here's the execute query with values of each column:
exec sp_NewProduct 'AR-5516','105',0,1,'Red',5,5,0.00,0.00,5,'2018-05-01',newid(),getdate()

Seems you are dealing with wrong procedure syntax! use following snippets:
Create or Alter proc [dbo].[sp_NewProduct] (#ProductName nvarchar(50), #ProductNumber nvarchar(25), #MakeFlag bit, #FinishedGoodsFlag bit, #Color nvarchar(15), #SafetyStockLevel smallint, #ReorderPoint smallint, #StandardCost money, #ListPrice money, #DaysToManufacture int, #SellStartDate date, #rowguid uniqueidentifier, #ModifiedDate datetime)
As
Begin
Insert dbo.product (Name,ProductNumber,MakeFlag,FinishedGoodsFlag,Color,SafetyStockLevel,ReorderPoint,StandardCost,ListPrice, DaysToManufacture,SellStartDate,rowguid,ModifiedDate) values (#ProductName,#ProductNumber,#MakeFlag,#FinishedGoodsFlag,#Color, #SafetyStockLevel,#ReorderPoint,#StandardCost,#ListPrice,#DaysToManufacture,#SellStartDate,#rowguid,#ModifiedDate)
End
Go
Exec sp_NewProduct 'AR-5516','105',0,1,'Red',5,5,0.00,0.00,5,'2018-05-01',#Id,#DateTime;
If you still getting error, store newid()'s value and GetDate()'s value in temporary variable, and call through this variable!
Declare #Id AS UniqueIdentifier = NewId()
Declare #DateTime as DateTime = GetDate()
Exec sp_NewProduct 'AR-5516','105',0,1,'Red',5,5,0.00,0.00,5,'2018-05-01',#Id,#DateTime;

SQL Server Full text search error

In my stored procedure, when a long word is passed, server shows error, with part of the word.
I am not sure what is happening .. the word seems to break, but I have given a big value for that variable
My procedure:
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
ALTER PROCEDURE [dbo].[SP_BuyOnlineSearchNew]-- '1','BMW X6 IN GOOD CONDITION FOR SALE','0','0'
#CategoryID varchar(20),
#SearchString varchar(max),
#RecentAd int,
#RowsPerPage INT,
#PageNumber INT,
#pricefrom INT,
#priceto INT,
#categories Varchar(2000),
#advancesearch varchar(2000),
#features varchar(2000),
#technicalfeatures varchar(2000),
#CompanyID varchar(20)
AS
BEGIN
SET NOCOUNT ON;
declare #OffetCount as int = (#PageNumber-1)*4
Select TotalCount = COUNT(CM.BuyOnlineID) OVER(),
CM.BuyOnlineID,cm.OfferPrice,cm.[Description], CM.Title,CONVERT(VARCHAR, [CM].[DatePosted],106) AS 'DatePosted',
CM.Price,
CM.isActive,
CM.GoogleLocation,
CM.IsImageVerified,
Row_Number() Over ( Order By CM.BuyOnlineID desc ) As Rownumber,
CM.AreaID,
(select top 1 image from [tblBuyOnlineImages] where BuyOnlineID=CM.BuyOnlineID) As imagepath,
(select count(*) from [tblBuyOnlineImages] where BuyOnlineID=CM.BuyOnlineID) As imagecount,
CM.categorytreename
FROM [dbo].[tblBuyOnlineMaster] AS CM
WHERE
(CONTAINS(CM.CategoryTreename, #categories))
order by CM.BuyOnlineID OFFSET #OffetCount ROWS FETCH NEXT 4 ROWS ONLY
SET NOCOUNT OFF
END

sql server function does not allow return statement

I am trying to create a function in sql server but it keeps telling me :
A RETURN statement with a return value cannot be used in this context.
I have many other functions and they all end with return #variable.
So I must be doing something wrong here and I just can't see it.
Hopefully someone can spot what I am doing wrong here so I can get the function compiling and can complete it.
CREATE function fnGetToInvoiceLoads (#TruckID int, #ClientID int, #OpdrachtID int, #OpcID int, #VertrekIDs varchar(max), #BestemmingIDs varchar(max), #CarIDs varchar(max))
returns table
as
begin
declare #Table table( RitChecked bit,
RitID int,
ClientID int
)
return #Table
end

Take a look at this example... You still need to add your logic to it. But it should give you a hint.
CREATE FUNCTION dbo.fnGetToInvoiceLoads (#TruckID int, #ClientID int, #OpdrachtID int, #OpcID int, #VertrekIDs varchar(max), #BestemmingIDs varchar(max), #CarIDs varchar(max))
RETURNS #result
TABLE (RitChecked bit, RitID int, ClientID int)
AS
BEGIN
-- Do your stuff....
INSERT #result(RitChecked, RitID, ClientID)
SELECT ....
RETURN
END
GO

Try this:
CREATE function fnGetToInvoiceLoads (#TruckID int, #ClientID int, #OpdrachtID int, #OpcID int, #VertrekIDs varchar(max), #BestemmingIDs varchar(max), #CarIDs varchar(max))
RETURNS #Table table --define #table here
(
RitChecked bit,
RitID int,
ClientID int
)
AS
BEGIN
INSERT INTO #Table
SELECT <some data>
RETURN
END

Error Msg 213: Column name or number of supplied values does not match table definition

I created a table with a stored procedure and I'm trying to insert data into it, but I'm getting this error
Msg 137, Level 15, State 2, Line 49
Must declare the scalar variable "#EmployeeID")
Please help!
Create Table Humanresource.Employee
(
EmployeeID char(3) constraint pkemployeeid primary key,
FirstName varchar(20) not null,
LastName varchar(20) not null,
Emp_Address varchar(30) not null,
Title char(30) constraint ckTitle check(Title in ('Executive','Senior Executive','Management Trainee','Event Manager','Senior Event Manager')),
Phone varchar(20) check(Phone like '[0-9][0-9]-[0-9][0-9][0-9][0-9]-[0-9][0-9][0-9]-[0-9][0-9][0-9]')
)
Stored procedure:
create procedure Auto_EmpID
#FirstName varchar(20),
#LastName varchar(20),
#Address varchar(30),
#Title varchar(30),
#Phone varchar(20)
AS
Declare #EmployeeID char(3)
if exists (select EmployeeID From HumanResource.Employee Where EmployeeID = 'E101')
BEGIN
Select
#EmployeeID = Max(right(EmployeeID, 2))
from HumanResource.Employee
Select
#EmployeeID = CASE
When #EmployeeID >= 0 AND #Employeeid < 9
Then 'E0' + convert(char(3), #EmployeeID + 1)
When #EmployeeID >= 9 AND #EmployeeID < 99
Then 'E' + convert(char(3), #EmployeeID + 1)
END
END
Else
set #Employeeid = 'E101'
insert into HumanResource.Employee
Values(#EmployeeID, #FirstName, #LastName, #Address, #Title, #Phone)
Return
Exec Auto_EmpID 'Bamidele','Ajose','Lagos','11-1111-111-111'
exec Auto_Empid 'Kunle','Awele','Ikeja','22-3332-655-897'
exec auto_empid 'John','George','Benin','33-5555-7654-443'
select * from humanresource.employee
insert into humanresource.employee(#EmployeeID, #FirstName, #LastName, #Address, #Title, #Phone)
values(#EmployeeID = 'e101', 'Baley', 'Carles', 'Los Angeles', '23-3445-434-344')

These 2 lines have a lot of problems:
insert into humanresource.employee(#EmployeeID,#FirstName,#LastName,#Address,#Title,#Phone)
values(#EmployeeID='e101','Baley','Carles','Los Angeles','23-3445-434-344')
You don't need to prefix the # symbol to column names. That should only be used with variables.
The column list contains 6 columns, but you are only supplying 5 values in the values clause. Please add the 6th value or remove the column where you don't want to insert.
If you want to use the value 'e101', you can directly insert that. If you want to use the variable #employeeID, just pass that. The assignment #employeeID = 'e101' is wrong inside the values clause. You could just do set #employeeID = 'e101 to assign the value before the insert statement.
#employeeID is declared as char(3), so even if you wanted to assign the value 'e101' to it before the insert statement, the value would get truncated to 'e10'. So, you must declare it as char(4) at least.
Another thing is that your stored procedure takes 5 input parameters, but you pass only 4 when calling it. If you want to pass only some parameters instead of all, you need to specify default values for each parameter while creating the procedure, something like this:
create procedure Auto_EmpID
#FirstName varchar(20) = 'somefirstname',
#LastName varchar(20) = 'somelastname',
#Address varchar(30) = 'somecity',
#Title varchar(30) = 'sometitle',
#Phone varchar(20) = '111-111-111'

Write a faster stored procedure in SQL Server?

I need more than 500000 rows that are stored in several tables. Getting all rows with my logic.
In this situation I write a stored procedure that is successfully executed in SQL Server browser, but when I run it from my C# code, a timeout exception occurred.
Can anyone suggest how I can improve my stored procedure?
CREATE PROCEDURE [dbo].[SP_ExtendedPTU]
(
#SQLForPTU as varchar(MAX)
)
AS
BEGIN TRAN
--DECLARE
--#SQLForPTU as varchar(MAX)
--SET #SQLForPTU= 'WHERE OrderID IN(4233)And OrderType =3'
BEGIN--1
DECLARE
#ProductionTracingUnitID as int,
#OrderID as int,
#OrderType as smallint,
#ProductID as int,
#LabLabDipID as int,
#ColorName as varchar(255),
#PantonNo as varchar(127),
#Shade as varchar(4),
#DyeingOrderQty as decimal(30, 17),
#JobOrderQty as decimal(30, 17),
#ProductionPipeLineQty as decimal(30, 17),
#ProductionFinishedQty as decimal(30, 17),
#DeliveryQty as decimal(30, 17),
#BuyerID as int,
#FactoryID as int,
#ProductionGraceQty as decimal(30, 17),
#WeightLossGainQty as decimal(30, 17),
#RateInLBS as decimal(30, 17),
#State as smallint,
#ProductionLossQty as decimal(30, 17),
#ReturnQty as decimal(30, 17),
#ActualDeliveryQty as decimal(30,17),
#ReadyStockInhand as decimal(30, 17),
#JobOrderQtyApproved as decimal(30, 17),
#OrderNumber as varchar(50),
#FactoryName as varchar(200),
#BuyerName as varchar(200),
#ProductName as varchar(200),
#YetToDelivery as decimal(30,17),
#StockInHand as decimal(30,17),
#YetToProduction as decimal(30,17),
#LCID as int,
#LCNo as varchar(300),
#PIQty as decimal(30,17),
#ChangingQty as decimal(30,17),
#SampleAdjQty as decimal(30,17),
#SampleAdjValue as decimal(30,17),
#MKTPersonID as int,
#MKTPersonName as varchar(500),
#MerchandiserID as int,
#MerchandiserName as varchar(500),
#AmendmentStatus as smallint,
#AcceptanceValue as decimal(30,17),
#MaturityValue as decimal(30,17),
#BillAcceptanceValue as decimal(30,17),
#BillMaturityValue as decimal(30,17),
#BillAcceptancePercentage as decimal(30,17),
#BillMaturityPercentage as decimal(30,17),
#ExportLCValue as decimal(30,17),
#Acceptance as varchar(100),
#Maturity as varchar(100),
#YarnCount as varchar(50),
#PTUDQty as decimal(30,17),
#ShadeFromOrder as smallint,
#EWYDLRelabNo as varchar(100),
#EWYDLColorNo as varchar(100),
#DeliveryTo as int,
#FactoryPersonnelID as int,
#BuyerPersonnelID as int,
#OrderRcvBy as int,
#OrderState as smallint
CREATE TABLE #TempTableOne(
ProductionTracingUnitID int,
OrderID int,
OrderType smallint,
ProductID int,
DyeingOrderQty decimal(30,17),
JobOrderQty decimal(30,17),
ProductionPipeLineQty decimal(30,17),
ProductionFinishedQty decimal(30,17),
DeliveryQty decimal(30,17),
BuyerID int,
FactoryID int,
ProductionGraceQty decimal(30,17),
WeightLossGainQty decimal(30,17),
RateInLBS decimal(30,17),
ProductionLossQty decimal(30,17),
ActualDeliveryQty decimal(30,17),
ReadyStockInhand decimal(30,17),
OrderNumber varchar(50),
FactoryName varchar(200),
BuyerName varchar(200),
ProductName varchar(200),
ColorName varchar(200),
LabLabDipID int,
ReturnQty decimal(30,17),
YetToDelivery decimal(30,17),
StockInHand decimal(30,17),
YetToProduction decimal(30,17),
LCID int,
LCNo varchar(300),
PIQty decimal(30,17),
ChangingQty decimal(30,17),
SampleAdjQty decimal(30,17),
SampleAdjValue decimal(30,17),
MKTPersonID int,
MKTPersonName varchar(500),
MerchandiserID int,
MerchandiserName varchar(500),
AmendmentStatus smallint,
Acceptance varchar(100),
Maturity varchar(100),
YarnCount varchar(50),
EWYDLRelabNo varchar(50),
EWYDLColorNo varchar(50),
ShadeFromOrder smallint
)
--ProductionTracingUnitID,OrderID,OrderType,ProductID,LabLabDipID,ColorName,PantonNo,EWYDLColorNo,Shade,EWYDLRelabNo,DyeingOrderQty,JobOrderQty,ProductionPipeLineQty,ProductionFinishedQty,DeliveryQty,BuyerID,FactoryID,ProductionGraceQty,WeightLossGainQty,RateInLBS,State,ProductionLossQty,ShadeFromOrder,ReturnQty,ActualDeliveryQty,ReadyStockInhand,JobOrderQtyApproved
DECLARE
#SQL as varchar(MAX)
SET #SQL=
'
DECLARE Cur_AB1 CURSOR GLOBAL FORWARD_ONLY KEYSET FOR
SELECT ProductionTracingUnitID,OrderID,OrderType,ProductID,DyeingOrderQty,JobOrderQty,ProductionPipeLineQty,ProductionFinishedQty,BuyerID,FactoryID,ProductionGraceQty,WeightLossGainQty,RateInLBS,ProductionLossQty,ActualDeliveryQty,ReadyStockInhand,ColorName,LabLabDipID,ReturnQty,EWYDLRelabNo,EWYDLColorNo,ShadeFromOrder FROM ProductionTracingUnit '+#SQLForPTU+'
'
EXEC (#SQL)
OPEN Cur_AB1
FETCH NEXT FROM Cur_AB1 INTO #ProductionTracingUnitID,#OrderID,#OrderType,#ProductID,#DyeingOrderQty,#JobOrderQty,#ProductionPipeLineQty,#ProductionFinishedQty,#BuyerID,#FactoryID,#ProductionGraceQty,#WeightLossGainQty,#RateInLBS,#ProductionLossQty,#ActualDeliveryQty,#ReadyStockInhand,#ColorName,#LabLabDipID,#ReturnQty,#EWYDLRelabNo,#EWYDLColorNo,#ShadeFromOrder
WHILE(##Fetch_Status <> -1)
BEGIN--2
SET #LCID=0
SET #LCNo=''
SET #PIQty=0
SET #AcceptanceValue =0
SET #MaturityValue= 0
SET #Acceptance= ''
SET #Maturity =''
SET #DeliveryQty=#ActualDeliveryQty-#ReturnQty
SET #YetToDelivery=#JobOrderQty-#ActualDeliveryQty+#ReturnQty
set #PTUDQty=(select sum(Qty) from PTUDistribution where ProductionTracingUnitID=#ProductionTracingUnitID )
IF(#PTUDQty>#YetToDelivery)
BEGIN--sih
SET #StockInHand =#YetToDelivery
END --sih
ELSE
BEGIN--sih2
SET #StockInHand =#PTUDQty
END --sih2
SET #YetToProduction=#JobOrderQty-#ReadyStockInhand-#ActualDeliveryQty+#ReturnQty
IF (#YetToProduction<0)
BEGIN
SET #YetToProduction=0
END
SET #ChangingQty=0
SET #SampleAdjQty=0
SET #SampleAdjValue=0
SET #MerchandiserID=0
SET #MerchandiserName=''
SET #MKTPersonID =0
SET #AmendmentStatus=0
SET #AcceptanceValue =0
SET #MaturityValue= 0
SET #Acceptance= ''
SET #Maturity =''
SET #MKTPersonName =''
SET #OrderNumber=''
IF(#OrderType=3)
BEGIN--jam1
SET #OrderNumber=(SELECT ISNULL(JobCode,'')+' - '+ISNULL(JobNo,'')+' / '+ISNULL(JobYear,'') FROM Job WHERE JobID=#OrderID)
SELECT #LCID=ISNULL(ExportLCID,0), #LCNo=ISNULL(ExportLCNo,''),#AmendmentStatus=AmendmentStatus,#ExportLCValue=Amount FROM ExportLC WHERE ExportLCID =(SELECT LCID FROM [PI] WHERE PIID=(SELECT PIID FROM Job WHERE JobID=#OrderID))
SELECT #PIQty=ISNULL(SUM(Qty),0), #SampleAdjQty=ISNULL(SUM(AdjQty),0), #SampleAdjValue=ISNULL(SUM(AdjValue),0) FROM PIProducts WHERE PIID=(SELECT PIID FROM Job WHERE JobID=#OrderID) AND ProductID=#ProductID
SET #ChangingQty=(SELECT ISNULL(SUM(Qty),0) FROM PIDeliverableProducts WHERE PIID=(SELECT PIID FROM Job WHERE JobID=#OrderID) AND ProductID=#ProductID)
SELECT #MKTPersonID=EWYDLMarketingEmpID, #MerchandiserID=CmsBCPID FROM [PI] WHERE PIID=(SELECT PIID FROM Job WHERE JobID=#OrderID)
SET #MerchandiserName=(SELECT ISNULL([Name],'') FROM ContactPersonnel WHERE ContactPersonnelID=#MerchandiserID)
SET #MKTPersonName =(SELECT [Name] FROM Employee WHERE EmployeeID=#MKTPersonID)
SET #BillAcceptanceValue=(select isnull(sum(Amount),0) from LCbill where EXportLCID=#LCID and [state] in (2,3,4))
SET #BillMaturityValue =(select isnull(sum(Amount),0) from LCbill where EXportLCID=#LCID and [state] in (5,6,7,8,9,10,12))
IF(#ExportLCValue>0 and #ExportLCValue is not null)
BEGIN
SET #BillAcceptancePercentage =(#BillAcceptanceValue*100)/#ExportLCValue -- bill Percentage
SET #BillMaturityPercentage =(#BillMaturityValue*100)/#ExportLCValue
SET #AcceptanceValue=(#ChangingQty*#RateInLBS)*(#BillAcceptancePercentage/100)--Percentage Wise PI Valu
SET #MaturityValue=(#ChangingQty*#RateInLBS)*(#BillMaturityPercentage/100)
IF((#ChangingQty*#RateInLBS)>0 and (#ChangingQty*#RateInLBS) is not null)
BEGIN
SET #AcceptanceValue=(#AcceptanceValue*100)/(#ChangingQty*#RateInLBS)-- PI ValuePercentage
SET #MaturityValue=(#MaturityValue*100)/(#ChangingQty*#RateInLBS)
END
SET #Acceptance=Convert(varchar(20),(CONVERT(float,round((#AcceptanceValue+#MaturityValue),0)))) +'%'
SET #Maturity =Convert(varchar(20),(CONVERT(float,round(#MaturityValue,0)))) +'%'
END
SET #FactoryName=''
IF(#FactoryID>0)
BEGIN--jam3
SET #FactoryName=(SELECT [Name] FROM Contractor WHERE ContractorID=#FactoryID)
END--jam3
SET #BuyerName=''
IF(#BuyerID>0)
BEGIN--jam4
SET #BuyerName=(SELECT [Name] FROM Contractor WHERE ContractorID=#BuyerID)
END--jam4
END--jam1
ELSE
BEGIN --jam2 IF Sample
SET #LCID=0
SET #LCNo=''
SET #OrderNumber=''
SET #PIQty=0
SET #DeliveryTo=0
SET #FactoryPersonnelID=0
SET #BuyerPersonnelID=0
SET #OrderRcvBy=0
SET #ChangingQty=0
SET #MerchandiserName=''
SET #MKTPersonName=''
SET #OrderState=0
SET #AmendmentStatus=0
SET #PIQty= (SELECT ISNULL(SUM(Qty),0) FROM SampleOrderDetail WHERE PTUID=#ProductionTracingUnitID)
SET #ChangingQty=#PIQty
SELECT #OrderNumber=ISNULL(SampleOrderNo,''), #DeliveryTo=ISNULL(DeliveryTo,0),#FactoryPersonnelID=ISNULL(FactoryPersonnelID,0),#BuyerPersonnelID=ISNULL(BuyerPersonnelID,0),#OrderRcvBy=ISNULL(OrderRcvBy,0),#OrderState=ISNULL(OrderState,0) FROM SampleOrder WHERE SampleOrderID=#OrderID
SET #AmendmentStatus=#OrderState
IF(#DeliveryTo=3)
BEGIN
SET #MerchandiserName=(SELECT ISNULL([Name],'') FROM ContactPersonnel WHERE ContactPersonnelID=#FactoryPersonnelID)
END
IF(#DeliveryTo=2)
BEGIN
SET #MerchandiserName=(SELECT ISNULL([Name],'') FROM ContactPersonnel WHERE ContactPersonnelID=#BuyerPersonnelID)
END
SET #MKTPersonName =(SELECT [Name] FROM Employee WHERE EmployeeID=#OrderRcvBy)
IF(#DeliveryTo=3)
BEGIN
SET #FactoryName=(SELECT [Name] FROM Contractor WHERE ContractorID=#FactoryID)
END
IF(#DeliveryTo=2)
BEGIN
SET #FactoryName=(SELECT [Name] FROM Contractor WHERE ContractorID=#BuyerID)
END
SET #BuyerName=''
IF(#BuyerID>0)
BEGIN--jam4
SET #BuyerName=(SELECT [Name] FROM Contractor WHERE ContractorID=#BuyerID)
END--jam4
END--jam2
SET #ProductName=''
SET #YarnCount =''
SELECT #ProductName=('['+ Code+ '] '+ [Name]), #YarnCount =[Count] FROM Yarncategory WHERE YarncategoryID=#ProductID
INSERT INTO #TempTableOne Values(ISNULL(#ProductionTracingUnitID,0),ISNULL(#OrderID,0),ISNULL(#OrderType,0),ISNULL(#ProductID,0),ISNULL(#DyeingOrderQty,0),ISNULL(#JobOrderQty,0),ISNULL(#ProductionPipeLineQty,0),ISNULL(#ProductionFinishedQty,0),ISNULL(#DeliveryQty,0),ISNULL(#BuyerID,0),ISNULL(#FactoryID,0),ISNULL(#ProductionGraceQty,0),ISNULL(#WeightLossGainQty,0),ISNULL(CONVERT (decimal(18,2),#RateInLBS),0),ISNULL(#ProductionLossQty,0),ISNULL(#ActualDeliveryQty,0),ISNULL(#ReadyStockInhand,0),ISNULL(#OrderNumber,''),ISNULL(#FactoryName,''),ISNULL(#BuyerName,''),ISNULL(#ProductName,''),ISNULL(#ColorName,''),ISNULL(#LabLabDipID,0),ISNULL(#ReturnQty,0),ISNULL(#YetToDelivery,0),ISNULL(#StockInHand,0),ISNULL(#YetToProduction,0),ISNULL(#LCID,0),ISNULL(#LCNo,''),ISNULL(#PIQty,0),ISNULL(#ChangingQty,0),ISNULL(#SampleAdjQty,0),ISNULL(#SampleAdjValue,0),ISNULL(#MKTPersonID,0),ISNULL(#MKTPersonName,''),ISNULL(#MerchandiserID,0),ISNULL(#MerchandiserName,''),ISNULL(#AmendmentStatus,0),ISNULL(#Acceptance,''),ISNULL(#Maturity,''),ISNULL(#YarnCount,''),ISNULL(#EWYDLRelabNo,''),ISNULL(#EWYDLColorNo,''),ISNULL(#ShadeFromOrder,0))
FETCH NEXT FROM Cur_AB1 INTO #ProductionTracingUnitID,#OrderID,#OrderType,#ProductID,#DyeingOrderQty,#JobOrderQty,#ProductionPipeLineQty,#ProductionFinishedQty,#BuyerID,#FactoryID,#ProductionGraceQty,#WeightLossGainQty,#RateInLBS,#ProductionLossQty,#ActualDeliveryQty,#ReadyStockInhand,#ColorName,#LabLabDipID,#ReturnQty,#EWYDLRelabNo,#EWYDLColorNo,#ShadeFromOrder
END--2
CLOSE Cur_AB1
DEALLOCATE Cur_AB1
SELECT * FROM #TempTableOne Order By OrderID
--Group By Product
SELECT ProductID,ProductName, YarnCount, SUM(PIQty) as PIQty, SUM(ChangingQty) AS ChangingQty, SUM(SampleAdjQty) AS SampleAdjQty, SUM(SampleAdjValue) as SampleAdjValue, SUM(DyeingOrderQty) as DyeingOrderQty,SUM(JobOrderQty)AS JobOrderQty,SUM(ProductionPipeLineQty)as ProductionPipeLineQty,SUM(ProductionFinishedQty) as ProductionFinishedQty,SUM(DeliveryQty)as DeliveryQty,SUM(ProductionGraceQty)AS ProductionGraceQty,SUM(WeightLossGainQty) as WeightLossGainQty,SUM(ProductionLossQty)as ProductionLossQty,SUM(ActualDeliveryQty)as ActualDeliveryQty,SUM(ReadyStockInhand)as ReadyStockInhand, SUM(ReturnQty) AS ReturnQty,SUM(YetToDelivery)AS YetToDelivery,SUM(StockInHand)AS StockInHand,SUM(YetToProduction)AS YetToProduction FROM #TempTableOne GROUP BY ProductID,ProductName,YarnCount Order By ProductID
--Group By Factory
SELECT FactoryID,FactoryName,SUM(PIQty) as PIQty, SUM(ChangingQty) AS ChangingQty, SUM(SampleAdjQty) AS SampleAdjQty, SUM(SampleAdjValue) as SampleAdjValue, SUM(DyeingOrderQty) as DyeingOrderQty,SUM(JobOrderQty)AS JobOrderQty,SUM(ProductionPipeLineQty)as ProductionPipeLineQty,SUM(ProductionFinishedQty) as ProductionFinishedQty,SUM(DeliveryQty)as DeliveryQty,SUM(ProductionGraceQty)AS ProductionGraceQty,SUM(WeightLossGainQty) as WeightLossGainQty,SUM(ProductionLossQty)as ProductionLossQty,SUM(ActualDeliveryQty)as ActualDeliveryQty,SUM(ReadyStockInhand)as ReadyStockInhand, SUM(ReturnQty) AS ReturnQty,SUM(YetToDelivery)AS YetToDelivery,SUM(StockInHand)AS StockInHand,SUM(YetToProduction)AS YetToProduction FROM #TempTableOne GROUP BY FactoryID,FactoryName
--Group By Order
SELECT OrderID,OrderNumber,LCNo,FactoryID,FactoryName,BuyerID,BuyerName,SUM(PIQty) AS PIQty,SUM(ChangingQty) AS ChangingQty,SUM(SampleAdjQty) AS SampleAdjQty,SUM(SampleAdjValue) AS SampleAdjValue,MKTPersonID,MKTPersonName,MerchandiserID,MerchandiserName,AmendmentStatus,Acceptance,Maturity,SUM(DyeingOrderQty) as DyeingOrderQty,SUM(JobOrderQty)AS JobOrderQty,SUM(ProductionPipeLineQty)as ProductionPipeLineQty,SUM(ProductionFinishedQty) as ProductionFinishedQty,SUM(DeliveryQty)as DeliveryQty,SUM(ProductionGraceQty)AS ProductionGraceQty,SUM(WeightLossGainQty) as WeightLossGainQty,SUM(ProductionLossQty)as ProductionLossQty,SUM(ActualDeliveryQty)as ActualDeliveryQty,SUM(ReadyStockInhand)as ReadyStockInhand, SUM(ReturnQty) AS ReturnQty,SUM(YetToDelivery)AS YetToDelivery,SUM(StockInHand)AS StockInHand,SUM(YetToProduction)AS YetToProduction FROM #TempTableOne GROUP BY OrderID,OrderNumber,OrderNumber,LCNo,FactoryID,FactoryName,BuyerID,BuyerName,MKTPersonID,MKTPersonName,MerchandiserID,MerchandiserName,AmendmentStatus,Acceptance,Maturity
--Default View
SELECT OrderID,OrderNumber,LCNo,FactoryID,FactoryName,BuyerID,BuyerName,ProductID,ProductName,YarnCount,PIQty,ChangingQty,SampleAdjQty,SampleAdjValue,RateInLBS,MKTPersonID,MKTPersonName,MerchandiserID,MerchandiserName,AmendmentStatus,Acceptance,Maturity, SUM(DyeingOrderQty) as DyeingOrderQty,SUM(JobOrderQty)AS JobOrderQty,SUM(ProductionPipeLineQty)as ProductionPipeLineQty,SUM(ProductionFinishedQty) as ProductionFinishedQty,SUM(DeliveryQty)as DeliveryQty,SUM(ProductionGraceQty)AS ProductionGraceQty,SUM(WeightLossGainQty) as WeightLossGainQty,SUM(ProductionLossQty)as ProductionLossQty,SUM(ActualDeliveryQty)as ActualDeliveryQty,SUM(ReadyStockInhand)as ReadyStockInhand, SUM(ReturnQty) AS ReturnQty,SUM(YetToDelivery)AS YetToDelivery,SUM(StockInHand)AS StockInHand,SUM(YetToProduction)AS YetToProduction FROM #TempTableOne GROUP BY OrderID,OrderNumber,LCNo,FactoryID,FactoryName,BuyerID,BuyerName,ProductID,ProductName,YarnCount,PIQty,ChangingQty,SampleAdjQty,SampleAdjValue,RateInLBS,MKTPersonID,MKTPersonName,MerchandiserID,MerchandiserName,AmendmentStatus,Acceptance,Maturity
DROP TABLE #TempTableOne
END--1
COMMIT TRAN

There are a host of issues here. Properly written, this procedure should run comfortably in about 15 seconds, depending on your indexes. The tips I can give you before I delve into detail are as follows:
First up, the only time that it is acceptable to use cursors when doing bulk processing is for partitioning updates in the bulk process. In other words, never use a cursor that needs to iterate for every record to be updated. The entire procedure needs to iterate 500,000 times, and that is going to be very slow. Rather use a temp table to insert the data into, and then use that table to perform updates on. It is possible to index a temporary table with temporary indexes, as shown in the code below. Make sure that any fields used in a WHERE clause is indexed, both for your temp tables and your live tables.
CREATE TABLE #Temp2 (
ProductionTracingUnitID int,
OrderID int,
OrderType smallint,
ProductID int,
DyeingOrderQty decimal(30, 17),
JobOrderQty decimal(30, 17),
ProductionPipeLineQty decimal(30, 17),
ProductionFinishedQty decimal(30, 17),
BuyerID int,
FactoryID int,
ProductionGraceQty decimal(30, 17),
WeightLossGainQty decimal(30, 17),
RateInLBS decimal(30, 17),
ProductionLossQty decimal(30, 17),
ActualDeliveryQty decimal(30,17),
ReadyStockInhand decimal(30, 17),
ColorName varchar(255),
LabLabDipID int,
ReturnQty decimal(30, 17),
EWYDLRelabNo varchar(100),
EWYDLColorNo varchar(100),
ShadeFromOrder smallint
)
CREATE NONCLUSTERED INDEX #IX_Temp2_1 ON #Temp2(ProductionTracingUnitID)
CREATE NONCLUSTERED INDEX #IX_Temp2_2 ON #Temp2(OrderID)
CREATE NONCLUSTERED INDEX #IX_Temp2_3 ON #Temp2(ProductID)
DECLARE #SQL as varchar(MAX)
SET #SQL=
'
SELECT ProductionTracingUnitID,OrderID,OrderType,ProductID,DyeingOrderQty,JobOrderQty,ProductionPipeLineQty,ProductionFinishedQty,BuyerID,FactoryID,ProductionGraceQty,
WeightLossGainQty,RateInLBS,ProductionLossQty,ActualDeliveryQty,ReadyStockInhand,ColorName,LabLabDipID,ReturnQty,EWYDLRelabNo,EWYDLColorNo,ShadeFromOrder
FROM ProductionTracingUnit '+#SQLForPTU
INSERT INTO #Temp2 (ProductionTracingUnitID, OrderID, OrderType, ProductID, DyeingOrderQty, JobOrderQty, ProductionPipeLineQty,
ProductionFinishedQty, BuyerID, FactoryID, ProductionGraceQty, WeightLossGainQty, RateInLBS, ProductionLossQty, ActualDeliveryQty,
ReadyStockInhand, ColorName, LabLabDipID, ReturnQty, EWYDLRelabNo, EWYDLColorNo, ShadeFromOrder)
EXEC (#SQL)
Next, when doing bulk processing, avoid the use of sub-queries that need to be executed per row. The following line would be very slow, at the best of times:
SELECT #LCID=ISNULL(ExportLCID,0), #LCNo=ISNULL(ExportLCNo,''),#AmendmentStatus=AmendmentStatus,#ExportLCValue=Amount
FROM ExportLC WHERE ExportLCID =(SELECT LCID FROM [PI] WHERE PIID=(SELECT PIID FROM Job WHERE JobID=#OrderID))
Consider rephrasing with joins, like this:
SELECT #LCID=ISNULL(ExportLCID,0), #LCNo=ISNULL(ExportLCNo,''),#AmendmentStatus=AmendmentStatus,#ExportLCValue=Amount
FROM ExportLC e, [PI] p, Job j
WHERE e.ExportLCID = p.LCID
AND p.PIID = j.PIID
AND j.JobID = #OrderID
Rewriting the procedure without the use of a cursor, this would look more along the lines of:
UPDATE #Temp2 SET LCID = ISNULL(ExportLCID, 0), LCNo = ISNULL(ExportLCNo, 0), ...
FROM #Temp2 t, ExportLC e, [PI] p, Job j
WHERE e.ExportLCID = p.LCID
AND p.PIID = j.PIID
AND j.JobID = t.OrderID
There is quite a bit of work to get this all working, and it is difficult for me to create a test environment for a such a compound process. However, I think this should give you enough to go on. Let me know if you get stuck.

You can use the execution plan in SQL Server to identify the bottlenecks.
Also please try to avoid using cursors if possible and also if the temp table contains huge amount of data, create a normal table instead of creating a #table

If you are parsing 500,000 rows a cursor is a non starter. This will cause all sorts of problems and as your data grows, it will become slower and slower. There are several ways to achieve better performance.
1) If possible, fetch data overnight - i.e. flattening the data into a snapshot table using a SQL job
2) Fetch as much data as possible in a batch select - i.e. rather than inserting into your temp table row by row in your loop, you can do a query to get all the data you need for a particular column
3) If you cannot do this overnight and it must always be done like this, I would suggest breaking this up into discreet Stored Procedures and looping from within your C# code. That way you can provide feedback to your user - perhaps in the form of a progress bar. This won't timeout, and it won't really matter how long it takes provided the user is aware something is being calculated, they get a rough idea of how long they have to wait, and they have a chance to back out if they want to.

Resources