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

Add datatypes (VAR)BINARY(n) and BINARY VARYING(n) as alias for (VAR)CHAR(n) CHARACTER SET OCTETS [CORE5064] #5351

Closed
firebird-automations opened this issue Jan 3, 2016 · 6 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @mrotteveel

Assigned to: @aafemt

Relate to CORE5494

Votes: 1

The CHAR(n) CHARACTER SET OCTETS and VARCHAR(n) CHARACTER SET OCTETS types are basically the SQL standard type BINARY and VARBINARY (or BINARY VARYING). Please add the following aliases for these types:

* BINARY(n) for CHAR(n) CHARACTER SET OCTETS
* VARBINARY(n) for VARCHAR(n) CHARACTER SET OCTETS
* BINARY VARYING(n) for VARCHAR(n) CHARACTER SET OCTETS

Using either the alias or the 'old' full definition should become an explicit subtype of CHAR and VARCHAR, instead of implicit based on the character set id as it is now. This can be achieved by setting the subtype in RDB$FIELD_SUB_TYPE (in RDB$FIELDS and RDB$FUNCTION_ARGUMENTS) to 1 (CS_BINARY). This has the added benefit that tools can use this information to determine the data type consistently based on type and subtype; this would be compatible with current usage of the sqlsubtype in XSQLVAR to communicate the character set id.

Commits: 7b9b408

@firebird-automations
Copy link
Collaborator Author

Commented by: @mariuz

related commit

7b9b408

@firebird-automations
Copy link
Collaborator Author

Modified by: @aafemt

assignee: Dimitry Sibiryakov [ aafemt ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @aafemt

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 4.0 Alpha 1 [ 10731 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Done successfully

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @mrotteveel

Link: This issue relate to CORE5494 [ CORE5494 ]

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

1 participant