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

Fix the issue when there is no associated entries for the given record #82

Merged
merged 9 commits into from
Oct 29, 2024

Conversation

daneshk
Copy link
Member

@daneshk daneshk commented Oct 28, 2024

Purpose

Fixes: ballerina-platform/ballerina-library#7304
Related issue: https://github.com/wso2-enterprise/internal-support-ballerina/issues/812

Examples

Checklist

  • Linked to an issue
  • Updated the specification
  • Updated the changelog
  • Added tests
  • Checked native-image compatibility

ballerina/sql_client.bal Outdated Show resolved Hide resolved
@Nuvindu Nuvindu changed the title Fix the issue when there is no associated enties for the given record Fix the issue when there is no associated entries for the given record Oct 28, 2024
ballerina/sql_client.bal Outdated Show resolved Hide resolved
ballerina/tests/mssql_api_subscription_client.bal Outdated Show resolved Hide resolved
ballerina/sql_client.bal Outdated Show resolved Hide resolved
ballerina/sql_client.bal Outdated Show resolved Hide resolved
ballerina/tests/mysql_api_subscription_client.bal Outdated Show resolved Hide resolved
Copy link
Member

@ayeshLK ayeshLK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@daneshk daneshk merged commit af6184a into ballerina-platform:main Oct 29, 2024
5 checks passed
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.

[Bug] Bal Persist fails when tries to retrieve record with a nonexistent associations
5 participants