RussellSpitzer commented on code in PR #13785:
URL: https://github.com/apache/iceberg/pull/13785#discussion_r2277865601


##########
docs/docs/configuration.md:
##########
@@ -63,7 +63,7 @@ Iceberg tables support table properties to configure table 
behavior, like the de
 | write.orc.bloom.filter.fpp                           | 0.05                  
      | False positive probability for Bloom filter (must > 0.0 and < 1.0)      
                                                                                
                                          |
 | write.location-provider.impl                         | null                  
      | Optional custom implementation for LocationProvider                     
                                                                                
                                          |
 | write.metadata.compression-codec                     | none                  
      | Metadata compression codec; none or gzip                                
                                                                                
                                          |
-| write.metadata.metrics.max-inferred-column-defaults  | 100                   
      | Defines the maximum number of top level columns for which metrics are 
collected. Number of stored metrics can be higher than this limit for a table 
with nested fields                            |
+| write.metadata.metrics.max-inferred-column-defaults  | 100                   
      | Defines the maximum number of columns for which metrics are collected. 
Top level primitive columns are counted first before nested columns             
                                           |

Review Comment:
   ```suggestion
   | write.metadata.metrics.max-inferred-column-defaults  | 100                 
        | Defines the maximum number of columns for which metrics are 
collected. Columns are included breadth first: top level fields first; then 
elements of nested structs; then elements of structs nested within those 
structs and so on.                               |
   ```



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to