Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

solve issue #61 sort the attributes of the original table (PQLAttribute) #74

Closed
wants to merge 2 commits into from

Conversation

rafaelsagastume
Copy link
Contributor

Good afternoon, Euler @eulerto

A pleasure to greet you, the solution that I recommend is to sort the attributes of the PQLAttribute list after pgquarrel compares by name since it is necessary, this process generates the SQL of the dumpCreateTable table with the original order of the attributes to be able to solve the ISSUE #61

source table

CREATE TABLE public.proceso_uno
(
id integer NOT NULL DEFAULT nextval('proceso_uno_id_seq'::regclass),
nombre_proceso character varying(500) COLLATE pg_catalog."default" NOT NULL DEFAULT 'Ninguno'::character varying,
fecha_ingreso timestamp without time zone DEFAULT now(),
fecha_proceso date NOT NULL,
campo text COLLATE pg_catalog."default",
numerico numeric(15,2) NOT NULL DEFAULT 15.2,
esta_activo boolean NOT NULL DEFAULT true,
id_padre integer,
propiedades bigint NOT NULL,
direccion character varying(500) COLLATE pg_catalog."default",
cantidad integer,
precio integer
);

target table [sql generated by pgquarrel]

CREATE TABLE public.proceso_uno (
id integer DEFAULT nextval('proceso_uno_id_seq'::regclass) NOT NULL,
nombre_proceso character varying(500) DEFAULT 'Ninguno'::character varying NOT NULL,
fecha_ingreso timestamp without time zone DEFAULT now(),
fecha_proceso date NOT NULL,
campo text,
numerico numeric(15,2) DEFAULT 15.2 NOT NULL,
esta_activo boolean DEFAULT true NOT NULL,
id_padre integer,
propiedades bigint NOT NULL,
direccion character varying(500),
cantidad integer,
precio integer
);

always open to your feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant