DDC-300: RunSqlTask needs the ability to parse an SQL file #374

Closed
opened 2026-01-22 12:36:21 +01:00 by admin · 7 comments
Owner

Originally created by @doctrinebot on GitHub (Feb 3, 2010).

Originally assigned to: @guilhermeblanco on GitHub.

Jira issue originally created by user brandon.parise:

I made the following assumptions with the patch...

  • Well-formed SQL file with 1 statement per line.
  • No empty lines or comments.
  • Will not continue processing SQL if an error occurs.

Should I assume differently? Or are there more assumptions I should consider?

Originally created by @doctrinebot on GitHub (Feb 3, 2010). Originally assigned to: @guilhermeblanco on GitHub. Jira issue originally created by user brandon.parise: I made the following assumptions with the patch... - Well-formed SQL file with 1 statement per line. - No empty lines or comments. - Will not continue processing SQL if an error occurs. Should I assume differently? Or are there more assumptions I should consider?
admin added the Improvement label 2026-01-22 12:36:21 +01:00
admin closed this issue 2026-01-22 12:36:22 +01:00
Author
Owner

@doctrinebot commented on GitHub (Feb 4, 2010):

Comment created by @beberlei:

My assumptions are phpMyAdmin biased. All statements are separated by ; - Lines dont matter. Problem here is that ; is a valid sign inside a literal.

@doctrinebot commented on GitHub (Feb 4, 2010): Comment created by @beberlei: My assumptions are phpMyAdmin biased. All statements are separated by ; - Lines dont matter. Problem here is that ; is a valid sign inside a literal.
Author
Owner

@doctrinebot commented on GitHub (Feb 4, 2010):

Comment created by @guilhermeblanco:

We should not be so granular during this execution.
Also, OPTIMIZE, EXPLAIN, SHOW actually returns ResultSets and break the executeUpdate.

The correct way to do it would be using a RowSet support of PDOStatement.

The patch is ok, but a small modification and it'll be great to be merged. I'm assigning this issue to me and marking it as work in progress. I'll try to commit it tonight.

Cheers,

@doctrinebot commented on GitHub (Feb 4, 2010): Comment created by @guilhermeblanco: We should not be so granular during this execution. Also, OPTIMIZE, EXPLAIN, SHOW actually returns ResultSets and break the executeUpdate. The correct way to do it would be using a RowSet support of PDOStatement. The patch is ok, but a small modification and it'll be great to be merged. I'm assigning this issue to me and marking it as work in progress. I'll try to commit it tonight. Cheers,
Author
Owner

@doctrinebot commented on GitHub (Feb 4, 2010):

Comment created by @guilhermeblanco:

First draft for patch: http://pastie.org/809510

@doctrinebot commented on GitHub (Feb 4, 2010): Comment created by @guilhermeblanco: First draft for patch: http://pastie.org/809510
Author
Owner

@doctrinebot commented on GitHub (Feb 5, 2010):

Comment created by @beberlei:

Problem is "nextRowset()" is not a method on the Doctrine\DBAL\Driver\Statement interface

@doctrinebot commented on GitHub (Feb 5, 2010): Comment created by @beberlei: Problem is "nextRowset()" is not a method on the Doctrine\DBAL\Driver\Statement interface
Author
Owner

@doctrinebot commented on GitHub (Mar 3, 2010):

Comment created by @guilhermeblanco:

In r7306 this issue was fixed.

@doctrinebot commented on GitHub (Mar 3, 2010): Comment created by @guilhermeblanco: In r7306 this issue was fixed.
Author
Owner

@doctrinebot commented on GitHub (Mar 3, 2010):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Mar 3, 2010): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Dec 13, 2015):

Imported 1 attachments from Jira into https://gist.github.com/58375ca36e51bcecb47d

@doctrinebot commented on GitHub (Dec 13, 2015): Imported 1 attachments from Jira into https://gist.github.com/58375ca36e51bcecb47d - [10344_RunSqlTaskWithSqlFile.patch](https://gist.github.com/58375ca36e51bcecb47d#file-10344_RunSqlTaskWithSqlFile-patch)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#374