[GitHub] [hudi] sydneyhoran commented on pull request #6726: [HUDI-4630] Add transformer capability to individual feeds in MultiTableDeltaStreamer

["sydneyhoran (via GitHub)" <gi...@apache.org>]

sydneyhoran commented on PR #6726:
URL: https://github.com/apache/hudi/pull/6726#issuecomment-1468270289

   Just a quick heads up - I ran the latest master MultiTableDeltaStreamer without `hoodie.deltastreamer.transformer.class` config and I got a NullPointerException due to the .split() in line:
   ```java
   List<String> transformerClassNameOverride = Arrays.asList(typedProperties.getString(Constants.TRANSFORMER_CLASS, null).split(","));
   ```
   because my `TRANSFORMER_CLASS` is null. I changed the function to the following with success:
   ```java
     private void populateTransformerProps(HoodieDeltaStreamer.Config cfg, TypedProperties typedProperties) {
       String transformerClass = typedProperties.getString(Constants.TRANSFORMER_CLASS, null);
       if (transformerClass != null && !transformerClass.trim().isEmpty()) {
         List<String> transformerClassNameOverride = Arrays.asList(transformerClass.split(","));
         cfg.transformerClassNames = transformerClassNameOverride;
       }
     }
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org