mariuz / flamerobin

FlameRobin is a database administration tool for Firebird RDBMS. Our goal is to build a tool that is: lightweight (small footprint, fast execution) cross-platform (Linux, Windows, Mac OS X, FreeBSD) dependent only on other Open Source software
http://flamerobin.org
MIT License
216 stars 67 forks source link

Add Support for Identity Columns #71

Open pgiacomo69 opened 4 years ago

pgiacomo69 commented 4 years ago

Thanks for your Work!

Currently, when showing table properties or generating its DDL, Identitity attributes for columns are completely ignored. I understand that to manage this property it needs several modifications on Column Property form, but for now it would be very useful to have this information at least showed in Table property page and "Show DDL".

luronumen commented 4 years ago

ACTUAL RESULT Identity column is displayed as "ID BIGINT NOT NULL" instead of "ID BIGINT GENERATED BY DEFAULT AS IDENTITY NOT NULL"

STEPS TO REPROCUDE THIS ISSUE

Jdochoa commented 3 years ago

Hi @pgiacomo69 , please see https://github.com/Jdochoa/flamerobin/tree/FB3 and help me with feedback

pgiacomo69 commented 3 years ago

Hi @pgiacomo69 , please see https://github.com/Jdochoa/flamerobin/tree/FB3 and help me with feedback

Ok, 0.9.3.5 Snapshot partially solves the issue, altough column property dialog still doesn't work well for identity columns, having it shown in DDL it's ok! Great Work!

luronumen commented 3 years ago

Hi @pgiacomo69 ,

Is this issue still reproducible in 0.9.3.8 Snapshot version?

pgiacomo69 commented 3 years ago

Actually, creating a table: CREATE TABLE table_name ( id integer generated by default as identity primary key, descr varchar(20) );

its ddl from 'Properties' is different:

CREATE TABLE TABLE_NAME ( ID integer GENERATED BY DEFAULT AS IDENTITY NOT NULL, DESCR varchar(20), CONSTRAINT INTEG_92 PRIMARY KEY (ID) );