Fix a bug in CastingGenerator for skipped cols

Fix typo again

Fix file name

Author: sahil1105

recipe/patches/0004-Bodo-Changes.patch

@@ -1,5 +1,5 @@
 diff --git a/cpp/src/arrow/dataset/file_parquet.cc b/cpp/src/arrow/dataset/file_parquet.cc
-index 1f8b6cc488..586a5122a5 100644
+index 1f8b6cc488..6ac6642fdb 100644
 --- a/cpp/src/arrow/dataset/file_parquet.cc
 +++ b/cpp/src/arrow/dataset/file_parquet.cc
 @@ -26,16 +26,23 @@
@@ -55,9 +55,9 @@ index 1f8b6cc488..586a5122a5 100644
 +                              field_ref.GetOneOrNone(*next));
 +        if (column) {
 +          if (this->cols_to_skip_.count(field->name())) {
-+            out_cols.emplace_back(std::move(column));
 +            // Maintain the original input type.
 +            out_schema_fields.emplace_back(field->WithType(column->type()));
++            out_cols.emplace_back(std::move(column));
 +            continue;
 +          }
 +          if (!column->type()->Equals(field->type())) {
@@ -69,8 +69,8 @@ index 1f8b6cc488..586a5122a5 100644
 +            column = converted.make_array();
 +            changed = true;
 +          }
-+          out_cols.emplace_back(std::move(column));
 +          out_schema_fields.emplace_back(field->Copy());
++          out_cols.emplace_back(std::move(column));
 +          // XXX Do we need to handle the else case? What happens when the column
 +          // doesn't exist, e.g. all null or all the same value?
 +        }