n3nash commented on a change in pull request #2359:
URL: https://github.com/apache/hudi/pull/2359#discussion_r569149254



##########
File path: 
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/AbstractHoodieWriteClient.java
##########
@@ -128,11 +133,26 @@ public AbstractHoodieWriteClient(HoodieEngineContext 
context, HoodieWriteConfig
    * @param rollbackPending whether need to cleanup pending commits
    * @param timelineService Timeline Service that runs as part of write client.
    */
+  @Deprecated
   public AbstractHoodieWriteClient(HoodieEngineContext context, 
HoodieWriteConfig writeConfig, boolean rollbackPending,

Review comment:
       That would be a backwards incompatible change. I'm OK to remove it if 
it's just for Uber. 




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to