substrait-io / substrait-java

Apache License 2.0
77 stars 72 forks source link

feat: deprecate Timestamp and TimestampTZ visit functions #273

Closed mbwhite closed 5 months ago

mbwhite commented 5 months ago

By marking the function as @Deprecated and providing a default implementation makes users code a lot cleaner.

Directly implementations of the interface previously would need to implement these methods and cope with the Deprecation warning.

With this change if they are using it, no problems; but they are not obliged to implement it.