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.

Sign up for GitHub

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

Jump to bottom

Add support for native queries #38

Merged
merged 51 commits into from
Aug 16, 2023
Merged

Add support for native queries #38

merged 51 commits into from
Aug 16, 2023

Conversation

kaneeldias
Copy link
Contributor

Purpose

Fixes: ballerina-platform/ballerina-library#4546

Examples

Checklist

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

kaneeldias and others added 30 commits June 19, 2023 13:04
@kaneeldias
[Automated] Update native jar versions in toml files
04c7edb
@kaneeldias
[Automated] Update native jar versions in toml files
3641c30
@kaneeldias
[Automated] Update native jar versions in toml files
802ee99
@kaneeldias
Init native support addition
d8b8310
@kaneeldias
Merge remote-tracking branch 'upstream/main' into execute
9b47cab 
# Conflicts:
#	gradle.properties
@kaneeldias
Fix test cases
fff1ff6
@kaneeldias
[Automated] Update native jar versions in toml files
18050c1
@kaneeldias
Improve test cases
c7431ac
@kaneeldias
Remove extra newlines
4697a1b
@kaneeldias
Add comments to make code clearer
1e88606
@kaneeldias
Add transaction test cases for native queries
95f23b6
@kaneeldias
[Automated] Update native jar versions in toml files
96e2263
@kaneeldias
[Automated] Update native jar versions in toml files
e568a0b
@kaneeldias
[Automated] Update native jar versions in toml files
9586946
@kaneeldias
[Automated] Update native jar versions in toml files
1f97d67
@daneshk
Merge pull request ballerina-platform#21 from kaneeldias/execute
3f32641 
Add support for executing native queries
@daneshk
[Automated] Update native jar versions in toml files
8bafac4
@daneshk
Fix the checkstyle issue
fecf1ef
@kalaiyarasiganeshalingam
[Automated] Update native jar versions in toml files
b47e748
@kalaiyarasiganeshalingam
Disable mssql test cases
7a0a198
@kalaiyarasiganeshalingam
Enable test cases
2680370
@kaneeldias
[Automated] Update native jar versions in toml files
de7ba17
@kaneeldias
[Automated] Update native jar versions in toml files
a057179
@kaneeldias
[Automated] Update native jar versions in toml files
5ac60d5
@kaneeldias
Merge remote-tracking branch 'upstream/main' into native-query
3d47cf6 
# Conflicts:
#	native/src/main/java/io/ballerina/stdlib/persist/sql/Utils.java
#	native/src/main/java/io/ballerina/stdlib/persist/sql/datastore/SQLProcessor.java
@kaneeldias
[Automated] Update native jar versions in toml files
2f51bce
@kaneeldias
[Automated] Update native jar versions in toml files
bddb911
@kaneeldias
Fix merge conflicts
081de52
@kaneeldias
[Automated] Update native jar versions in toml files
244cdc6
@kaneeldias
[Automated] Update native jar versions in toml files
97bbb57
@kaneeldias
[Automated] Update native jar versions in toml files
1c8dfce
@kaneeldias
[Automated] Update native jar versions in toml files
3039be9
@kaneeldias
[Automated] Update native jar versions in toml files
d5f23ee
@kaneeldias
[Automated] Update native jar versions in toml files
ace63f2
@kaneeldias
[Automated] Update native jar versions in toml files
8fdac43
@kaneeldias
[Automated] Update native jar versions in toml files
24134ca
@kaneeldias
[Automated] Update native jar versions in toml files
8e60605
@kaneeldias
[Automated] Update native jar versions in toml files
0fcb51a
@kaneeldias
[Automated] Update native jar versions in toml files
dfeac50
@kaneeldias
[Automated] Update native jar versions in toml files
9d7a77f
@kaneeldias
Merge remote-tracking branch 'upstream/main' into native-query
87c9508 
# Conflicts:
#	ballerina/Dependencies.toml
#	gradle.properties
#	native/src/main/java/io/ballerina/stdlib/persist/sql/Constants.java
@kaneeldias
[Automated] Update native jar versions in toml files
bb6fa0d
@kaneeldias
Fix merge conflicts
dbee282
@kaneeldias
Support native queries
7048938
@kaneeldias
Update changelog
90575d0
@kaneeldias
Remove unnecessary comment
80fdf12
@codecov
Copy link

codecov bot commented Aug 15, 2023 •
edited
Loading

Codecov Report

Patch coverage: 81.05% and project coverage change: -0.21% ⚠️

Comparison is base (3309478) 83.75% compared to head (09cc651) 83.54%.
Report is 1 commits behind head on main.

Additional details and impacted files 
@@             Coverage Diff              @@
##               main      #38      +/-   ##
============================================
- Coverage     83.75%   83.54%   -0.21%     
- Complexity      152      165      +13     
============================================
  Files            20       21       +1     
  Lines          1151     1246      +95     
  Branches        335      355      +20     
============================================
+ Hits            964     1041      +77     
- Misses          134      149      +15     
- Partials         53       56       +3
Files Changed Coverage Δ
ballerina/stream_types.bal 70.42% <75.00%> (+2.98%) ⬆️
...ina/stdlib/persist/sql/datastore/SQLProcessor.java 82.45% <81.66%> (-0.88%) ⬇️
ballerina/types.bal 100.00% <100.00%> (ø)
...in/java/io/ballerina/stdlib/persist/sql/Utils.java 90.00% <100.00%> (+2.50%) ⬆️
...a/stdlib/persist/sql/datastore/MSSQLProcessor.java 100.00% <100.00%> (ø)
...a/stdlib/persist/sql/datastore/MySQLProcessor.java 100.00% <100.00%> (ø)

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

@kaneeldias kaneeldias marked this pull request as ready for review August 15, 2023 11:03
daneshk
daneshk previously approved these changes Aug 16, 2023
View reviewed changes
Copy link
Member

@daneshk daneshk left a comment

Choose a reason for hiding this comment

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

LGTM

@kaneeldias
Merge remote-tracking branch 'upstream/main' into native-query2
09cc651 
# Conflicts:
#	ballerina/Ballerina.toml
#	ballerina/Dependencies.toml
@kaneeldias kaneeldias merged commit 5aee510 into ballerina-platform:main Aug 16, 2023
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daneshk daneshk daneshk approved these changes

@niveathika niveathika Awaiting requested review from niveathika

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add support for executing native queries in SQL databases
3 participants
@kaneeldias @daneshk @kalaiyarasiganeshalingam