You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/08/01 03:29:45 UTC

[GitHub] [spark] cloud-fan commented on a diff in pull request #37340: [MINOR][SQL] Improve the comments about null tracking for UnsafeRow

cloud-fan commented on code in PR #37340:
URL: https://github.com/apache/spark/pull/37340#discussion_r934111262


##########
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java:
##########
@@ -46,10 +46,10 @@
 /**
  * An Unsafe implementation of Row which is backed by raw memory instead of Java objects.
  *
- * Each tuple has three parts: [null bit set] [values] [variable length portion]
+ * Each tuple has three parts: [null-tracking portion] [values] [variable length portion]

Review Comment:
   ```suggestion
    * Each tuple has three parts: [null-tracking bit set] [values] [variable length portion]
   ```



-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org