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

API, Core: implement types timestamp_ns and timestamptz_ns #8971

Closed
wants to merge 9 commits into from
Closed

API, Core: implement types timestamp_ns and timestamptz_ns #8971

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
3c77542
API: implement types timestamp_ns and timestamptz_ns
jacobmarble Sep 26, 2023
5fd82c2
Core: add types timestamp_ns and timestamptz_ns
jacobmarble Sep 28, 2023
24e6143
chore: update ORC and Parquet
jacobmarble Nov 3, 2023
3b87435
chore: cast timestamp and get unit once
jacobmarble Nov 3, 2023
74b90d9
chore: clean up switch/default/if/else logic
jacobmarble Nov 3, 2023
5278960
chore: cleanup exceptions thrown for timestamp unit
jacobmarble Nov 3, 2023
77bd16d
chore: deprecate TimestampType.with[out]Zone()
jacobmarble Nov 3, 2023
c910484
fix: interpret timestamp strings as micros or nanos
jacobmarble Nov 6, 2023
cb109a4
cleanup: revert unrelated change
jacobmarble Nov 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
chore: deprecate TimestampType.with[out]Zone()
  • Loading branch information
@jacobmarble
jacobmarble committed Nov 7, 2023
commit 77bd16db12c5797612369460f0a067e6b8d17e9e
4 changes: 4 additions & 0 deletions api/src/main/java/org/apache/iceberg/types/Types.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -218,10 +218,14 @@ public static class TimestampType extends PrimitiveType {
private static final TimestampType INSTANCE_NANOS_WITHOUT_ZONE =
new TimestampType(false, ChronoUnit.NANOS);

/** @deprecated use {@link #microsWithZone()} instead. */
@Deprecated
public static TimestampType withZone() {
return INSTANCE_MICROS_WITH_ZONE;
}

/** @deprecated use {@link #microsWithoutZone()} instead. */
@Deprecated
public static TimestampType withoutZone() {
return INSTANCE_MICROS_WITHOUT_ZONE;
}
Expand Down