Skip to content

Comments

Add cdc listener and service implementation#2

Merged
niveathika merged 11 commits intoballerina-platform:mainfrom
niveathika:main
May 8, 2025
Merged

Add cdc listener and service implementation#2
niveathika merged 11 commits intoballerina-platform:mainfrom
niveathika:main

Conversation

@niveathika
Copy link
Contributor

@niveathika niveathika commented May 7, 2025

Purpose

$subject

Will send the test cases as separate PR. This reduces lines and gives effective review.
Fixes:

Examples

Checklist

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

@niveathika niveathika requested a review from daneshk as a code owner May 7, 2025 05:23
@niveathika niveathika requested review from Copilot and removed request for daneshk May 7, 2025 05:23
Copy link

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.

Pull Request Overview

This PR adds the CDC listener and service implementation by introducing new platform configurations and dependency updates for CDC, Debezium, Kafka, and Jackson, as well as new package definitions.

  • Introduces a [platform.java21] section with GraalVM compatibility and several new dependencies in both the build-config and ballerina projects.
  • Updates Dependencies.toml with new package definitions (crypto, data.jsondata, lang.object, observe, random, time) and bumps the distribution version.
  • Modifies the GitHub Actions workflow by adding an additional build flag for excluding cdc-examples.

Reviewed Changes

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

File Description
build-config/resources/Ballerina.toml Added the [platform.java21] section and new dependency listings for Debezium, Kafka, etc.
ballerina/Dependencies.toml Updated distribution version and added multiple package dependencies and modules.
ballerina/Ballerina.toml Added new platform.java21 dependencies to ensure GraalVM compatibility.
.github/workflows/trivy-scan.yml Added an additional build flag to the workflow configuration.
Files not reviewed (16)
  • ballerina/annotations.bal: Language not supported
  • ballerina/build.gradle: Language not supported
  • ballerina/errors.bal: Language not supported
  • ballerina/extern_functions.bal: Language not supported
  • ballerina/init.bal: Language not supported
  • ballerina/mssql_listener.bal: Language not supported
  • ballerina/mysql_listener.bal: Language not supported
  • ballerina/oracledb_listener.bal: Language not supported
  • ballerina/postgresql_listener.bal: Language not supported
  • ballerina/service.bal: Language not supported
  • ballerina/types.bal: Language not supported
  • ballerina/utils.bal: Language not supported
  • build-config/spotbugs-exclude.xml: Language not supported
  • examples/build.bat: Language not supported
  • gradle.properties: Language not supported
  • native/build.gradle: Language not supported
Comments suppressed due to low confidence (1)

ballerina/Dependencies.toml:76

  • [nitpick] The organization identifier 'ballerinai' in the observe package instead of the usual 'ballerina' could lead to inconsistency in dependency organization. Consider using a consistent organization naming to avoid confusion.
org = "ballerinai"

@niveathika niveathika force-pushed the main branch 5 times, most recently from a0812fe to 826ec00 Compare May 8, 2025 16:39
@codecov
Copy link

codecov bot commented May 8, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

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

@niveathika niveathika merged commit b018d40 into ballerina-platform:main May 8, 2025
4 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.

2 participants