feat(arrow-avro): Configurable Arrow timezone ID for Avro timestamps

[mzabaluev]

Which issue does this PR close?

• Closes Configurable Arrow representation of UTC timestamps for Avro reader #9279.

Rationale for this change

Enable an alternative representation of UTC timestamp data types with the "UTC" timezone ID, which is useful for interoperability with applications preferring that form.

What changes are included in this PR?

In the ReaderBuilder API, add a new method with_tz that allows users to specify the timezone ID for Avro logical types that represent UTC timestamps. The choices are between "+00:00" and "UTC" and can be selected by the new Tz enumeration.

Are these changes tested?

Added unit tests to verify the representation with different Tz parameter values.

Are there any user-facing changes?

A new with_tz method is added to arrow_avro::reader::Builder.

[EmilyMatt]

This makes more sense in my opinion than using "+00:00" exclusively

[jecsand838]

@mzabaluev LMGTM. Really like this improvement to arrow-avro!

For me the big things are:

1. An e2e test in reader/mod.rs using the new with_tz.
2. Better documentation on the public with_tz method.

[jecsand838]

Would you mind adding an e2e test using this?

Also may help to explain in the doc comment what the default value is.

[jecsand838]

This maybe a bit cleaner.

Suggested change

	Codec::TimestampMillis(Some(Tz::OffsetZero)) => Self::TimestampMillisUtc,
	Codec::TimestampMillis(Some(Tz::Utc)) => Self::TimestampMillisUtc,
	Codec::TimestampMillis(None) => Self::TimestampMillisLocal,
	Codec::TimestampMicros(Some(Tz::OffsetZero)) => Self::TimestampMicrosUtc,
	Codec::TimestampMicros(Some(Tz::Utc)) => Self::TimestampMicrosUtc,
	Codec::TimestampMicros(None) => Self::TimestampMicrosLocal,
	Codec::TimestampNanos(Some(Tz::OffsetZero)) => Self::TimestampNanosUtc,
	Codec::TimestampNanos(Some(Tz::Utc)) => Self::TimestampNanosUtc,
	Codec::TimestampNanos(None) => Self::TimestampNanosLocal,
	Codec::TimestampMillis(Some(_)) => Self::TimestampMillisUtc,
	Codec::TimestampMillis(None) => Self::TimestampMillisLocal,
	Codec::TimestampMicros(Some(_)) => Self::TimestampMicrosUtc,
	Codec::TimestampMicros(None) => Self::TimestampMicrosLocal,
	Codec::TimestampNanos(Some(_)) => Self::TimestampNanosUtc,
	Codec::TimestampNanos(None) => Self::TimestampNanosLocal,

[mzabaluev]

I was wary of matching Some(_) because if Tz ever gets non-UTC variants, the suggested code will still compile, but will be wrong.

[jecsand838]

Then down here you could do:

Suggested change

	Self::TimestampMillis(tz) => DataType::Timestamp(
	TimeUnit::Millisecond,
	tz.as_ref().map(|tz| tz.to_string().into()),
	),
	Self::TimestampMicros(tz) => DataType::Timestamp(
	TimeUnit::Microsecond,
	tz.as_ref().map(|tz| tz.to_string().into()),
	),
	Self::TimestampNanos(tz) => DataType::Timestamp(
	TimeUnit::Nanosecond,
	tz.as_ref().map(|tz| tz.to_string().into()),
	),
	Self::TimestampMillis(tz) => DataType::Timestamp(
	TimeUnit::Millisecond,
	tz.as_ref().map(|tz| tz.as_str().into()),
	),
	Self::TimestampMicros(tz) => DataType::Timestamp(
	TimeUnit::Microsecond,
	tz.as_ref().map(|tz| tz.as_str().into()),
	),
	Self::TimestampNanos(tz) => DataType::Timestamp(
	TimeUnit::Nanosecond,
	tz.as_ref().map(|tz| tz.as_str().into()),
	),

[alamb]

Marking as draft as I think this PR is no longer waiting on feedback and I am trying to make it easier to find PRs in need of review. Please mark it as ready for review when it is ready for another look

[mzabaluev]

@alamb I have added tests, the same builder option is also added to the async reader now that it's merged.

[jecsand838]

LGTM