address feedback

Author: wgtmac

src/iceberg/manifest/manifest_reader.cc

@@ -601,7 +601,7 @@ bool ManifestReader::ShouldDropStats(const std::vector<std::string>& columns) {
   // select record_count, which is a primitive type.
   if (!columns.empty()) {
     const std::unordered_set<std::string_view> selected(columns.cbegin(), columns.cend());
-    if (selected.contains(ManifestReader::kAllColumns)) {
+    if (selected.contains(Schema::kAllColumns)) {
       return false;
     }
     std::unordered_set<std::string_view> intersection;

src/iceberg/meson.build

@@ -167,6 +167,7 @@ install_headers(
         'arrow_c_data.h',
         'catalog.h',
         'constants.h',
+        'delete_file_index.h',
         'exception.h',
         'file_format.h',
         'file_io.h',

src/iceberg/test/manifest_reader_test.cc

@@ -426,8 +426,7 @@ TEST_P(TestManifestReader, TestDropStats) {
 
 TEST(ManifestReaderStaticTest, TestShouldDropStats) {
   EXPECT_FALSE(ManifestReader::ShouldDropStats({}));
-  EXPECT_FALSE(
-      ManifestReader::ShouldDropStats({std::string(ManifestReader::kAllColumns)}));
+  EXPECT_FALSE(ManifestReader::ShouldDropStats({std::string(Schema::kAllColumns)}));
   EXPECT_TRUE(ManifestReader::ShouldDropStats({"file_path", "file_format", "partition"}));
   EXPECT_TRUE(
       ManifestReader::ShouldDropStats({"file_path", "file_format", "record_count"}));