zhang-arvin opened a new pull request, #929: URL: https://github.com/apache/incubator-xtable/pull/929
Fixes #926 ## What changes were proposed in this pull request? `ParquetConversionSource` never populated `recordCount` on the `InternalDataFile`s it creates (both the full-sync and incremental paths), leaving it at the primitive default of 0. Both targets serialize that faithfully, so every registered file showed `record_count = 0` and engines answering aggregates from metadata (Spark's optimized `count(*)`, Iceberg's `.files`, Delta's `numRecords`) returned 0 rows for non-empty tables. ## How was this patch tested? - `./mvnw -pl xtable-core -am -DskipTests compile` - Existing `ITParquetConversionSource` suite covers the conversion paths (local run pending CI). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
