You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@phoenix.apache.org by GitBox <gi...@apache.org> on 2020/09/11 05:08:27 UTC
[GitHub] [phoenix] kadirozde commented on a change in pull request #875: PHOENIX-6120 Covered column handling for SINGLE_CELL_ARRAY storage sc…
kadirozde commented on a change in pull request #875:
URL: https://github.com/apache/phoenix/pull/875#discussion_r486776349
##########
File path: phoenix-core/src/main/java/org/apache/phoenix/index/IndexMaintainer.java
##########
@@ -1030,35 +1032,40 @@ public Put buildUpdateMutation(KeyValueBuilder kvBuilder, ValueGetter valueGette
for (Pair<ColumnReference, ColumnReference> colRefPair : colRefPairs) {
ColumnReference indexColRef = colRefPair.getFirst();
ColumnReference dataColRef = colRefPair.getSecond();
- Expression expression = new SingleCellColumnExpression(new PDatum() {
- @Override
- public boolean isNullable() {
- return false;
- }
-
- @Override
- public SortOrder getSortOrder() {
- return null;
- }
-
- @Override
- public Integer getScale() {
- return null;
- }
-
- @Override
- public Integer getMaxLength() {
- return null;
- }
-
- @Override
- public PDataType getDataType() {
- return null;
+
+ byte[] value = null;
+ if (this.dataTableImmutableStorageScheme == this.immutableStorageScheme) {
Review comment:
It is not clear to me how this extra check is useful here and how it fixes the issue. Why is this check necessary, given that the index is supposed to inherit the storage scheme from the data table currently? Is it possible that they can be different.
----------------------------------------------------------------
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