yuw1 opened a new issue, #10496: URL: https://github.com/apache/gravitino/issues/10496
### What would you like to be improved? The `createTable` method in `GravitinoLanceTableOperations.java` contains redundant setter calls during the response construction: 1. **Redundant `setLocation`**: `response.setLocation(tableLocation)` is called initially but is immediately overwritten by the actual table property at the end. 2. **Duplicate `setProperties`**: `response.setProperties(t.properties())` is called twice. ### How should we improve? Refactor the response construction to remove these "dead store" assignments: 1. **Remove the first `setLocation(tableLocation)` call.** 2. **Remove the first `setProperties(t.properties())` call.** 3. **Use a local variable** for `t.properties()` to unify the final assignments for `properties`, `location`, `storageOptions`, and `version`. -- 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]
