zhjwpku commented on PR #187: URL: https://github.com/apache/iceberg-cpp/pull/187#issuecomment-3210964444
> Enhanced the forward declaration system in `type_fwd.h` to reduce compilation dependencies and improve build times. > > ## Changes Made > * Added missing forward declarations for commonly used classes > * Reorganized declarations into logical groups with descriptive comments > * Removed outdated TODO comment regarding (These classes were already properly declared): I don't think this is true, so I'd prefer to leave the TODO comment as is. > > * `StructLike` > * `MetadataUpdate` > * `UpdateRequirement` > * `AppendFiles` Other than that, the PR looks good. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
