MYSQL query

Hi,
I constructed the following query :
SELECT ESD.search_type   FROM extreme_search_data ESD where ESD.dep_date < curdate()+30 and ESD.search_type in ('O','R') ,(SELECT @rownum:=0) r INTO OUTFILE 'file2' FIELDS TERMINATED BY ','

This gives an error saying wrong syntax.

But if i remove the where clause It works properly.

Working query :
SELECT ESD.search_type   FROM extreme_search_data ESD,(SELECT @rownum:=0) r INTO OUTFILE 'file2' FIELDS TERMINATED BY ','

How do i combine the where clause into the above working query

Thanks
Rohit BajajAsked:
Who is Participating?
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

Chris StanyonWebDevCommented:
What do you intend to do with the rownum - you seem to be trying to select it you're not doing anything with it. Assuming you want it to be a row counter and output to the file as well, you'll also need to increment it. Have a look at this:

SELECT ESD.search_type, @rownum:=@rownum + 1 row_number
INTO OUTFILE 'file2' FIELDS TERMINATED BY ','
FROM extreme_search_data ESD, (SELECT @rownum:=0) r
WHERE ESD.dep_date < CURDATE()+30 AND ESD.search_type IN ('O','R');

Open in new window

0
Rohit BajajAuthor Commented:
Hi,
The query mentioned by you is giving an error :
Error Code: 1064. You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INTO OUTFILE 'file2' FIELDS TERMINATED BY ','
FROM extreme_search_data ESD, (SEL' at line 4


I am assuming the SELECT @rownum:=0 part is selecting the first row from the output and dumping it into a file.
This was a query i got in a project. Which was working.
But i need to add where clause to some columns.
But when adding where clause it gives syntax error
0
Chris StanyonWebDevCommented:
Hmmm. The query runs fine for me. Double check it matches exactly what I posted.

The @rownum is a generated variable used to put a row number into the file - it has nothing to do with selecting anything. If your WHERE clause selects 10 records then they will be numbered 1,2,3,4 etc. It's just a row counter. I presumed you wanted that because I couldn't think of another reason why you included it in your query.

Something else I've just noticed. You seem to be adding 30 to the current date. That won't add 30 days to it, it will just add 30 to the numerical value of the date, so today plus 30 = 20131135 - 35th November!! You should use the DATE_ADD function in your query:

SELECT ESD.search_type, @rownum:=@rownum + 1 row_number
INTO OUTFILE 'file2' FIELDS TERMINATED BY ','
FROM extreme_search_data ESD, (SELECT @rownum:=0) r
WHERE ESD.dep_date < DATE_ADD(CURDATE(), INTERVAL 30 DAY) AND ESD.search_type IN ('O','R');

Open in new window

If you need to add 1 month, then use INTERVAL 1 MONTH instead
0
Newly released Acronis True Image 2019

In announcing the release of the 15th Anniversary Edition of Acronis True Image 2019, the company revealed that its artificial intelligence-based anti-ransomware technology – stopped more than 200,000 ransomware attacks on 150,000 customers last year.

Rohit BajajAuthor Commented:
Hi,
Thanks for pointing out the month thing.

I just checked your query from an online mySql checker :
http://www.piliapp.com/mysql-syntax-check/

Please see if it gives you also the error on this online syntax checker.
I think i am copying the query correctly
0
Chris StanyonWebDevCommented:
Yeah, the Syntax checker balks, but only the INTO OUTFILE bit - not on the WHERE clause - stilll, it runs fine on my system :)

Remove the INTO line just to check your query is returning what you need. Also, there's no real need to alias your ESD table:

SELECT search_type, @rownum:=@rownum + 1 row_number
FROM extreme_search_data, (SELECT @rownum:=0) r
WHERE dep_date < DATE_ADD(CURDATE(), INTERVAL 1 MONTH)
AND search_type IN ('O','R');

Open in new window

0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
Rohit BajajAuthor Commented:
HI,
The following query works :

SELECT CONCAT(CONCAT(CONCAT(CONCAT(CONCAT(CONCAT(CONCAT(CONCAT(CONCAT(CONCAT(CONCAT(CONCAT
      (if(@rownum=0,'<items>',''),CONCAT('<item>','<sku>')),CONCAT(p.id,'</sku>')),
      CONCAT('<departure_city>',p.origin)),CONCAT('</departure_city>','<destination_city>')),
      CONCAT(p.destination,'</destination_city>')),CONCAT('<travel_start_date>',p.dep_date)),
      CONCAT('T00:00:00+05:30</travel_start_date>','<travel_end_date>')),CONCAT(p.arrival_date,'T00:00:00+05:30</travel_end_date>')),
      CONCAT('<price>',p.price)),
      CONCAT('</price>','<flight_length>')),
      CONCAT(@rownum:=@rownum+1,'</flight_length>')),
      CONCAT('</item>',if(@rownum=(select count(*) from extreme_search_data),'</items>','')))
      FROM extreme_search_data p ,(SELECT @rownum:=0) r where dep_date < DATE_ADD(CURDATE(), INTERVAL "+duration+" DAY) and search_type in ("+searchType+") INTO OUTFILE '"+ filename1 +"' FIELDS TERMINATED BY ','
0
Rohit BajajAuthor Commented:
I've requested that this question be closed as follows:

Accepted answer: 0 points for Robinsuri's comment #a39658522

for the following reason:

I found it myself
0
Chris StanyonWebDevCommented:
Seriously! That's the query you're going with. How does that query represent anything like you originally asked for.

Why use 4 lines of code when you can use 20.

Good luck maintaining that one.
0
Rohit BajajAuthor Commented:
SELECT ESD.search_type   FROM extreme_search_data ESD, (SELECT @rownum:=0) r where ESD.dep_date < curdate()+30 and ESD.search_type in ('O','R')  INTO OUTFILE 'file2' FIELDS TERMINATED BY ','


The above is the exact query i meant. I posted a bigger query mistakenly.
This worked on my machine.
0
Chris StanyonWebDevCommented:
Your query doesn't really make sense. From the original, you've just moved the (SELECT @rownum:=0) - which does absolutely nothing. What exactly are you expecting that to do!

If you don't need it, then just drop it.

Your query is also selecting dates less than 50th November!!!

Please have a read of this - http://support.experts-exchange.com/customer/portal/articles/481419 - and then explain the Grade C

Thanks
0
Rohit BajajAuthor Commented:
HI,
Having look at the link given by you.
Thx on the 50th November part. I agree with it.

This query is actually an already written query in a project.
and i was given to modify it.
will update shortly
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
MySQL Server

From novice to tech pro — start learning today.