(gravitino) branch branch-1.2 updated: [Cherry-pick to branch-1.2] [#10496] improvement(lance): Clean up redundant code in GravitinoLanceTableOperations#createTable (#10497) (#10533)

[yuqi4733]

This is an automated email from the ASF dual-hosted git repository.

yuqi4733 pushed a commit to branch branch-1.2
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-1.2 by this push:
     new d9c652798a [Cherry-pick to branch-1.2] [#10496] improvement(lance): 
Clean up redundant code in GravitinoLanceTableOperations#createTable (#10497) 
(#10533)
d9c652798a is described below

commit d9c652798af67774ed35629aea8ebaa692ded4cf
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Tue Mar 24 20:31:44 2026 +0800

    [Cherry-pick to branch-1.2] [#10496] improvement(lance): Clean up redundant 
code in GravitinoLanceTableOperations#createTable (#10497) (#10533)
    
    **Cherry-pick Information:**
    - Original commit: 837e2b19f2c42e42eab6bb760647e7d32c1b5752
    - Target branch: `branch-1.2`
    - Status: ✅ Clean cherry-pick (no conflicts)
    
    Co-authored-by: Yuw1 <imwan...@126.com>
    Co-authored-by: Qi Yu <y...@datastrato.com>
---
 .../common/ops/gravitino/GravitinoLanceTableOperations.java | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git 
a/lance/lance-common/src/main/java/org/apache/gravitino/lance/common/ops/gravitino/GravitinoLanceTableOperations.java
 
b/lance/lance-common/src/main/java/org/apache/gravitino/lance/common/ops/gravitino/GravitinoLanceTableOperations.java
index b9e36ecb47..b3039a656d 100644
--- 
a/lance/lance-common/src/main/java/org/apache/gravitino/lance/common/ops/gravitino/GravitinoLanceTableOperations.java
+++ 
b/lance/lance-common/src/main/java/org/apache/gravitino/lance/common/ops/gravitino/GravitinoLanceTableOperations.java
@@ -158,19 +158,16 @@ public class GravitinoLanceTableOperations implements 
LanceTableOperations {
             .asTableCatalog()
             .createTable(
                 tableIdentifier, columns.toArray(new Column[0]), null, 
createTableProperties);
+    Map<String, String> properties = t.properties();
 
     CreateTableResponse response = new CreateTableResponse();
-    response.setProperties(t.properties());
-    response.setLocation(tableLocation);
     // Extract storage options from table properties. All storage options 
stores in table
     // properties.
-    
response.setStorageOptions(LancePropertiesUtils.getLanceStorageOptions(t.properties()));
+    
response.setStorageOptions(LancePropertiesUtils.getLanceStorageOptions(properties));
     response.setVersion(
-        Optional.ofNullable(t.properties().get(LANCE_TABLE_VERSION))
-            .map(Long::valueOf)
-            .orElse(null));
-    response.setLocation(t.properties().get(LANCE_LOCATION));
-    response.setProperties(t.properties());
+        
Optional.ofNullable(properties.get(LANCE_TABLE_VERSION)).map(Long::valueOf).orElse(null));
+    response.setLocation(properties.get(LANCE_LOCATION));
+    response.setProperties(properties);
     return response;
   }