jackye1995 commented on code in PR #11156:
URL: https://github.com/apache/iceberg/pull/11156#discussion_r1765495993


##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3138,6 +3149,8 @@ components:
           description: May be null if the table is staged as part of a 
transaction
         metadata:
           $ref: '#/components/schemas/TableMetadata'
+        planning-mode:
+          $ref: '#/components/schemas/PlanningMode'
         config:
           type: object

Review Comment:
   Looks like from the sync meeting, we are saying the planning mode should be 
in config, something like:
   
   ```
   config:
     type: object
     properties:
        planning-mode:
          $ref: '#/components/schemas/PlanningMode'
     additionalProperties:
       type: string
   ```
   
   Is this the right understanding?



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