db-migrate/node-db-migrate

Cannot migrate stored procedure

Open

#625 aberto em 24 de mai. de 2019

Ver no GitHub
 (13 comments) (2 reactions) (0 assignees)JavaScript (2.296 stars) (367 forks)batch import
Upstreamfeaturehelp wanted

Description

I'm submitting a...

  • Bug report
  • Feature request
  • Question

Current behavior

When created a stored procedure it's necessary to change the delimiter e.g. DELIMITER $$

However, when using a SQL-file it returns ER_PARSE_ERROR:

Expected behavior

Be able to change the delimiter, create the SP and return delimiter back to standard ;

Minimal reproduction of the problem with instructions

DELIMITER // CREATE PROCEDURE testProc() BEGIN SELECT 1; END // DELIMITER ;

What is the motivation / use case for changing the behavior?

Cannot create or migrate stored procedures

Environment

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Guia do colaborador

Pilha de tecnologia
javascriptnodejssqlmysql
Domain
backenddatabase
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
basic MySQL stored procedure syntaxfamiliarity with db migrate
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65
Direção de pesquisa
Investigate how db migrate mysql parses SQL files, focusing on the handling of DELIMITER statements. The issue likely resides in the SQL splitter or migration runner that currently treats DELIMITER as a regular SQL statement causing parse errors. Examine the source code in 'lib/db migrate mysql.js' or similar files, and consider implementing a preprocessing step to handle delimiter changes before passing SQL to the MySQL driver. Review the existing comments for any proposed solutions or workarounds.