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")