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 (367 forks)batch import
Upstreamfeaturehelp wanted

Métricas do repositório

Stars
 (2.296 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Direção de pesquisa
Investigue como o db migrate mysql analisa arquivos SQL, focando no tratamento das instruções DELIMITER. O problema provavelmente está no divisor SQL ou no executor de migração que atualmente trata DELIMITER como uma instrução SQL regular, causando erros de análise. Examine o código fonte em 'lib/db migrate mysql.js' ou arquivos similares, e considere implementar uma etapa de pré processamento para lidar com alterações de delimitador antes de passar o SQL para o driver MySQL. Revise os comentários existentes para quaisquer soluções ou workarounds propostos.
Pilha de tecnologia
javascriptnodejsmysql
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Node.jsMySQL
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65