py-mysql2pgsql icon indicating copy to clipboard operation
py-mysql2pgsql copied to clipboard

Fixing case sensitive fk

Open Nepherhotep opened this issue 10 years ago • 4 comments

The script can't handle indexes and constraints for camel cased columns. The fix just adds quotes to support it (the same as it is done for table names).

Nepherhotep avatar Jul 14 '15 20:07 Nepherhotep

:+1: thanks! This fixed the rather egregious problems I was having with columns named "userID" and the like.

Why this hasn't been merged in over 3 months, I have no idea.

mattlqx avatar Nov 02 '15 22:11 mattlqx

It looks like the repo is abandoned. By the way, I faced with another issue - for Django 1.8 it's required to use fk constraints with options "DEFERRABLE INITIALLY DEFERRED". I didn't create pull request as the one above wasn't merged, just fixed it in my database with migration altering existing fk contraints. But I suggest before going live to update the script as well (may be there are pull requests, I don't know).

Nepherhotep avatar Nov 04 '15 15:11 Nepherhotep

Hey, I'm listening. :) I'm not active as I'm not working with MySQL, PostgreSQL or Python currently. Checking each patch is taking time, and I'm the kind of person that prefers defer the task if can't be sure of quality. PS: I'm planning to expand my home network. If there would be a place for test MySQL server I'll try to come back. But again, first I'll add some test's to check that I'm not breaking anything with patches.

kworr avatar Nov 04 '15 17:11 kworr

:) Well, it's opensource anyway, nobody owes to maintain repo. This script is a great tool, personally for me it helped to migrate a big production project from MySQL, so great thanks for sharing it to community. I suppose it is worth to add in description - maintainer help wanted :)

Nepherhotep avatar Nov 06 '15 09:11 Nepherhotep