difin commented on code in PR #6449:
URL: https://github.com/apache/hive/pull/6449#discussion_r3314457960
##########
ql/src/java/org/apache/hadoop/hive/ql/ddl/view/create/CreateViewOperation.java:
##########
@@ -82,6 +84,10 @@ public int execute() throws HiveException {
if (desc.getProperties() != null) {
oldview.getTTable().getParameters().putAll(desc.getProperties());
}
+ if (!desc.usesStorageHandler()) {
+ // External logical view is replaced with a native Hive view
+ clearStorageHandlerProp(oldview);
+ }
Review Comment:
Added this test case to `iceberg_native_logical_view.q`:
`v_ice` before replacement is Iceberg-native logical view that is replaced
with Hive-native logical view:
```
create or replace view v_ice
as select first_name from src_ice where dept_id = 2;
select * from v_ice;
desc formatted v_ice;
drop view v_ice;
```
--
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]
