zhjwpku commented on issue #14: URL: https://github.com/apache/iceberg-cpp/issues/14#issuecomment-2586231279
While looking the std::expected usage, I'm thinking no matter we choose the std::expected way or Arrow's Result way, it is no harm that we add a Status data structure, it's useful in the following two cases: 1. directly return Status, example: `Status Close();`, the returned value can state whether the call success or failed for some reason. 2. return std::expected<value_type, Status> or implicitly wrapped by Result<value_type>, example: `std::expected<int64_t, Status> Tell()`, either return the current access position of a opened file or the failure reason. Thought? @wgtmac @pitrou @gaborkaszab @mapleFU -- 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
