HuaHuaY commented on code in PR #687:
URL: https://github.com/apache/iceberg-cpp/pull/687#discussion_r3322982125
##########
src/iceberg/manifest/manifest_merge_manager.h:
##########
@@ -56,6 +57,9 @@ class ICEBERG_EXPORT ManifestMergeManager {
ManifestMergeManager(const ManifestMergeManager&) = delete;
ManifestMergeManager& operator=(const ManifestMergeManager&) = delete;
+ /// \brief Configure an optional executor for manifest merging.
+ ManifestMergeManager& PlanWith(OptionalExecutor executor);
Review Comment:
The documentation is on the Executor interface. There are too many
`PlanWith` and putting comments there will result in too much redundancy.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
