[incubator-paimon] branch master updated: [format] Fix the issue of OrcWriterFactoryTest#testNotOverrideInMemoryManager NullPointerException in unit test. (#921) (#922)

[lzljs3620320]

This is an automated email from the ASF dual-hosted git repository.

lzljs3620320 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-paimon.git


The following commit(s) were added to refs/heads/master by this push:
     new 056435798 [format] Fix the issue of 
OrcWriterFactoryTest#testNotOverrideInMemoryManager NullPointerException in 
unit test. (#921) (#922)
056435798 is described below

commit 056435798f22f6d70412376385d9b5d1f4434148
Author: liming.1018 <69444450+limin...@users.noreply.github.com>
AuthorDate: Thu Apr 20 12:08:45 2023 +0800

    [format] Fix the issue of 
OrcWriterFactoryTest#testNotOverrideInMemoryManager NullPointerException in 
unit test. (#921) (#922)
---
 .../src/main/java/org/apache/paimon/format/orc/OrcWriterFactory.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/paimon-format/src/main/java/org/apache/paimon/format/orc/OrcWriterFactory.java
 
b/paimon-format/src/main/java/org/apache/paimon/format/orc/OrcWriterFactory.java
index 5ed8b6d04..7299d48e9 100644
--- 
a/paimon-format/src/main/java/org/apache/paimon/format/orc/OrcWriterFactory.java
+++ 
b/paimon-format/src/main/java/org/apache/paimon/format/orc/OrcWriterFactory.java
@@ -71,7 +71,7 @@ public class OrcWriterFactory implements FormatWriterFactory {
      *     VectorizerRowBatch.
      */
     public OrcWriterFactory(Vectorizer<InternalRow> vectorizer, Configuration 
configuration) {
-        this(vectorizer, null, configuration);
+        this(vectorizer, new Properties(), configuration);
     }
 
     /**
@@ -87,7 +87,7 @@ public class OrcWriterFactory implements FormatWriterFactory {
             Properties writerProperties,
             Configuration configuration) {
         this.vectorizer = checkNotNull(vectorizer);
-        this.writerProperties = writerProperties;
+        this.writerProperties = checkNotNull(writerProperties);
         this.confMap = new HashMap<>();
 
         // Todo: Replace the Map based approach with a better approach