Skip to content

Java Runtime Implementation for Query#43958

Merged
SasinduDilshara merged 141 commits intoballerina-platform:java-based-query-runtimefrom
Chamika36:jQuery-sync
Apr 8, 2025
Merged

Java Runtime Implementation for Query#43958
SasinduDilshara merged 141 commits intoballerina-platform:java-based-query-runtimefrom
Chamika36:jQuery-sync

Conversation

@Chamika36
Copy link
Contributor

@Chamika36 Chamika36 commented Apr 1, 2025

Purpose

Optimize query expressions performance

Fixes #43779

Approach

#43779 (comment)

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

  • Read the Contributing Guide
  • Updated Change Log
  • Checked Tooling Support (#)
  • Added necessary tests
    • Unit Tests
    • Spec Conformance Tests
    • Integration Tests
    • Ballerina By Example Tests
  • Increased Test Coverage
  • Added necessary documentation
    • API documentation
    • Module documentation in Module.md files
    • Ballerina By Examples

@CLAassistant
Copy link

CLAassistant commented Apr 1, 2025

CLA assistant check
All committers have signed the CLA.

@Chamika36 Chamika36 self-assigned this Apr 4, 2025
@Chamika36 Chamika36 requested a review from Copilot April 4, 2025 10:37
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 41 out of 41 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/query/pipeline/StreamPipeline.java:130

  • The 'reset()' method is empty; if this method is intended for reusing the pipeline, consider implementing an appropriate reset mechanism or adding documentation to clarify its purpose.
public void reset() {

@SasinduDilshara
Copy link
Contributor

Merging this PR is required for test the changes with the Ballerina Full build pipeline.
Hence merging this.

@Chamika36
Once the full build got passed, lets add a PR from this branch to master

@SasinduDilshara SasinduDilshara merged commit 7c3a449 into ballerina-platform:java-based-query-runtime Apr 8, 2025
2 checks passed
@Chamika36
Copy link
Contributor Author

Merging this PR is required for test the changes with the Ballerina Full build pipeline. Hence merging this.

@Chamika36 Once the full build got passed, lets add a PR from this branch to master

will trigger the full build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants