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.

Sign up for GitHub

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

Jump to bottom

Optimize helidon optionals - io.helidon.common.mapper.OptionalValue #505

Merged
merged 4 commits into from
Oct 9, 2024
Merged

Optimize helidon optionals - io.helidon.common.mapper.OptionalValue #505

merged 4 commits into from
Oct 9, 2024

Commits on Oct 8, 2024

  1. Optimize helidon optionals 

    Helidon docs says that `Optional` is expensive and don't use it unless you need to.
In the generator, optionals are more of an implementation detail and null or the default value is returned.
For better performance, this PR replaces optionals with contains / get as Helidon suggests.
    @re-thc
    re-thc committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    d19fd83 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into master

    @SentryMan
    SentryMan authored Oct 8, 2024
    Configuration menu
    Copy the full SHA
    f5a00e1 View commit details
    Browse the repository at this point in the history

  3. fix build

    @re-thc
    re-thc committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    bfdaa2a View commit details
    Browse the repository at this point in the history

  4. Merge remote-tracking branch 'origin/master'

    @re-thc
    re-thc committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    f2aa8ad View commit details
    Browse the repository at this point in the history