swamirishi commented on PR #7468: URL: https://github.com/apache/ozone/pull/7468#issuecomment-2495765441
> QQ, why aren't we bump the [client version](https://github.com/apache/ozone/blob/master/hadoop-hdds/common/src/main/java/org/apache/hadoop/ozone/ClientVersion.java) instead to (client supports server side liststatus limits or something like that), so that we can identify older clients that does not know about/respect the server side limit, and then we can treat these clients from the server side? > > At least my understanding is that about the current draft that it pushes this client information via a request metadata instead of with the client's version, but please correct me if I am wrong. You are right we can achieve this by a client version bump and that is eventual goal. But because our request feature validator is broken we cannot bump up our client version. We already have a patch upstream which fixes that https://github.com/apache/ozone/pull/6932 . I thought to have a patch in the interim. But after discussing about this offline with @errose28 @umamaheswararao @swagle we came to a conclusion to abandon this change and work on fixing the request feature validator and perform a client version bump up instead. -- 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]
