db-migrate/node-db-migrate

Cannot migrate stored procedure

Open

#625 aperta il 24 mag 2019

Vedi su GitHub
 (13 commenti) (2 reazioni) (0 assegnatari)JavaScript (367 fork)batch import
Upstreamfeaturehelp wanted

Metriche repository

Star
 (2296 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga su come db migrate mysql analizza i file SQL, concentrandosi sulla gestione delle istruzioni DELIMITER. Il problema probabilmente risiede nello splitter SQL o nel runner di migrazione che attualmente tratta DELIMITER come una normale istruzione SQL causando errori di parsing. Esamina il codice sorgente in 'lib/db migrate mysql.js' o file simili, e considera l'implementazione di un passaggio di pre elaborazione per gestire i cambiamenti del delimitatore prima di passare l'SQL al driver MySQL. Rivedi i commenti esistenti per eventuali soluzioni proposte o workaround.
Tech stack
javascriptnodejsmysql
Dominio
backenddatabase
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsMySQL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65