Skip to content

Commit

Permalink
[opt](paimon)Remove unnecessary fields (apache#44906)
Browse files Browse the repository at this point in the history
### What problem does this PR solve?

Related PR: apache#44274

Problem Summary:

We have already set the serialization table in `TFileScanRangeParams`,
so there is no need to repeat it for each split.
  • Loading branch information
wuwenchi authored Dec 6, 2024
1 parent de2ce48 commit 0f74869
Showing 1 changed file with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -183,9 +183,7 @@ private void setPaimonParams(TFileRangeDesc rangeDesc, PaimonSplit paimonSplit)
fileDesc.setDbId(((PaimonExternalTable) source.getTargetTable()).getDbId());
fileDesc.setTblId(source.getTargetTable().getId());
fileDesc.setLastUpdateTime(source.getTargetTable().getUpdateTime());
fileDesc.setPaimonTable(encodeObjectToString(source.getPaimonTable()));
// The hadoop conf should be same with
// PaimonExternalCatalog.createCatalog()#getConfiguration()
// The hadoop conf should be same with PaimonExternalCatalog.createCatalog()#getConfiguration()
fileDesc.setHadoopConf(source.getCatalog().getCatalogProperty().getHadoopProperties());
Optional<DeletionFile> optDeletionFile = paimonSplit.getDeletionFile();
if (optDeletionFile.isPresent()) {
Expand Down

0 comments on commit 0f74869

Please sign in to comment.