TimestampConversion.getRecommendedSchema() question

[Sean Timm]

org.apache.avro.data.TimeConversions.TimestampConversion does not
implement getRecommendedSchema(). Is this intentional? When trying to
dynamically generate an Avro schema from a pojo that contains a DateTime
object using ReflectData, I get an unsupported operation exception.

I think the implementation should be as simple as

@Override
public Schema getRecommendedSchema() {
      return
LogicalTypes.timestampMillis().addToSchema(Schema.create(Schema.Type.LONG));
}

If folks agree, I can create a Jira and pull request.

Thanks,
Sean