amogh-jahagirdar commented on issue #6930:
URL: https://github.com/apache/iceberg/issues/6930#issuecomment-1444088767
I think it's expected, although do agree the name is a bit confusing. In
most cases (I think) we want to just iterate over the current and the ancestor
lineage, having the API return an Iterable which is inclusive makes it easier
to iterate over (rather than have a variable for the current, and then loop
over the parent).
If there's a valid use case for excluding the current, perhaps another API
which accepts a `boolean includeCurrent` could be offered. Then for
compatibility purpose the original API calls the new API with `includeCurrent =
true`
--
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]
