errose28 commented on PR #9994: URL: https://github.com/apache/ozone/pull/9994#issuecomment-4156468925
Thanks for adding this @ArafatKhan2198. Do we need the new configuration `ozone.recon.csv.export.max.records`? This is an admin only endpoint with authorization, so I don't think we need to worry about intentionally admins abusing this API. I think the configuration may create confusion if the number of records exceeds the default limit. -- 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]
