Skip to content

Improve performance of evaluations by delaying number formatting until requested #880

Description

@garethbowen

Description

All classes that extend ValueEvaluation init booleanValue, stringValue, and numberValue fields in constructors. But most of the time the numberValue isn't needed and it's not necessarily trivial to parse as the underlying code also tries to parse it as a date, and catches errors if it fails.

Modify the Evaluation classes to only attempt to parse the number when toNumber is called, and cache the result so subsequent accesses are free.

Related Issues

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Projects

    Status
    Backlog

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions