nsivabalan commented on code in PR #8111: URL: https://github.com/apache/hudi/pull/8111#discussion_r1169353294
########## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java: ########## @@ -2971,13 +2977,10 @@ private void autoAdjustConfigsForConcurrencyMode(boolean isLockProviderPropertyS // This is targeted at Single writer with async table services // If user does not set the lock provider, likely that the concurrency mode is not set either // Override the configs for metadata table - writeConfig.setValue(WRITE_CONCURRENCY_MODE.key(), - WriteConcurrencyMode.OPTIMISTIC_CONCURRENCY_CONTROL.value()); writeConfig.setValue(HoodieLockConfig.LOCK_PROVIDER_CLASS_NAME.key(), InProcessLockProvider.class.getName()); - LOG.info(String.format("Automatically set %s=%s and %s=%s since user has not set the " + LOG.info(String.format("Automatically set %s=%s since user has not set the " Review Comment: why removing concurrency mode ? when metadata is enabled and if there any async tasble services, we want to enable OCC (i.e. write concurrency mode, in process lock provider, lazy rollack clean policy). -- 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