[ 
https://issues.apache.org/jira/browse/PARQUET-2374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17788449#comment-17788449
 ] 

ASF GitHub Bot commented on PARQUET-2374:
-----------------------------------------

wgtmac commented on code in PR #1187:
URL: https://github.com/apache/parquet-mr/pull/1187#discussion_r1400746050


##########
parquet-hadoop/src/main/java/org/apache/parquet/hadoop/ColumnChunkPageReadStore.java:
##########
@@ -80,10 +80,12 @@ static final class ColumnChunkPageReader implements 
PageReader {
     private final byte[] dataPageAAD;
     private final byte[] dictionaryPageAAD;
 
+    ParquetMetricsCallback metricsCallback;

Review Comment:
   Did you forget to push your local changes? @parthchandra 





> Add metrics support for parquet file reader
> -------------------------------------------
>
>                 Key: PARQUET-2374
>                 URL: https://issues.apache.org/jira/browse/PARQUET-2374
>             Project: Parquet
>          Issue Type: Improvement
>          Components: parquet-mr
>    Affects Versions: 1.13.1
>            Reporter: Parth Chandra
>            Priority: Major
>
> ParquetFileReader is used by many engines - Hadoop, Spark among them. These 
> engines report various metrics to measure performance in different 
> environments and it is usually useful to be able to get low level metrics out 
> of the file reader and writers.
> It would be very useful to allow a simple interface to report the metrics. 
> Callers can then implement the interface to record the metrics in any 
> subsystem they choose.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to