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

[Bug] Intermittent debugger failures on Ballerina projects with Bal persist sources #43538

Closed
NipunaRanasinghe opened this issue Nov 1, 2024 · 2 comments · Fixed by #43578
Closed
Assignees
Labels
Area/Debugger Issues related to the ballerina debugging tools Priority/High Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) Type/Bug
Milestone

Comments

@NipunaRanasinghe
Copy link
Contributor

NipunaRanasinghe commented Nov 1, 2024

Description

The Ballerina debugger intermittently fails to show breakpoint hits when debugging Ballerina projects that include Bal persist sources.

As discussed in internal-support-ballerina/issues/803, we observed that this issue arises because the debugger relies on the Project API, which does not fully support scenarios involving changes related to Bal persist sources.

To address this, we need certain fixes (or possibly new APIs) from the Project API. Once these improvements are available, we plan to release the debugger-side fixes accordingly.

Steps to Reproduce

No response

Affected Version(s)

No response

OS, DB, other environment details and versions

No response

Related area

-> Debugger

Related issue(s) (optional)

#43537

Suggested label(s) (optional)

No response

Suggested assignee(s) (optional)

No response

@NipunaRanasinghe NipunaRanasinghe added Type/Bug Area/Debugger Issues related to the ballerina debugging tools Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) labels Nov 1, 2024
@NipunaRanasinghe NipunaRanasinghe self-assigned this Nov 1, 2024
@NipunaRanasinghe NipunaRanasinghe moved this from In Progress to On Hold in Ballerina Team Main Board Nov 1, 2024
@NipunaRanasinghe NipunaRanasinghe added this to the 2201.11.0 milestone Nov 6, 2024
@NipunaRanasinghe
Copy link
Contributor Author

NipunaRanasinghe commented Nov 12, 2024

Please find the meeting notes from my discussion with @sameerajayasoma , @azinneera, and @gayaldassanayake on the above.

  • We can consider using a different file extension for custom definition files (such as for persist and the upcoming service definitions feature) instead of .bal. New extension types (e.g., .baldef or .balspec) could help avoid unforeseen issues in the platform. This change would also resolve the debugger issue, as the debugger would no longer try to treat these files as standard Ballerina source files.

  • Until we reach this solution, we’ll implement a temporary workaround by adding certain Project API utilities directly in the debugger.

Copy link

This issue is NOT closed with a proper Reason/ label. Make sure to add proper reason label before closing. Please add or leave a comment with the proper reason label now.

      - Reason/EngineeringMistake - The issue occurred due to a mistake made in the past.
      - Reason/Regression - The issue has introduced a regression.
      - Reason/MultipleComponentInteraction - Issue occured due to interactions in multiple components.
      - Reason/Complex - Issue occurred due to complex scenario.
      - Reason/Invalid - Issue is invalid.
      - Reason/Other - None of the above cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area/Debugger Issues related to the ballerina debugging tools Priority/High Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) Type/Bug
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant