Merge pull request #2175 from ballerina-platform/perf-improvement-9.x

Remove Locks and Replace Clone Methods in Parser
ballerina-platform · Mar 6, 2025 · 40f50ac · 40f50ac
2 parents 8d2374b + 8b46aa8
commit 40f50ac
Show file tree

Hide file tree

Showing 59 changed files with 758 additions and 667 deletions.
diff --git a/.github/workflows/build-timestamped-master.yml b/.github/workflows/build-timestamped-master.yml
@@ -15,5 +15,5 @@ jobs:
     call_workflow:
         name: Run Build Workflow
         if: ${{ github.repository_owner == 'ballerina-platform' }}
-        uses: ballerina-platform/ballerina-library/.github/workflows/build-timestamp-master-template.yml@main
+        uses: ballerina-platform/ballerina-library/.github/workflows/build-timestamp-master-template.yml@2201.10.x
         secrets: inherit
diff --git a/.github/workflows/build-with-bal-test-graalvm.yml b/.github/workflows/build-with-bal-test-graalvm.yml
@@ -30,7 +30,7 @@ jobs:
     call_stdlib_workflow:
         name: Run StdLib Workflow
         if: ${{ github.event_name != 'schedule' || (github.event_name == 'schedule' && github.repository_owner == 'ballerina-platform') }}
-        uses: ballerina-platform/ballerina-library/.github/workflows/build-with-bal-test-graalvm-template.yml@main
+        uses: ballerina-platform/ballerina-library/.github/workflows/build-with-bal-test-graalvm-template.yml@2201.10.x
         with:
             lang_tag: ${{ inputs.lang_tag }}
             lang_version: ${{ inputs.lang_version }}

diff --git a/.github/workflows/central-publish.yml b/.github/workflows/central-publish.yml
@@ -15,7 +15,7 @@ jobs:
     call_workflow:
         name: Run Central Publish Workflow
         if: ${{ github.repository_owner == 'ballerina-platform' }}
-        uses: ballerina-platform/ballerina-library/.github/workflows/central-publish-template.yml@main
+        uses: ballerina-platform/ballerina-library/.github/workflows/central-publish-template.yml@2201.10.x
         secrets: inherit
         with:
             environment: ${{ github.event.inputs.environment }}
diff --git a/.github/workflows/process-load-test-result.yml b/.github/workflows/process-load-test-result.yml
@@ -6,7 +6,7 @@ on:
 jobs:
     call_stdlib_process_load_test_results_workflow:
         name: Run StdLib Process Load Test Results Workflow
-        uses: ballerina-platform/ballerina-library/.github/workflows/process-load-test-results-template.yml@main
+        uses: ballerina-platform/ballerina-library/.github/workflows/process-load-test-results-template.yml@2201.10.x
         with:
             results: ${{ toJson(github.event.client_payload.results) }}
         secrets:

diff --git a/.github/workflows/publish-release.yml b/.github/workflows/publish-release.yml
@@ -9,7 +9,7 @@ jobs:
     call_workflow:
         name: Run Release Workflow
         if: ${{ github.repository_owner == 'ballerina-platform' }}
-        uses: ballerina-platform/ballerina-library/.github/workflows/release-package-template.yml@main
+        uses: ballerina-platform/ballerina-library/.github/workflows/release-package-template.yml@2201.10.x
         secrets: inherit
         with:
             package-name: graphql

diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -10,5 +10,5 @@ jobs:
     call_workflow:
         name: Run PR Build Workflow
         if: ${{ github.repository_owner == 'ballerina-platform' }}
-        uses: ballerina-platform/ballerina-library/.github/workflows/pull-request-build-template.yml@main
+        uses: ballerina-platform/ballerina-library/.github/workflows/pull-request-build-template.yml@2201.10.x
         secrets: inherit
diff --git a/.github/workflows/trigger-load-tests.yml b/.github/workflows/trigger-load-tests.yml
@@ -22,7 +22,7 @@ jobs:
     call_stdlib_trigger_load_test_workflow:
         name: Run StdLib Load Test Workflow
         if: ${{ github.event_name != 'schedule' || (github.event_name == 'schedule' && github.repository_owner == 'ballerina-platform') }}
-        uses: ballerina-platform/ballerina-library/.github/workflows/trigger-load-tests-template.yml@main
+        uses: ballerina-platform/ballerina-library/.github/workflows/trigger-load-tests-template.yml@2201.10.x
         with:
             repo_name: 'module-ballerina-graphql'
             runtime_artifacts_url: 'https://api.github.com/repos/ballerina-platform/module-ballerina-graphql/actions/artifacts'

