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

Decisions on charactersets happen in to many places [JDBC269] #316

Closed
firebird-automations opened this issue Aug 19, 2012 · 5 comments
Closed

Comments

@firebird-automations
Copy link

Submitted by: @mrotteveel

There are too many places in the Jaybird code were decisions on charactersets are made; some parts retrieve a new instance of the Encoding implementation for the connection characterset whenever it needs it, instead of storing it once and reusing it.

Potential improvements:
* Factor some parts of EncodingFactory to a separate class to act as 'the characterset' (including things like getIscEncodingSize and getEncoding)
* Have the connection keep the instance of its connection characterset so it is available for reuse
* ...

Commits: 4bffb5e b1c9bd9

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

assignee: Roman Rokytskyy [ rrokytskyy ] => Mark Rotteveel [ avalanche1979 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

Fix Version: Jaybird 3.0 [ 10440 ]

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

Most of the encoding information has been moved to Encoding and related factories, the impact of these changes still needs to be documented.

There are still a number of changes that could improve this further, but I will defer that to a future version.

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

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

resolution: Fixed [ 1 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

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

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

2 participants