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

Remove Ballerina handle usage in client classes #84

Merged
merged 52 commits into from
Aug 3, 2023
Merged

Remove Ballerina handle usage in client classes #84

merged 52 commits into from
Aug 3, 2023

Conversation

ayeshLK
Copy link
Member

@ayeshLK ayeshLK commented Jul 28, 2023

Purpose

$subject

Part of: #4639

Examples

N/A

Checklist

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

@ayeshLK ayeshLK marked this pull request as draft July 28, 2023 14:45
@ayeshLK
Remove redundant suppression
4a20371
@ayeshLK
Update doc-comment on JMS connection configuration
13d66a3
@ayeshLK
Update JMS session init
2652028
@ayeshLK
Update jms:createSession API
cae5ea4
@ayeshLK
Resolve merge conflicts
d53a0dc
@ayeshLK
Refactor JMS connection native-init
9f7ee09
@ayeshLK
Update ballerina session init logic
ed6e0b4
@ayeshLK
Refactor JMS session init related java inter-op logic
a54c665
@ayeshLK
Fix checkstyle validation failure
c6094a1
@ayeshLK
Introduce a record to represent JMS destination
ae7f122
@ayeshLK
Update doc comment
6e75663
@ayeshLK
Refactor native code-base
8118cae
@ayeshLK
Update message-producer init logic
ca8f45a
@ayeshLK
Add interop API to create the JMS message
347fc4b
@ayeshLK
Add new extern API to set the content for the JMS text message
c66dcf4
@ayeshLK
Add new non-public API to ballerina JMS session to create JMS message…
35cb84e 
… handle
@ayeshLK
Refactor the message-producer API
f6e71cb
@ayeshLK ayeshLK changed the title Refactor the java.jms API Remove Ballerina handle usage from java.jms module Aug 2, 2023
@ayeshLK ayeshLK marked this pull request as ready for review August 2, 2023 10:45
@ayeshLK ayeshLK changed the title Remove Ballerina handle usage from java.jms module Remove Ballerina handle in client classes Aug 2, 2023
@ayeshLK ayeshLK changed the title Remove Ballerina handle in client classes Remove Ballerina handle usage in client classes Aug 2, 2023
@shafreenAnfar shafreenAnfar merged commit 754ca7e into ballerina-platform:master Aug 3, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shafreenAnfar shafreenAnfar shafreenAnfar approved these changes

Assignees
No one assigned
Labels
Skip GraalVM Check
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ayeshLK @shafreenAnfar