tweaks for pg13

src/arrow_parquet/arrow_to_pg.rs

@@ -17,8 +17,8 @@ use pgrx::{
 use crate::{
     pgrx_utils::{
         array_element_typoid, collect_valid_attributes, domain_array_base_elem_typoid,
-        is_array_type, is_composite_type, is_supported_array_element_type, is_supported_composite_type,
-        tuple_desc,
+        is_array_type, is_composite_type, is_supported_array_element_type,
+        is_supported_composite_type, tuple_desc,
     },
     type_compat::{
         fallback_to_text::{reset_fallback_to_text_context, FallbackToText},

src/lib.rs

@@ -2353,6 +2353,7 @@ mod tests {
         Spi::run(parquet_metadata_command).unwrap();
     }
 
+    #[cfg(not(feature = "pg13"))]
     #[pg_test]
     #[should_panic(
         expected = "type mismatch for column \"location\" between table and parquet file"

src/pgrx_utils.rs

@@ -59,16 +59,19 @@ pub(crate) fn is_array_type(typoid: Oid) -> bool {
     unsafe { type_is_array(typoid) }
 }
 
-pub(crate) fn is_supported_array_element_type(array_element_id: Oid) -> bool {
+pub(crate) fn is_supported_array_element_type(_array_element_id: Oid) -> bool {
     #[cfg(feature = "pg13")]
-    return u32::from(array_element_id) < FirstNormalObjectId;
-    #[cfg(not(feature = "pg13"))]
+    if u32::from(_array_element_id) >= FirstNormalObjectId {
+        // we don't support arrays of user-defined composite types in pg 13
+        return !is_composite_type(_array_element_id);
+    }
+
     true
 }
 
-pub(crate) fn is_supported_composite_type(composite_id: Oid) -> bool {
+pub(crate) fn is_supported_composite_type(_composite_id: Oid) -> bool {
     #[cfg(feature = "pg13")]
-    return u32::from(composite_id) < FirstNormalObjectId;
+    return !u32::from(_composite_id) >= FirstNormalObjectId;
     #[cfg(not(feature = "pg13"))]
     true
 }