133

I am trying to delete from a few tables at once. I've done a bit of research, and came up with this

DELETE FROM `pets` p,
            `pets_activities` pa
      WHERE p.`order` > :order
        AND p.`pet_id` = :pet_id
        AND pa.`id` = p.`pet_id`

However, I am getting this error

Uncaught Database_Exception [ 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 'p, pets_activities pa...

I've never done a cross table delete before, so I'm inexperienced and stuck for now!

What am I doing wrong?

Sam
  • 7,252
  • 16
  • 46
  • 65
alex
  • 479,566
  • 201
  • 878
  • 984

7 Answers7

240

Use a JOIN in the DELETE statement.

DELETE p, pa
      FROM pets p
      JOIN pets_activities pa ON pa.id = p.pet_id
     WHERE p.order > :order
       AND p.pet_id = :pet_id

Alternatively you can use...

DELETE pa
      FROM pets_activities pa
      JOIN pets p ON pa.id = p.pet_id
 WHERE p.order > :order
   AND p.pet_id = :pet_id

...to delete only from pets_activities

See this.

For single table deletes, yet with referential integrity, there are other ways of doing with EXISTS, NOT EXISTS, IN, NOT IN and etc. But the one above where you specify from which tables to delete with an alias before the FROM clause can get you out of a few pretty tight spots more easily. I tend to reach out to an EXISTS in 99% of the cases and then there is the 1% where this MySQL syntax takes the day.

Nexaspx
  • 371
  • 4
  • 20
cadman
  • 2,476
  • 3
  • 16
  • 7
  • 7
    I tried this "delete all in 1 query" with joining 6 large tables (everyone about ~15k rows) and the query took 155 seconds to delete 63 rows in 6 tables :O – Klemen Tusar Jul 30 '12 at 21:29
  • 1
    @cadman This is the real right answer; there may be arguments against using it, but it's very useful on occasion – Simon Christian Mar 06 '13 at 12:14
  • 1
    +1 I agree that this in the real right answer, since the question was not "should you" but "how to". However, I would be interested in hearing about the 1% because I can't think of a single situation where this would be preferred. – Erick Robertson Jan 09 '14 at 02:47
  • 2
    @techouse, did you join and filter on indices? 15k x 15k x 15k x 15k 15k x 15k is 11 million. Did a `SELECT` take similarly long? – Paul Draper May 22 '14 at 20:07
  • 7
    You can also use LEFT JOIN, which is usefull if the second table had no matching entries, else nothing will be deleted. – Lexib0y Nov 13 '15 at 12:02
  • Thanks, but I met another question when execute `delete` in `safe-update` mode, please see http://stackoverflow.com/questions/37892657/how-to-delete-the-latest-some-records-in-safe-updates-mode – zhuguowei Jun 18 '16 at 02:23
21

Since this appears to be a simple parent/child relationship between pets and pets_activities, you would be better off creating your foreign key constraint with a deleting cascade.

That way, when a pets row is deleted, the pets_activities rows associated with it are automatically deleted as well.

Then your query becomes a simple:

delete from `pets`
    where `order` > :order
      and `pet_id` = :pet_id
paxdiablo
  • 854,327
  • 234
  • 1,573
  • 1,953
  • Delete cascades are inviting horrible accidents. – Erick Robertson Jul 26 '10 at 03:18
  • Thanks Paxdiablo, I'm not sure how to do this yet in MySQL but I'll consider it. – alex Jul 26 '10 at 03:20
  • 3
    @Erick, provided you've set up referential integrity, cascading deletes can cause no more trouble than delete on its own. We already know that `pa` is a proper child of `p` due to the `id/pet_id` mapping. – paxdiablo Jul 26 '10 at 03:24
  • Cascading deletes just means that things will be deleted which you aren't explicitly saying to delete. That's always dangerous. – Erick Robertson Jul 26 '10 at 03:26
  • 3
    You _are_ explicitly saying you want to delete them. Just not at the time of delete. You don't explicitly say you want triggers fired at the time of insertion either but that doesn't make them dangerous. – paxdiablo Jul 26 '10 at 03:42
  • I disagree. I prefer to use a database as a database, and not as an execution platform. – Erick Robertson Jul 26 '10 at 03:48
  • I agree with Erick on this one - can be hazardous. Somewhat related, SQL Server 2005 doesn't handle cascade deletes if on the same table. The safest means is to process data in relational order – OMG Ponies Jul 26 '10 at 03:50
  • 14
    Well, you guys have your own thoughts but it seems like you're discounting a lot of the power of DBMS'. Cascading deletes are as much a part of data management as triggers, stored procedures or constraints and they're only dangerous if you don't know what you're doing. Still, I won't argue the point further, we'll just have to agree to disagree. – paxdiablo Jul 26 '10 at 04:03
  • 3
    Getting data in isn't the issue - it's situations where you have to reconstruct data that wasn't supposed to be deleted which is a real hassle. Only Oracle has Flashback, a temporal compliment to it's database for inline reconstituting accidentally deleted data. SQL Server has snapshots, but it's nowhere near as slick as Flashback. That's why Erick, myself, and numerous others I've worked with prefer explicit deletion. Like the case with SQL Server, it's not always a cut'n'dried operation either, or hasn't been in the past. – OMG Ponies Jul 26 '10 at 04:25
  • Again, not an issue if it's set up correctly - the problems you mention are no different to deleting rows accidentally _without_ cascade. Seriously, if someone doesn't know how to design and implement databases, they have no business trying to do it. – paxdiablo Jul 26 '10 at 08:32
  • 2
    It probably comes as no surprise, but I also do not use triggers, stored procedures, or constraints. I prefer to put this kind of logic in my execution platform and use my database for storing and retrieving data only. This doesn't mean I don't know how to use or understand these features, however. – Erick Robertson Jul 26 '10 at 11:26
  • 8
    Erick, now you've piqued my interest. How do you ensure data integrity within the database without constraints? – paxdiablo Jul 26 '10 at 23:46
  • 5
    @Erick said, "I also do not use triggers, stored procedures, or constraints." Ah, you use Excel. :-) – james.garriss Aug 24 '13 at 18:17
  • 5
    I just want to follow up on this. I have changed my position on deleting cascades in this situation. I have been a part of a new SQL environment which used them, and used them well and they were very organized. In this system it worked very well to our advantage to have these cascades in place. It certainly prevented orphaned data and was not dangerous. The problem is that everyone working with the database needs to understand how to use them safely. But there are always risks when junior devs are making database changes unsupervised. – Erick Robertson Oct 16 '13 at 17:22
  • One reason not to use cascading deletes in MySQL is that [cascaded deletes don't trigger triggers](https://bugs.mysql.com/bug.php?id=11472). This is also true for [MariaDB](https://jira.mariadb.org/browse/MDEV-19402). – Adrian Wiik Jan 12 '23 at 11:54
  • @AdrianWiik: I would probably consider that a reason to not use MySQL rather than to not use cascading deletes :-) – paxdiablo Jan 13 '23 at 07:26
