fix review

Author: HuaHuaY

cpp/src/parquet/arrow/schema.cc

@@ -704,8 +704,8 @@ Status ResolveList(const GroupNode& group, const Node& list_node,
   auto check_two_level_list_repeated = [](const GroupNode& group) -> Status {
     // When it is repeated, the LIST-annotated 2-level structure can only serve as an
     // element within another LIST-annotated 2-level structure.
-    if (group.is_repeated() && group.parent() != nullptr &&
-        !group.parent()->logical_type()->is_list()) {
+    if (group.is_repeated() &&
+        (group.parent() == nullptr || !group.parent()->logical_type()->is_list())) {
       return Status::Invalid("LIST-annotated groups must not be repeated.");
     }
     return {};
@@ -725,6 +725,7 @@ Status ResolveList(const GroupNode& group, const Node& list_node,
 
     if (list_group.logical_type()->is_none() && HasListElementName(list_group, group)) {
       // Backward-compatibility rule 4
+      RETURN_NOT_OK(check_two_level_list_repeated(group));
       return GroupToStruct(list_group, current_levels, ctx, out, child_field);
     }