Skip to content

Add system property for UTC timestamps. - #295

Merged
kqarryzada merged 2 commits into
masterfrom
DS-51419-add-calendar-system-property
Apr 24, 2026
Merged

Add system property for UTC timestamps. #295
kqarryzada merged 2 commits into
masterfrom
DS-51419-add-calendar-system-property

Conversation

@kqarryzada

Copy link
Copy Markdown
Collaborator

In previous releases where JAX-B was used for timestamp deserialization,
all timestamps that occurred in the default region were placed in a
timezone with an ID of "GMT+00:00". In many cases, it may be desirable
to use UTC as the default value instead.

This commit adds a new system property that can be enabled so that
deserialized timestamps will result in Calendar objects with a timezone
corresponding to UTC. This also updates existing customizable
properties to use Java system properties as well, without changing
their default value.

Reviewer: vyhhuang
Reviewer: dougbulkley

JiraIssue: DS-51419

In previous releases where JAX-B was used for timestamp deserialization,
all timestamps that occurred in the default region were placed in a
timezone with an ID of "GMT+00:00". In many cases, it may be desirable
to use UTC as the default value instead.

This commit adds a new system property that can be enabled so that
deserialized timestamps will result in Calendar objects with a timezone
corresponding to UTC. This also updates existing customizable
properties to use Java system properties as well, without changing
their default value.

Reviewer: vyhhuang
Reviewer: dougbulkley

JiraIssue: DS-51419
@kqarryzada kqarryzada self-assigned this Apr 24, 2026

@vyhhuang vyhhuang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes look fine to me.

@kqarryzada
kqarryzada merged commit dd5c137 into master Apr 24, 2026
6 checks passed
@kqarryzada
kqarryzada deleted the DS-51419-add-calendar-system-property branch April 24, 2026 21:43
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.

3 participants