19

Use this

DELETE FROM `articles`, `comments` 
USING `articles`,`comments` 
WHERE `comments`.`article_id` = `articles`.`id` AND `articles`.`id` = 4

or

DELETE `articles`, `comments` 
FROM `articles`, `comments` 
WHERE `comments`.`article_id` = `articles`.`id` AND `articles`.`id` = 4
RN Kushwaha
  • 2,081
  • 3
  • 29
  • 40
  • 1
    Found a good reference for using this and a few other options at http://www.mysqltutorial.org/mysql-delete-statement.aspx – Mavelo Jan 09 '17 at 15:08
3

I don't have a mysql database to test on at the moment, but have you tried specifying what to delete prior to the from clause? For example:

DELETE p, pa FROM `pets` p,
        `pets_activities` pa
  WHERE p.`order` > :order
    AND p.`pet_id` = :pet_id
    AND pa.`id` = p.`pet_id`

I think the syntax you used is limited to newer versions of mysql.

Brandon Horsley
  • 7,956
  • 1
  • 29
  • 28
  • 1
    That query executed successfully, however, it didn't delete any rows (but I believe it should have). – alex Jul 26 '10 at 04:43
3

To anyone reading this in 2017, this is how I've done something similar.

DELETE pets, pets_activities FROM pets inner join pets_activities
on pets_activities.id = pets.id WHERE pets.`order` > :order AND 
pets.`pet_id` = :pet_id

Generally, to delete rows from multiple tables, the syntax I follow is given below. The solution is based on an assumption that there is some relation between the two tables.

DELETE table1, table2 FROM table1 inner join table2 on table2.id = table1.id
WHERE [conditions]
Kalesh Kaladharan
  • 1,038
  • 1
  • 8
  • 26
2

The syntax looks right to me ... try to change it to use INNER JOIN ...

Have a look at this.

Nexaspx
  • 371
  • 4
  • 20
Andre Gallo
  • 2,261
  • 5
  • 23
  • 46
1

I found this article which showing you how to delete data from multiple tables by using MySQL DELETE JOIN statement with good explanation.

enter image description here

Ravi Makwana
  • 2,782
  • 1
  • 29
  • 41