You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by da...@apache.org on 2025/10/27 17:42:23 UTC
(doris) branch master updated: [fix](nereids) fix copy into load orc/parquet format fail (#57362)
This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new fba8f6a5b1b [fix](nereids) fix copy into load orc/parquet format fail (#57362)
fba8f6a5b1b is described below
commit fba8f6a5b1bd06f2835d9f4717256a02f2317621
Author: hui lai <la...@selectdb.com>
AuthorDate: Tue Oct 28 01:42:16 2025 +0800
[fix](nereids) fix copy into load orc/parquet format fail (#57362)
### What problem does this PR solve?
introduced by https://github.com/apache/doris/issues/50471
Fix copy into load orc/parquet format fail:
```
mysql> copy into orc_test from (select p_partkey, p_name, p_mfgr, p_brand, p_type, p_size from @regression_test_customer('copy_into/orc/part.orc') ) properties ('file.type' = 'orc', 'copy.async' = 'false');
ERROR 1105 (HY000): errCode = 2, detailMessage = Cannot invoke "String.toLowerCase()" because the return value of "java.util.Map.get(Object)" is null
```
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR
should merge into -->
---
.../apache/doris/nereids/trees/plans/commands/info/CopyIntoInfo.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CopyIntoInfo.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CopyIntoInfo.java
index 330cfb60319..3d5b74b133d 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CopyIntoInfo.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/CopyIntoInfo.java
@@ -266,7 +266,10 @@ public class CopyIntoInfo {
analyzer, context, cascadesContext);
}
- dataDescProperties.put(FileFormatProperties.PROP_COMPRESS_TYPE, copyIntoProperties.getCompression());
+ String compression = copyIntoProperties.getCompression();
+ if (compression != null) {
+ dataDescProperties.put(FileFormatProperties.PROP_COMPRESS_TYPE, compression);
+ }
dataDescription = new DataDescription(tableNameInfo.getTbl(), null, Lists.newArrayList(filePath),
copyFromDesc.getFileColumns(), separator, fileFormatStr, null, false,
legacyColumnMappingList, legacyFileFilterExpr, null, LoadTask.MergeType.APPEND, null,
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org