Skip to content

SparkCastOptions should represent timezone as Option<Tz> instead of String #1147

@andygrove

Description

@andygrove

What is the problem the feature request solves?

In Comet's native cast logic, we currently pass timezones around as strings and use empty strings to represent lack of timezones. This results in repeated parsing of timestamps that could be avoided.

Describe the potential solution

Change timezone from String to Option<Tz>

Additional context

See SparkCastOptions, added in #1146.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requesthelp wantedExtra attention is needed

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions