Skip to content

Commit 8f4c4dd

Browse files
guojn1githubgxll
authored andcommitted
[fix][dingo-calcite] Fix the error message issue for alterTableColumn
1 parent 8379943 commit 8f4c4dd

2 files changed

Lines changed: 7 additions & 0 deletions

File tree

dingo-calcite/src/main/java/io/dingodb/calcite/DingoDdlExecutor.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1749,6 +1749,9 @@ public void execute(SqlAlterChangeColumn sqlAlterChangeColumn, CalcitePrepare.Co
17491749
}
17501750
}
17511751
DingoSqlColumn dingoSqlColumn = sqlAlterChangeColumn.dingoSqlColumn;
1752+
if (dingoSqlColumn == null) {
1753+
throw DingoErrUtil.newStdErr(ErrNotSupportedYet);
1754+
}
17521755
String name = dingoSqlColumn.name.getSimple();
17531756
Column column = table.getColumns().stream()
17541757
.filter(col -> col.getSchemaState() == SchemaState.SCHEMA_PUBLIC

dingo-exec/src/main/java/io/dingodb/exec/operator/DistributeOperator.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
import io.dingodb.codec.CodecService;
2020
import io.dingodb.codec.KeyValueCodec;
2121
import io.dingodb.common.CommonId;
22+
import io.dingodb.common.log.LogUtils;
2223
import io.dingodb.common.partition.RangeDistribution;
2324
import io.dingodb.common.util.ByteArrayUtils;
2425
import io.dingodb.common.util.Optional;
@@ -88,6 +89,9 @@ public boolean push(Context context, @Nullable Object[] tuple, Vertex vertex) {
8889
KeyValueCodec indexCodec = CodecService.getDefault()
8990
.createKeyValueCodec(indexTable.getCodecVersion(), indexTable.version,
9091
indexTable.tupleType(), indexTable.keyMapping());
92+
if (param.getDistributions().isEmpty() && "replicaTable".equalsIgnoreCase(indexTable.getName())) {
93+
return true;
94+
}
9195
partId = indexPs.calcPartId(indexTuple, wrap(indexCodec::encodeKey), param.getDistributions());
9296
NavigableMap<ByteArrayUtils.ComparableByteArray, RangeDistribution> distribution =
9397
MetaService.root().getRangeDistribution(param.getTable().tableId);

0 commit comments

Comments
 (0)