Skip to content

Commit 50d119f

Browse files
shangxinliclaude
andcommitted
docs: restore builder pattern context in PendingUpdate documentation
The builder pattern context explains why constructors are protected - subclasses should be created through their builder methods, not directly instantiated. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 3f087ef commit 50d119f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/iceberg/pending_update.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828

2929
namespace iceberg {
3030

31-
/// \brief Non-template base class for table metadata changes
31+
/// \brief Non-template base class for table metadata changes using builder pattern
3232
///
3333
/// This base class allows storing different types of PendingUpdate operations
3434
/// in the same collection (e.g., in Transaction). It provides the common Commit()
@@ -62,7 +62,7 @@ class ICEBERG_EXPORT PendingUpdateBase {
6262
PendingUpdateBase() = default;
6363
};
6464

65-
/// \brief Template class for type-safe table metadata changes
65+
/// \brief Template class for type-safe table metadata changes using builder pattern
6666
///
6767
/// PendingUpdate extends PendingUpdateBase with a type-safe Apply() method that
6868
/// returns the specific result type for each operation. Subclasses implement

0 commit comments

Comments
 (0)