[GitHub] [orc] deshanxiao commented on a diff in pull request #1465: ORC-1409: [Docs] Add stream order description in ORC spec.

["deshanxiao (via GitHub)" <gi...@apache.org>]

deshanxiao commented on code in PR #1465:
URL: https://github.com/apache/orc/pull/1465#discussion_r1166401368


##########
site/specification/ORCv1.md:
##########
@@ -886,6 +886,12 @@ uses three streams PRESENT, DATA, and LENGTH, which stores the length
 of each value. The details of each type will be presented in the
 following subsections.
 
+Note that the order of these streams is **not fixed**. In the example
+of the integer column mentioned above, the order of the PRESENT stream
+and the DATA stream cannot be determined in advance. Instead, we need
+to determine the type of stream based on the Stream Kind, rather than

Review Comment:
   Thank you @wgtmac . Updated.



-- 
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-unsubscribe@orc.apache.org

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