Merge branch 'prestodb:master' into okie-fix

CONTRIBUTING.md

@@ -478,6 +478,7 @@ We use the [Fork and Pull model](https://docs.github.com/en/pull-requests/collab
 * Ask reviewers to provide examples or code references without trying to find them on your own
 * Protest lack of documentation for a code section
     * Instead, review the related code, then draft initial documentation as a separate commit
+* Use AI tools to generate documentation
 * Submit without test cases or clear justification for lack thereof
 * Request review when there are tests failing
 

presto-iceberg/src/test/java/com/facebook/presto/iceberg/rest/TestIcebergSmokeRestNestedNamespace.java

@@ -339,10 +339,12 @@ public void testView()
         assertUpdate(session, "CREATE VIEW view_orders AS SELECT * from orders");
         assertQuery(session, "SELECT * FROM view_orders", "SELECT * from orders");
         assertThat(computeActual("SHOW CREATE VIEW view_orders").getOnlyValue())
-                .isEqualTo(format("CREATE VIEW iceberg.\"%s\".view_orders AS\n" +
-                        "SELECT *\n" +
-                        "FROM\n" +
-                        "  orders", schemaName));
+                .isEqualTo(format("CREATE VIEW iceberg.\"%s\".view_orders SECURITY %s AS\n" +
+                                "SELECT *\n" +
+                                "FROM\n" +
+                                "  orders",
+                        schemaName,
+                        "DEFINER"));
         assertUpdate(session, "DROP VIEW view_orders");
     }
 

presto-main/src/main/java/com/facebook/presto/sql/rewrite/ShowQueriesRewrite.java

@@ -472,7 +472,8 @@ protected Node visitShowCreate(ShowCreate node, Void context)
                 }
 
                 Query query = parseView(viewDefinition.get().getOriginalSql(), objectName, node);
-                String sql = formatSql(new CreateView(createQualifiedName(objectName), query, false, Optional.empty()), Optional.of(parameters)).trim();
+                CreateView.Security security = (viewDefinition.get().isRunAsInvoker()) ? CreateView.Security.INVOKER : CreateView.Security.DEFINER;
+                String sql = formatSql(new CreateView(createQualifiedName(objectName), query, false, Optional.of(security)), Optional.of(parameters)).trim();
                 return singleValueQuery("Create View", sql);
             }
 

presto-parser/src/main/java/com/facebook/presto/sql/SqlFormatter.java

@@ -583,8 +583,7 @@ protected Void visitCreateView(CreateView node, Integer indent)
 
             node.getSecurity().ifPresent(security ->
                     builder.append(" SECURITY ")
-                            .append(security.toString())
-                            .append(" "));
+                            .append(security.toString()));
 
             builder.append(" AS\n");
 

presto-spark-base/src/test/java/com/facebook/presto/spark/TestPrestoSparkQueryRunner.java

@@ -1202,7 +1202,7 @@ public void testCreateAlterTable()
     public void testCreateDropView()
     {
         // create table with default format orc
-        String createViewSql = "CREATE VIEW hive.hive_test.hive_view AS\n" +
+        String createViewSql = "CREATE VIEW hive.hive_test.hive_view SECURITY DEFINER AS\n" +
                 "SELECT *\n" +
                 "FROM\n" +
                 "  orders";

presto-tests/src/main/java/com/facebook/presto/tests/AbstractTestDistributedQueries.java

@@ -968,10 +968,11 @@ public void testViewMetadata()
 
         // test SHOW CREATE VIEW
         String expectedSql = formatSqlText(format(
-                "CREATE VIEW %s.%s.%s AS %s",
+                "CREATE VIEW %s.%s.%s SECURITY %s AS %s",
                 getSession().getCatalog().get(),
                 getSession().getSchema().get(),
                 "meta_test_view",
+                "DEFINER",
                 query)).trim();
 
         actual = computeActual("SHOW CREATE VIEW meta_test_view");