yihua opened a new pull request, #8080:
URL: https://github.com/apache/hudi/pull/8080

   ### Change Logs
   
   In 0.13.0 and latest master, the table service client 
`BaseHoodieTableServiceClient` is instantiated without any timeline server 
instance, even if the regular write client has one.  This causes the table 
service client to start a new embedded timeline server and overwrite the write 
config passed in from the constructor so that the write config points to the 
newly started timeline server.
   
   As the regular write client such as `SparkRDDWriteClient` directly passes in 
the same writeConfig instance, the regular write client's write config is also 
affected, causing the regular write client to use the newly started embedded 
timeline server always, instead of the timeline server instance passed in from 
the constructor or the one instantiated by the regular write client itself.
   
   This means that the Deltastreamer's long-lived timeline server is never 
going to be used because of this issue.
   
   This PR fixes the issue by properly initiating the table service client with 
the timeline server from the regular write client.
   
   ### Impact
   
   This makes sure that the timeline server instance passed to the regular 
write client is used by the write transaction.
   
   ### Risk level
   
   medium
   
   ### Documentation Update
   
   N/A
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Change Logs and Impact were stated clearly
   - [ ] Adequate tests were added if applicable
   - [ ] CI passed
   


-- 
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-unsubscr...@hudi.apache.org

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

Reply via email to