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

pingtimeout pushed a commit to branch benchmarks-ppc
in repository https://gitbox.apache.org/repos/asf/polaris-tools.git

commit 7fc222f9dd7c229f357face9b749b3d32415213a
Author: Pierre Laporte <pie...@pingtimeout.fr>
AuthorDate: Mon May 26 10:06:50 2025 +0200

    Update action names for more readable reports
    
    This commit updates the names that are used in Gatling reports so that
    they are shorter and consistent across different actions.  Naming rules
    are:
    
    * `Create XXX`
    * `Fetch single XXX`
    * `Fetch children XXX` - when applicable, fetch all entities under a
      parent entity (i.e. a parent Namespace)
    * `Update XXX`
    * `Check XXX exists`
---
 .../apache/polaris/benchmarks/actions/NamespaceActions.scala   | 10 +++++-----
 .../org/apache/polaris/benchmarks/actions/TableActions.scala   | 10 +++++-----
 .../org/apache/polaris/benchmarks/actions/ViewActions.scala    | 10 +++++-----
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git 
a/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/NamespaceActions.scala
 
b/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/NamespaceActions.scala
index 7a5f5f0..5872618 100644
--- 
a/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/NamespaceActions.scala
+++ 
b/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/NamespaceActions.scala
@@ -147,7 +147,7 @@ case class NamespaceActions(
    * Typically, start 1 user and increase by 1 user every second until some 
arbitrary maximum value.
    */
   val createNamespace: ChainBuilder =
-    retryOnHttpStatus(maxRetries, retryableHttpCodes, "Create namespace")(
+    retryOnHttpStatus(maxRetries, retryableHttpCodes, "Create Namespace")(
       http("Create Namespace")
         .post("/api/catalog/v1/#{catalogName}/namespaces")
         .header("Authorization", "Bearer #{accessToken}")
@@ -173,7 +173,7 @@ case class NamespaceActions(
    * There is no limit to the number of users that can fetch namespaces 
concurrently.
    */
   val fetchNamespace: ChainBuilder = exec(
-    http("Fetch Namespace")
+    http("Fetch single Namespace")
       
.get("/api/catalog/v1/#{catalogName}/namespaces/#{namespaceMultipartPath}")
       .header("Authorization", "Bearer #{accessToken}")
       .check(status.is(200))
@@ -206,15 +206,15 @@ case class NamespaceActions(
    * structure.
    */
   val fetchAllChildrenNamespaces: ChainBuilder = exec(
-    http("Fetch all Namespaces under specific parent")
+    http("Fetch children Namespaces")
       
.get("/api/catalog/v1/#{catalogName}/namespaces?parent=#{namespaceMultipartPath}")
       .header("Authorization", "Bearer #{accessToken}")
       .check(status.is(200))
   )
 
   val updateNamespaceProperties: ChainBuilder =
-    retryOnHttpStatus(maxRetries, retryableHttpCodes, "Update namespace 
properties")(
-      http("Update Namespace Properties")
+    retryOnHttpStatus(maxRetries, retryableHttpCodes, "Update Namespace")(
+      http("Update Namespace")
         
.post("/api/catalog/v1/#{catalogName}/namespaces/#{namespaceMultipartPath}/properties")
         .header("Authorization", "Bearer #{accessToken}")
         .header("Content-Type", "application/json")
diff --git 
a/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/TableActions.scala
 
b/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/TableActions.scala
index 1d5b951..67e0ac0 100644
--- 
a/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/TableActions.scala
+++ 
b/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/TableActions.scala
@@ -149,7 +149,7 @@ case class TableActions(
    *
    * There is no limit to the number of users that can create tables 
concurrently.
    */
-  val createTable: ChainBuilder = retryOnHttpStatus(maxRetries, 
retryableHttpCodes, "Create table")(
+  val createTable: ChainBuilder = retryOnHttpStatus(maxRetries, 
retryableHttpCodes, "Create Table")(
     http("Create Table")
       
.post("/api/catalog/v1/#{catalogName}/namespaces/#{multipartNamespace}/tables")
       .header("Authorization", "Bearer #{accessToken}")
@@ -181,7 +181,7 @@ case class TableActions(
    * There is no limit to the number of users that can fetch tables 
concurrently.
    */
   val fetchTable: ChainBuilder = exec(
-    http("Fetch Table")
+    http("Fetch single Table")
       
.get("/api/catalog/v1/#{catalogName}/namespaces/#{multipartNamespace}/tables/#{tableName}")
       .header("Authorization", "Bearer #{accessToken}")
       .check(status.is(200))
@@ -209,7 +209,7 @@ case class TableActions(
    * given namespace, supporting bulk retrieval of table metadata.
    */
   val fetchAllTables: ChainBuilder = exec(
-    http("Fetch all Tables under parent namespace")
+    http("Fetch children Tables")
       
.get("/api/catalog/v1/#{catalogName}/namespaces/#{multipartNamespace}/tables")
       .header("Authorization", "Bearer #{accessToken}")
       .check(status.is(200))
@@ -222,8 +222,8 @@ case class TableActions(
    * There is no limit to the number of users that can update table properties 
concurrently.
    */
   val updateTable: ChainBuilder =
-    retryOnHttpStatus(maxRetries, retryableHttpCodes, "Update table metadata")(
-      http("Update table metadata")
+    retryOnHttpStatus(maxRetries, retryableHttpCodes, "Update Table")(
+      http("Update Table")
         
.post("/api/catalog/v1/#{catalogName}/namespaces/#{multipartNamespace}/tables/#{tableName}")
         .header("Authorization", "Bearer #{accessToken}")
         .header("Content-Type", "application/json")
diff --git 
a/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/ViewActions.scala
 
b/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/ViewActions.scala
index 14bf698..99ff175 100644
--- 
a/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/ViewActions.scala
+++ 
b/benchmarks/src/gatling/scala/org/apache/polaris/benchmarks/actions/ViewActions.scala
@@ -134,7 +134,7 @@ case class ViewActions(
       )
     }
 
-  val createView: ChainBuilder = retryOnHttpStatus(maxRetries, 
retryableHttpCodes, "Create view")(
+  val createView: ChainBuilder = retryOnHttpStatus(maxRetries, 
retryableHttpCodes, "Create View")(
     http("Create View")
       
.post("/api/catalog/v1/#{catalogName}/namespaces/#{multipartNamespace}/views")
       .header("Authorization", "Bearer #{accessToken}")
@@ -171,7 +171,7 @@ case class ViewActions(
   )
 
   val fetchView: ChainBuilder = exec(
-    http("Fetch View")
+    http("Fetch single View")
       
.get("/api/catalog/v1/#{catalogName}/namespaces/#{multipartNamespace}/views/#{viewName}")
       .header("Authorization", "Bearer #{accessToken}")
       .check(status.is(200))
@@ -195,7 +195,7 @@ case class ViewActions(
   )
 
   val fetchAllViews: ChainBuilder = exec(
-    http("Fetch all Views under parent namespace")
+    http("Fetch children Views")
       
.get("/api/catalog/v1/#{catalogName}/namespaces/#{multipartNamespace}/views")
       .header("Authorization", "Bearer #{accessToken}")
       .check(status.is(200))
@@ -208,8 +208,8 @@ case class ViewActions(
    * There is no limit to the number of users that can update table properties 
concurrently.
    */
   val updateView: ChainBuilder =
-    retryOnHttpStatus(maxRetries, retryableHttpCodes, "Update View metadata")(
-      http("Update View metadata")
+    retryOnHttpStatus(maxRetries, retryableHttpCodes, "Update View")(
+      http("Update View")
         
.post("/api/catalog/v1/#{catalogName}/namespaces/#{multipartNamespace}/views/#{viewName}")
         .header("Authorization", "Bearer #{accessToken}")
         .header("Content-Type", "application/json")

Reply via email to