difin commented on code in PR #6449:
URL: https://github.com/apache/hive/pull/6449#discussion_r3269944324


##########
ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveStorageHandler.java:
##########
@@ -1028,6 +1030,47 @@ default void setMergeTaskDeleteProperties(TableDesc 
tableDesc) {
     throw new UnsupportedOperationException("Storage handler does not support 
getting custom delete merge schema.");
   }
 
+  /**
+   * @return {@code true} if this handler may store CREATE VIEW text and 
column metadata in an external catalog
+   *         (native view) rather than only as a classic HMS virtual view.
+   */
+  default boolean supportsNativeViewCatalog() {
+    return false;
+  }
+
+  /**
+   * HMS table-level parameters to set on the metastore stub for a 
native-catalog view (markers, etc.).
+   * Keys should be removed when {@linkplain 
#clearNativeViewHmsTableProperties(Map)} is invoked for the same
+   * handler class recorded under {@link 
Constants.NATIVE_VIEW_STORAGE_HANDLER_CLASS_PARAM}.
+   */
+  default Map<String, String> getNativeViewHmsTableProperties() {

Review Comment:
   I agree that the word "native" was misleading outside of Iceberg handler.
   Removed/rephrased the "iceberg native views" names that were used outside of 
the iceberg handler.



-- 
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]

Reply via email to