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

Application class update #130

Merged

Conversation

arina-ielchiieva
Copy link
Collaborator

  • Changes discussed in PR.
  1. Renamed commandhandler package to extensions.
  2. Moved CustomApplication to extensions package.
  3. OutputFormat and AbstractOutputFormat are made public.
  4. Formats and command handlers holders are made immutable:
    4.1. Config class in Sqlline wraps all given constructor parameters into unmodifiable objects.
    4.2. Two clone methods were added into Config class to clone its content when formats / command handlers are changed.
    4.3. Two methods to update formats / command handlers are added into Sqlline class which clones and re-assigns Config.
  5. Added unit test to demonstrate how application configuration can be reset at runtime.
  6. Updated javadocs.
  • Added possibility to override SqlLine session options in Application.

1. Renamed commandhandler package to extensions.
2. Moved CustomApplication to extensions package.
3. OutputFormat and AbstractOutputFormat are made public.
4. Formats and command handlers holders are made immutable:
4.1. Config class in Sqlline wraps all given constructor parameters into unmodifiable objects.
4.2. Two clone methods were added into Config class to clone its content when formats / command handlers are changed.
4.3. Two methods to update formats / command handlers are added into Sqlline class which clones and re-assigns Config.
5. Added unit test to demonstrate how application configuration can be reset at runtime.
6. Updated javadocs.
@julianhyde julianhyde merged commit 0b6ca84 into julianhyde:106-application Sep 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants