GeoArrowType::from_extension_field and GeoArrowType::from_arrow_field should return Result<Option<Self>>

[sunng87]

While working with GeoArrow data and other types of arrow data, I need to check arrow field with GeoArrowType::from_extension_field or GeoArrowType::from_arrow_field to see if it's geoarrow arrays and convert it accordingly.

The current API assumes the input is geoarrow array, and will return Err when extension not found, or extension is from geoarrow. This makes it difficult know if it's not geoarrow array or just invalid metadata.

I think it's better to change return type to Option so we can return None if it's not a geoarray type. This is going to be a breaking change. If it sounds good to you I can create a PR for this.

[kylebarron]

👍 I think this makes sense. Would you be able to put up a PR?

[sunng87]

Great! Will work on it this weekend.