Fix failing tests - I

Author: zsmiskolczi

iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergMetaHook.java

@@ -409,14 +409,20 @@ private void handleAlterTableOperationBasedOnType(
       handleChangeColumn(hmsTable);
     } else {
       setWriteModeDefaults(
-          icebergTable != null ?
-              icebergTable :
-              icebergMaterializedView.getStorageTable(),
+          getIcebergTbl(),
           hmsTable.getParameters(), context);
       assertNotCrossTableMetadataLocationChange(hmsTable.getParameters(), context);
     }
   }
 
+  private Table getIcebergTbl() {
+    if (icebergMaterializedView != null) {
+      return icebergMaterializedView.getStorageTable();
+    }
+
+    return icebergTable;
+  }
+
   /**
    * Perform a check on the current iceberg table whether a metadata change can be performed. A table is eligible if
    * the current metadata uuid and the new metadata uuid matches.

iceberg/patched-iceberg-core/src/main/java/org/apache/iceberg/view/BaseMetastoreViewCatalog.java

@@ -22,6 +22,7 @@
 import java.util.Arrays;
 import java.util.List;
 import java.util.Map;
+import org.apache.commons.lang3.StringUtils;
 import org.apache.iceberg.BaseMetastoreCatalog;
 import org.apache.iceberg.CatalogProperties;
 import org.apache.iceberg.EnvironmentContext;
@@ -214,9 +215,16 @@ private View create(ViewOperations ops) {
       Preconditions.checkState(
               null != defaultNamespace, "Cannot create view without specifying a default namespace");
 
-      StorageTable storageTable = ImmutableStorageTable.of(
-              Arrays.asList(this.identifier.namespace().levels()), storageTableName
-      );
+      StorageTable storageTable = null;
+      if (!StringUtils.isEmpty(storageTableName)) {
+        ImmutableStorageTable.of(
+            Arrays.asList(this.identifier.namespace().levels()), storageTableName
+        );
+      }
+
+      if (createTime == null) {
+        createTime = System.currentTimeMillis();
+      }
 
       ViewVersion viewVersion =
               ImmutableViewVersion.builder()

iceberg/patched-iceberg-core/src/main/java/org/apache/iceberg/view/ViewVersionParser.java

@@ -141,8 +141,8 @@ public static ViewVersion fromJson(JsonNode node) {
             Namespace.of(JsonUtil.getStringArray(JsonUtil.get(DEFAULT_NAMESPACE, node)));
 
     StorageTable storageTable = null;
-    JsonNode serializedStorageTable = JsonUtil.get(STORAGE_TABLE, node);
-    if (serializedStorageTable != null) {
+    if (node.hasNonNull(STORAGE_TABLE)) {
+      JsonNode serializedStorageTable = JsonUtil.get(STORAGE_TABLE, node);
       JsonNode serializedNamespace = JsonUtil.get(NAMESPACE, serializedStorageTable);
       Preconditions.checkArgument(
           serializedNamespace.isArray(),
@@ -154,8 +154,8 @@ public static ViewVersion fromJson(JsonNode node) {
     }
 
     RefreshState refreshState = null;
-    JsonNode serializedRefreshState = JsonUtil.get(REFRESH_STATE, node);
-    if (serializedRefreshState != null) {
+    if (node.hasNonNull(REFRESH_STATE)) {
+      JsonNode serializedRefreshState = JsonUtil.get(REFRESH_STATE, node);
       Integer viewVersionId = JsonUtil.getInt(VIEW_VERSION_ID, serializedRefreshState);
 
       JsonNode serializedSourceStates = JsonUtil.get(SOURCE_STATES, serializedRefreshState);