This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git
The following commit(s) were added to refs/heads/master by this push:
new e426fbd [Planner] Align Date and Datetime by 16 bytes (#6105)
e426fbd is described below
commit e426fbd6bed9c4245bc85807c5bd39c2b948f720
Author: EmmyMiao87 <[email protected]>
AuthorDate: Sat Jul 10 10:10:04 2021 +0800
[Planner] Align Date and Datetime by 16 bytes (#6105)
The previous alignment of Doris is up to 8 bytes.
For types with more than 8 bytes, such as Date, Datetime is not aligned.
This PR is mainly to relax the maximum 8-byte limitation
Also, because the data type Decimal v1 no longer exists,
the logic of the 40-byte Decimal v1 is also discarded.
---
.../src/main/java/org/apache/doris/analysis/TupleDescriptor.java | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/analysis/TupleDescriptor.java
b/fe/fe-core/src/main/java/org/apache/doris/analysis/TupleDescriptor.java
index 1cfdf47..0b4553c 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/analysis/TupleDescriptor.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/TupleDescriptor.java
@@ -203,10 +203,7 @@ public class TupleDescriptor {
}
if (slotSize > 1) {
// insert padding
- int alignTo = Math.min(slotSize, 8);
- if (slotSize == 40) {
- alignTo = 4;
- }
+ int alignTo = slotSize;
offset = (offset + alignTo - 1) / alignTo * alignTo;
}
@@ -233,7 +230,6 @@ public class TupleDescriptor {
}
this.byteSize = offset;
- // LOG.debug("tuple is {}", byteSize);
}
/**
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
