[fix](orc) fix the count(*) pushdown issue in orc format (apache#24446)

In previous, when querying hive table in orc format, and the file is splitted.
the result of select count(*) may be multiple of the real row number.

This is because the number of rows should be got after orc strip prune,
otherwise, it may return wrong result

be/src/vec/exec/format/orc/vorc_reader.cpp

@@ -245,8 +245,6 @@ Status OrcReader::_create_file_reader() {
         }
         return Status::InternalError("Init OrcReader failed. reason = {}", _err_msg);
     }
-    _remaining_rows = _reader->getNumberOfRows();
-
     return Status::OK();
 }
 
@@ -789,6 +787,9 @@ Status OrcReader::set_fill_columns(
         auto& selected_type = _row_reader->getSelectedType();
         int idx = 0;
         _init_select_types(selected_type, idx);
+
+        _remaining_rows = _row_reader->getNumberOfRows();
+
     } catch (std::exception& e) {
         return Status::InternalError("Failed to create orc row reader. reason = {}", e.what());
     }

be/src/vec/exec/format/orc/vorc_reader.h

@@ -489,6 +489,8 @@ class OrcReader : public GenericReader {
     void set_remaining_rows(int64_t rows) { _remaining_rows = rows; }
 
 private:
+    // This is only for count(*) short circuit read.
+    // save the total number of rows in range
     int64_t _remaining_rows = 0;
     RuntimeProfile* _profile = nullptr;
     RuntimeState* _state = nullptr;

fe/fe-core/src/main/java/org/apache/doris/tablefunction/HdfsTableValuedFunction.java

@@ -70,7 +70,7 @@ public HdfsTableValuedFunction(Map<String, String> params) throws AnalysisExcept
                 // because HADOOP_FS_NAME contains upper and lower case
                 locationProperties.put(HdfsResource.HADOOP_FS_NAME, params.get(key));
             } else {
-                throw new AnalysisException(key + " is invalid property");
+                locationProperties.put(key, params.get(key));
             }
         }
 

regression-test/data/external_table_p2/tvf/test_hdfs_tvf_compression.out

@@ -248,3 +248,21 @@ c133	TEXT	Yes	false	\N	NONE
 
 -- !plain_2 --
 
+-- !count_parquet_0 --
+1062734
+
+-- !count_parquet_1 --
+1062734
+
+-- !count_orc_0 --
+2777636
+
+-- !count_orc_1 --
+2777636
+
+-- !count_text_0 --
+144730
+
+-- !count_text_1 --
+144730
+

regression-test/suites/external_table_p2/tvf/test_hdfs_tvf_compression.groovy

@@ -105,7 +105,70 @@ suite("test_hdfs_tvf_compression", "p2,external,tvf,external_remote,external_rem
             "column_separator" = '\001',
             "compress_type" = "plain") where c2="abc" order by c3,c4,c10 limit 5;
         """
+
+        // test count(*) push down
+        def test_data_dir = "hdfs://${nameNodeHost}:${hdfsPort}"
+        // parquet
+        sql """set file_split_size=0;"""
+        qt_count_parquet_0 """ 
+        select count(*) from 
+        HDFS(
+            "uri" = "${test_data_dir}/test_data/ckbench_hits.part-00000.snappy.parquet",
+            "fs.defaultFS" = "${baseFs}",
+            "format" = "parquet"
+        );
+        """
+
+        sql """set file_split_size=388608;"""
+        qt_count_parquet_1 """ 
+        select count(*) from 
+        HDFS(
+            "uri" = "${test_data_dir}/test_data/ckbench_hits.part-00000.snappy.parquet",
+            "fs.defaultFS" = "${baseFs}",
+            "format" = "parquet"
+        );
+        """
+
+        // orc
+        sql """set file_split_size=0;"""
+        qt_count_orc_0 """ 
+        select count(*) from 
+        HDFS(
+            "uri" = "${test_data_dir}/test_data/ckbench_hits.000000_0.orc",
+            "fs.defaultFS" = "${baseFs}",
+            "format" = "orc"
+        );
+        """
+
+        sql """set file_split_size=388608;"""
+        qt_count_orc_1 """ 
+        select count(*) from 
+        HDFS(
+            "uri" = "${test_data_dir}/test_data/ckbench_hits.000000_0.orc",
+            "fs.defaultFS" = "${baseFs}",
+            "format" = "orc"
+        );
+        """
 
+        // text
+        sql """set file_split_size=0;"""
+        qt_count_text_0 """ 
+        select count(*) from 
+        HDFS(
+            "uri" = "${test_data_dir}/test_data/tpcds_catalog_returns_data-m-00000.txt",
+            "fs.defaultFS" = "${baseFs}",
+            "format" = "csv"
+        );
+        """
 
+        sql """set file_split_size=388608;"""
+        qt_count_text_1 """ 
+        select count(*) from 
+        HDFS(
+            "uri" = "${test_data_dir}/test_data/tpcds_catalog_returns_data-m-00000.txt",
+            "fs.defaultFS" = "${baseFs}",
+            "format" = "csv"
+        );
+        """
     }
 }