[GitHub] [hudi] nsivabalan commented on a change in pull request #2654: [HUDI-1615] [WIP] diff to fix null target schema in delastreamer

[GitBox <gi...@apache.org>]

nsivabalan commented on a change in pull request #2654:
URL: https://github.com/apache/hudi/pull/2654#discussion_r591130448



##########
File path: hudi-utilities/src/main/java/org/apache/hudi/utilities/deltastreamer/DeltaSync.java
##########
@@ -352,6 +356,10 @@ public void refreshTimeline() throws IOException {
       Option<Dataset<Row>> transformed =
           dataAndCheckpoint.getBatch().map(data -> transformer.get().apply(jssc, sparkSession, data, props));
       checkpointStr = dataAndCheckpoint.getCheckpointForNextBatch();
+      if(transformed.isPresent()){

Review comment:
       @vinothchandar : do you think we could do something like this. store the df schema and use it later when we initialize the writeclient config, where in we use this schema when target schema is null ? 
   This is just WIP diff. haven't fully completed yet. but should give you an idea. 




----------------------------------------------------------------
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.

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