diff --git a/.github/workflows/trivy-scan.yml b/.github/workflows/trivy-scan.yml
@@ -9,5 +9,5 @@ jobs:
     call_workflow:
         name: Run Trivy Scan Workflow
         if: ${{ github.repository_owner == 'ballerina-platform' }}
-        uses: ballerina-platform/ballerina-library/.github/workflows/trivy-scan-template.yml@main
+        uses: ballerina-platform/ballerina-library/.github/workflows/trivy-scan-template.yml@2201.10.x
         secrets: inherit
diff --git a/ballerina-tests/Ballerina.toml b/ballerina-tests/Ballerina.toml
@@ -1,7 +1,7 @@
 [package]
 org = "ballerina"
 name = "graphql_tests"
-version = "1.13.2"
+version = "1.13.3"
 
 [build-options]
 observabilityIncluded = true
diff --git a/ballerina-tests/Dependencies.toml b/ballerina-tests/Dependencies.toml
@@ -10,7 +10,7 @@ distribution-version = "2201.9.0"
 [[package]]
 org = "ballerina"
 name = "auth"
-version = "2.11.0"
+version = "2.11.2"
 dependencies = [
 	{org = "ballerina", name = "crypto"},
 	{org = "ballerina", name = "jballerina.java"},
@@ -44,7 +44,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "crypto"
-version = "2.7.0"
+version = "2.7.2"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"},
 	{org = "ballerina", name = "time"}
@@ -67,7 +67,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "graphql"
-version = "1.13.2"
+version = "1.13.3"
 dependencies = [
 	{org = "ballerina", name = "auth"},
 	{org = "ballerina", name = "cache"},
@@ -99,7 +99,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "graphql_tests"
-version = "1.13.2"
+version = "1.13.3"
 dependencies = [
 	{org = "ballerina", name = "constraint"},
 	{org = "ballerina", name = "file"},
@@ -123,7 +123,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "http"
-version = "2.11.0"
+version = "2.11.9"
 dependencies = [
 	{org = "ballerina", name = "auth"},
 	{org = "ballerina", name = "cache"},
@@ -155,7 +155,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "io"
-version = "1.6.0"
+version = "1.6.3"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"},
 	{org = "ballerina", name = "lang.value"}
@@ -172,7 +172,7 @@ version = "0.0.0"
 [[package]]
 org = "ballerina"
 name = "jwt"
-version = "2.11.0"
+version = "2.12.1"
 dependencies = [
 	{org = "ballerina", name = "cache"},
 	{org = "ballerina", name = "crypto"},
@@ -355,7 +355,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "time"
-version = "2.4.0"
+version = "2.5.0"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"}
 ]
@@ -388,7 +388,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "websocket"
-version = "2.11.0"
+version = "2.11.2"
 dependencies = [
 	{org = "ballerina", name = "auth"},
 	{org = "ballerina", name = "constraint"},

diff --git a/ballerina-tests/tests/01_listener_test.bal b/ballerina-tests/tests/01_listener_test.bal
@@ -124,7 +124,8 @@ function testAttachServiceWithMutationToHttp1BasedListenerAndClient() returns er
 }
 
 @test:Config {
-    groups: ["listener", "subscriptions"]
+    groups: ["listener", "subscriptions"],
+    enable: false
 }
 function testAttachServiceWithSubscriptionToHttp1BasedListener() returns error? {
     string document = string `subscription { messages }`;

diff --git a/ballerina-tests/tests/02_request_validation.bal b/ballerina-tests/tests/02_request_validation.bal
@@ -147,7 +147,8 @@ isolated function testInvalidRequestBody() returns error? {
 }
 
 @test:Config {
-    groups: ["request_validation", "websocket", "subscriptions"]
+    groups: ["request_validation", "websocket", "subscriptions"],
+    enable: false
 }
 isolated function testInvalidWebSocketRequestWithEmptyQuery() returns error? {
     string document = "";
@@ -161,7 +162,8 @@ isolated function testInvalidWebSocketRequestWithEmptyQuery() returns error? {
 }
 
 @test:Config {
-    groups: ["request_validation", "websocket", "subscriptions"]
+    groups: ["request_validation", "websocket", "subscriptions"],
+    enable: false
 }
 isolated function testInvalidWebSocketRequestWithInvalidQuery() returns error? {
     string url = "ws://localhost:9099/subscriptions";
@@ -172,11 +174,12 @@ isolated function testInvalidWebSocketRequestWithInvalidQuery() returns error? {
     check wsClient->writeMessage({"type": WS_SUBSCRIBE, id: "1", payload: payload});
     string expectedErrorMsg = "Invalid format: payload does not conform to the format required by the" +
         " 'graphql-transport-ws' subprotocol: Status code: 1003";
-    validateConnectionClousureWithError(wsClient, expectedErrorMsg);
+    validateConnectionClosureWithError(wsClient, expectedErrorMsg);
 }
 
 @test:Config {
-    groups: ["request_validation", "websocket", "subscriptions"]
+    groups: ["request_validation", "websocket", "subscriptions"],
+    enable: false
 }
 isolated function testInvalidWebSocketRequestWithoutQuery() returns error? {
     string url = "ws://localhost:9099/subscriptions";
@@ -186,11 +189,12 @@ isolated function testInvalidWebSocketRequestWithoutQuery() returns error? {
     check wsClient->writeMessage({"type": WS_SUBSCRIBE, id: "1", payload: {}});
     string expectedErrorMsg = "Invalid format: payload does not conform to the format required by the" +
         " 'graphql-transport-ws' subprotocol: Status code: 1003";
-    validateConnectionClousureWithError(wsClient, expectedErrorMsg);
+    validateConnectionClosureWithError(wsClient, expectedErrorMsg);
 }
 
 @test:Config {
-    groups: ["request_validation", "websocket", "subscriptions"]
+    groups: ["request_validation", "websocket", "subscriptions"],
+    enable: false
 }
 isolated function testInvalidVariableInWebSocketPayload() returns error? {
     string document = check getGraphqlDocumentFromFile("subscriptions_with_variable_values");
@@ -202,11 +206,12 @@ isolated function testInvalidVariableInWebSocketPayload() returns error? {
     check sendSubscriptionMessage(wsClient, document, variables = variables);
     string expectedErrorMsg = "Invalid format: payload does not conform to the format required by the" +
         " 'graphql-transport-ws' subprotocol: Status code: 1003";
-    validateConnectionClousureWithError(wsClient, expectedErrorMsg);
+    validateConnectionClosureWithError(wsClient, expectedErrorMsg);
 }
 
 @test:Config {
-    groups: ["request_validation", "websocket", "subscriptions"]
+    groups: ["request_validation", "websocket", "subscriptions"],
+    enable: false
 }
 isolated function testEmptyWebSocketPayload() returns error? {
     string url = "ws://localhost:9099/subscriptions";
@@ -216,11 +221,12 @@ isolated function testEmptyWebSocketPayload() returns error? {
     check wsClient->writeMessage(payload);
     string expectedErrorMsg = "Invalid format: payload does not conform to the format required by the" +
         " 'graphql-transport-ws' subprotocol: Status code: 1003";
-    validateConnectionClousureWithError(wsClient, expectedErrorMsg);
+    validateConnectionClosureWithError(wsClient, expectedErrorMsg);
 }
 
 @test:Config {
-    groups: ["request_validation", "websocket", "subscriptions"]
+    groups: ["request_validation", "websocket", "subscriptions"],
+    enable: false
 }
 isolated function testInvalidWebSocketPayload() returns error? {
     string url = "ws://localhost:9099/subscriptions";
@@ -229,5 +235,5 @@ isolated function testInvalidWebSocketPayload() returns error? {
     check wsClient->writeMessage(payload);
     string expectedErrorMsg = "Invalid format: payload does not conform to the format required by the"
         + " 'graphql-transport-ws' subprotocol: Status code: 1003";
-    validateConnectionClousureWithError(wsClient, expectedErrorMsg);
+    validateConnectionClosureWithError(wsClient, expectedErrorMsg);
 }
diff --git a/ballerina-tests/tests/27_context.bal b/ballerina-tests/tests/27_context.bal
@@ -317,7 +317,8 @@ isolated function testContextWithSubscriptions() returns error? {
 }
 
 @test:Config {
-    groups: ["context", "subscriptions"]
+    groups: ["context", "subscriptions"],
+    enable: false
 }
 isolated function testContextWithInvalidScopeInSubscriptions() returns error? {
     string url = "ws://localhost:9092/context";