You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/07/05 08:33:45 UTC
[GitHub] [incubator-druid] gianm commented on a change in pull request
#6794: Query vectorization.
gianm commented on a change in pull request #6794: Query vectorization.
URL: https://github.com/apache/incubator-druid/pull/6794#discussion_r300589926
##########
File path: processing/src/main/java/org/apache/druid/segment/data/CompressedVSizeColumnarMultiIntsSupplier.java
##########
@@ -38,17 +38,21 @@
* Format -
* byte 1 - version
* offsets - {@link ColumnarInts} of length num of rows + 1 representing offsets of starting index of first element of
- * each row in values index and last element equal to length of values column, the last element in the offsets
- * represents the total length of values column.
+ * each row in values index and last element equal to length of values column, the last element in the offsets
Review comment:
It does sound like the same thing. I actually didn't mean to change anything here, it was just IDE autoformatting. I restored it.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org