Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Locks and Replace Clone Methods in Parser #2175

Merged
merged 14 commits into from
Mar 6, 2025

Conversation

ThisaruGuruge
Copy link
Member

@ThisaruGuruge ThisaruGuruge commented Mar 6, 2025

Purpose

$Subject

Part of: ballerina-platform/ballerina-library#7652

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility
  • No commons package changes (if there are any, please update the GraphQL version in GraphQL tools and Ballerina dev tools)
  • No compiler package changes (if there are any, please update the GraphQL version in Ballerina dev tools)

@ThisaruGuruge ThisaruGuruge force-pushed the perf-improvement-9.x branch from e3c1c0c to 8b46aa8 Compare March 6, 2025 11:20
Copy link

sonarqubecloud bot commented Mar 6, 2025

Copy link

codecov bot commented Mar 6, 2025

Codecov Report

Attention: Patch coverage is 94.22572% with 22 lines in your changes missing coverage. Please review.

Project coverage is 85.22%. Comparing base (4e3b3de) to head (8b46aa8).
Report is 16 commits behind head on 2201.9.x.

Files with missing lines Patch % Lines
ballerina/modules/parser/lexer.bal 88.33% 7 Missing ⚠️
ballerina/field_validator_visitor.bal 94.25% 5 Missing ⚠️
ballerina/modules/parser/argument_node.bal 87.50% 2 Missing ⚠️
ballerina/response_generator.bal 84.61% 2 Missing ⚠️
ballerina/websocket_service.bal 50.00% 2 Missing ⚠️
ballerina/common_utils.bal 94.73% 1 Missing ⚠️
ballerina/context.bal 93.33% 1 Missing ⚠️
ballerina/executor_visitor.bal 92.85% 1 Missing ⚠️
ballerina/observe_utils.bal 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##             2201.9.x    #2175      +/-   ##
==============================================
- Coverage       85.80%   85.22%   -0.58%     
  Complexity        823      823              
==============================================
  Files             126      126              
  Lines            9713     9713              
  Branches         3839     3831       -8     
==============================================
- Hits             8334     8278      -56     
- Misses           1034     1088      +54     
- Partials          345      347       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@DimuthuMadushan DimuthuMadushan merged commit 40f50ac into 2201.9.x Mar 6, 2025
6 checks passed
@DimuthuMadushan DimuthuMadushan deleted the perf-improvement-9.x branch March 7, 2025 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants