amogh-jahagirdar opened a new pull request, #9134: URL: https://github.com/apache/iceberg/pull/9134
This change fixes some very confusing naming from my original implementation of the fastForward/replaceBranch APIs. Prior to this change, the APIs had the following naming: ``` fastForward(String name, String source) replaceBranch(String name, String source) ``` Name is the branch that would be fast forwarded and source is the "destination" we would fast forward to. This doesn't really align with how most people would think of a source/target when it comes to fast forward or replace operations (like Git). This change renames it to: ``` fastForward(String source, String target) replaceBranch(String source, String target) ``` Note: This is just a rename of parameters and error messages. There is no change in behavior in the API. -- 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
