agnes-xinyi-lu opened a new issue, #9936: URL: https://github.com/apache/gravitino/issues/9936
### What would you like to be improved? in this earlier [PR](https://github.com/apache/gravitino/commit/62ed9c34dc0ae9996eb943ab381b46ec15c24584) Gravitino changed namespaces decoding from `percent-encoding` to `application/x-www-form-urlencoded`. In Iceberg [rest client side](https://github.com/apache/iceberg/blob/d95d9f0ad8baacd1a61331be9c5e09587a89f1f2/core/src/main/java/org/apache/iceberg/rest/ResourcePaths.java#L111) it always uses URLEncoder for both namespace and table strings, should we make it unified and do URLDecoding on table name as well? We are hitting some issues when switching traffic from Hive to rest catalog, because some workloads added trailing spaces to the table name, which gets automatically handled by HMS, but through rest catalog, the empty space could not be decoded by gravitino server side, thus can't pass the original table string plus the empty space to HMS, causing "Table Not Exist" error on client side. While it's arguable whether we should allow client side to add random empty spaces to the table name during read, but it does feel like a caveat on the server side to not decode the same way client side encodes. ### How should we improve? Same way as how namespaces are decoded right now. -- 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]
