aokolnychyi commented on code in PR #9813:
URL: https://github.com/apache/iceberg/pull/9813#discussion_r1561801551


##########
core/src/main/java/org/apache/iceberg/BaseFilesTable.java:
##########
@@ -46,15 +45,37 @@ abstract class BaseFilesTable extends BaseMetadataTable {
 
   @Override
   public Schema schema() {
-    StructType partitionType = Partitioning.partitionType(table());
+    // avoid returning an empty struct, which is not always supported.

Review Comment:
   I am a bit lost with all the schemas here. Let me see if we can simplify 
this block.



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to