-
Notifications
You must be signed in to change notification settings - Fork 23
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
Enhancement: Extension for Reflection-Free Usage in Eclipse JNoSQL #95
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Signed-off-by: Otavio Santana <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request substantially enhances the Eclipse JNoSQL framework, enabling developers to utilize the platform without reliance on reflection. The primary goals of this enhancement are improved performance, enhanced CDI Lite compatibility, and the promotion of cleaner, more maintainable code.
Changes Introduced:
Java Annotation Processor Inclusion: Instead of reflection, the pull request includes a Java Annotation Processor, enabling compile-time checks and optimizations. This approach maintains the existing APIs, simplifying the developers' transition without learning new ones.
Seamless CDI Lite Compatibility: With this enhancement, Eclipse JNoSQL can seamlessly integrate with CDI Lite environments, providing the benefits of CDI Lite's features.
Warning for Limitations: The documentation now includes clear warnings about limitations in the entity model, such as the lack of Java Record support, the requirement for a default constructor, and the need for getters and setters for annotated fields.
Usage Guide:
The accompanying documentation provides detailed steps to implement this enhancement, ensuring a smooth transition to a reflection-free environment. Developers will find concise instructions on removing the reflection engine dependency, including the Java Annotation Processor and addressing the limitations introduced by the extension.
The provided documentation aims to facilitate a seamless adoption of this enhancement while maximizing its performance, compatibility, and codebase quality benefits.
Testing:
The enhancements introduced in this pull request have undergone extensive testing across various database engines and scenarios. The changes have been rigorously tested to ensure stability and compatibility with existing codebases.
Impact:
This enhancement significantly contributes to Eclipse JNoSQL's evolution by offering an alternative approach to reflection and promoting cleaner coding practices. Developers can expect improved application performance, enhanced CDI Lite compatibility, and better project maintainability.
This pull request aligns with the project's goals and is expected to positively impact the Eclipse JNoSQL community, providing developers greater flexibility and control over their applications.