You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by bl...@apache.org on 2022/07/01 15:34:41 UTC

[iceberg] branch master updated: Spec: Sort-order order-id is mandatory (#5177)

This is an automated email from the ASF dual-hosted git repository.

blue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/master by this push:
     new 509bdc1777 Spec: Sort-order order-id is mandatory (#5177)
509bdc1777 is described below

commit 509bdc1777615f88d15aec40d41eb9e4ba00b7ad
Author: Fokko Driesprong <fo...@apache.org>
AuthorDate: Fri Jul 1 17:34:36 2022 +0200

    Spec: Sort-order order-id is mandatory (#5177)
    
    Looking at the code, we expect this field:
    
    https://github.com/apache/iceberg/blob/master/core/src/main/java/org/apache/iceberg/SortOrderParser.java#L154
    
    And we always write it on the Java side.
---
 open-api/rest-catalog-open-api.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/open-api/rest-catalog-open-api.yaml b/open-api/rest-catalog-open-api.yaml
index 2ba3af6994..e4c6200a97 100644
--- a/open-api/rest-catalog-open-api.yaml
+++ b/open-api/rest-catalog-open-api.yaml
@@ -1068,6 +1068,7 @@ components:
     SortOrder:
       type: object
       required:
+        - order-id
         - fields
       properties:
         order-id: