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

Rename all GENERATOR references to SEQUENCE #54

Open
luronumen opened this issue Aug 11, 2020 · 6 comments
Open

Rename all GENERATOR references to SEQUENCE #54

luronumen opened this issue Aug 11, 2020 · 6 comments

Comments

@luronumen
Copy link

According to the https://firebirdsql.org/refdocs/langrefupd21-ddl-sequence.html documentation, SEQUENCE is the SQL-compliant term for what InterBase and Firebird have always called a GENERATOR and SEQUENCE is the recommended syntax from Firebird 2.0 onward.

Please rename all GENERATOR references to SEQUENCE:

  • Rename Generators node to to Sequences
  • Rename all CREATE, ALTER, etc instructions from Generators to Sequences
  • etc
@PizzaProgram
Copy link

Simply "renaming" would be NOT the best solution, since FlameRobin needs to be able to manipulate old databases too!
So this could be either:

  1. automatic (based on installed engine + OSD version! ) or
  2. manually option settable per database at connection properties.

I prefer the 2th method, because personally I don't like the new format.
GEN_ID(...) is so nice short, fast, and easy to understand.
The 1th method is harder to test. (Needs someone to install ALL kind of DB versions and try all kinds of old DBs with different ODS.
)

@arvanus
Copy link
Collaborator

arvanus commented Aug 14, 2020

Hi @luronumen! Thanks for your effort posting the problems that you found, it's helping a lot.
Please, keep posting any problem you find, the more info, the better!

About this issue: I think that unless it became unsupported in the next releases, there is a lot of other problems that are easier to fix before we can make effort changing a "cosmetic" suggested syntax.
By the way, if there is an greater impact between using generator or sequence I really don't know!!

Anyway I still don't know FR code too well to change a thing like this without losing a lots of time, but off course that if someone can help, this patch will be welcome.

@luronumen
Copy link
Author

I totally agree with you @arvanus !
Changing all references from GENERATOR to SEQUENCE can be very risky and can insert many side effects in FlameRobin.
We better wait for the http://tracker.firebirdsql.org/browse/CORE-6298 to be fixed before working on this issue.

Let's first focus on fixing issues marked as a bug here in the project by reducing our list of open / pending bugs.

Best Regards,
Luciano

@Jdochoa
Copy link
Contributor

Jdochoa commented Aug 22, 2020

I'm work in change to sequences, this change include the support to:
{ CREATE | RECREATE } { SEQUENCE | GENERATOR } [ START WITH ]
[ INCREMENT BY ]

@luronumen
Copy link
Author

Thank you very much for your work and effort to keep FlameRobin up to date @Jdochoa! I really appreciate it!

@luronumen
Copy link
Author

Please use the 5.11. SEQUENCE (GENERATOR) Language Reference for Firebird 2.5 to implement this feature.

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

No branches or pull requests

5 participants