rdblue commented on a change in pull request #3810:
URL: https://github.com/apache/iceberg/pull/3810#discussion_r776490107



##########
File path: core/src/main/java/org/apache/iceberg/TableProperties.java
##########
@@ -111,6 +111,12 @@ private TableProperties() {
   public static final String DELETE_AVRO_COMPRESSION = 
"write.delete.avro.compression-codec";
   public static final String AVRO_COMPRESSION_DEFAULT = "gzip";
 
+  public static final String ORC_STRIPE_SIZE_BYTES = 
"write.orc.stripe-size-bytes";
+  public static final long ORC_STRIPE_SIZE_BYTES_DEFAULT = 64L * 1024 * 1024; 
// 64 MB
+
+  public static final String ORC_BLOCK_SIZE_BYTES = 
"write.orc.block-size-bytes";

Review comment:
       @omalley or @shardulm94, is this a config that we should expose as an 
Iceberg table setting? Is it still needed for object stores?




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