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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to