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
Create a legacy configuration property to map COUNT as INTEGER instead of BIGINT [CORE5514] #5783
Comments
Commented by: @asfernandes Why? |
Commented by: Beto Neto (betoneto.tbo) My ERP application has a lot of queries hardcoded in different artifacts (pas, dfm, etc). There is also customized SQLs persisted inside client's database (reports, charts, etc). For us is very heavy to create a migration routine to change this artifacts to force an "cast as integer" or fix to use the correct data type mapper. If there was a legacy configuration property to do this (like "AuthClient = Legacy_Auth") will be awesome. |
Commented by: @livius2 I can confirm this problem e.g. in Delphi. |
Commented by: @asfernandes The component can declare for the Firebord API what field type it expects. If it doesn't do it, the bug is in it. |
Commented by: Beto Neto (betoneto.tbo) Yes, and it declares! But this is the problem! All declarations are in TIntegerField (32 bits) and now the query returns a Bigint (64bits). |
Commented by: @dyemanov So what? Result will be silently casted to Integer. Of course, it may not fit, but do you really expect more than 4G records in tables? Anyway, getting a runtime overflow error may still be better than getting e.g. negative result due to wrapping. |
Commented by: @asfernandes Beto, even files already compiled by the ancient tool GPRE works after the change in COUNT. |
Commented by: Beto Neto (betoneto.tbo) The problem is that my dbexpress driver is mapping the new count as a FMTBcd, and this is resulting an exception "TIntegerField is not compatible with FMTBcdField". I'm using Delphi 7 with an dbexpress driver owned from upscene. I known that is an deprecated IDE, but that is what my company has. |
Commented by: @aafemt Isn't this ticket covered by SET BIND command? |
Commented by: Beto Neto (betoneto.tbo) Dimitry, can you provide more details or samples of SET BIND? |
Submitted by: Beto Neto (betoneto.tbo)
Votes: 1
Create a legacy configuration property to map COUNT as INTEGER instead of BIGINT in firebird.conf.
The text was updated successfully, but these errors were encountered: