[GitHub] [ignite-3] Berkof commented on a change in pull request #401: IGNITE-13667: Add schema columns mapping from user order to system order

[GitBox <gi...@apache.org>]

Berkof commented on a change in pull request #401:
URL: https://github.com/apache/ignite-3/pull/401#discussion_r731518449



##########
File path: modules/schema/src/main/java/org/apache/ignite/internal/schema/Column.java
##########
@@ -24,15 +24,20 @@
 import org.jetbrains.annotations.NotNull;
 
 /**
- * Column description for a type schema. Column contains a column name, a column type and a nullability flag.
+ * Column descriptor which contains a column name, a type and a nullability flag.
  * <p>
  * Column instances are comparable in lexicographic order, native type first and then column name. Nullability
  * flag is not taken into account when columns are compared.
+ * <p>
+ * Because of columns must be written to a row in specific order, column write order may differs from the user order

Review comment:
       ```suggestion
    * Because of columns must be written to a row in specific order, column write order may differs from the user order.
   ```




-- 
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: notifications-unsubscribe@ignite.apache.org

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