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