Rename all GENERATOR references to SEQUENCE about flamerobin HOT 6 OPEN

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

from flamerobin.

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.

from flamerobin.

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. )

from flamerobin.

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

from flamerobin.

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

from flamerobin.

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

from flamerobin.