Skip to content

fix: Resolve CI failures and critical code quality issues in Cassandra connector#1

Draft
Nandakumar-Balagopal wants to merge 16 commits into
msmygit:attempt2from
Nandakumar-Balagopal:fix-cassandra-ci-issues
Draft

fix: Resolve CI failures and critical code quality issues in Cassandra connector#1
Nandakumar-Balagopal wants to merge 16 commits into
msmygit:attempt2from
Nandakumar-Balagopal:fix-cassandra-ci-issues

Conversation

@Nandakumar-Balagopal

@Nandakumar-Balagopal Nandakumar-Balagopal commented Feb 18, 2026

Copy link
Copy Markdown

Description

  • Fix JNA dependency scope issue (add runtime dependency)
  • Remove static field in NativeCassandraSession (multi-connector fix)
  • Remove incorrect schema metadata configuration in CassandraClientModule
  • Remove hardcoded sleep anti-pattern in CassandraPageSink
  • Remove duplicate VARCHAR mapping in CassandraType
  • Fix checkstyle violations

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.
  • If adding new dependencies, verified they have an OpenSSF Scorecard score of 5.0 or higher (or obtained explicit TSC approval for lower scores).

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

General Changes
* ... 
* ... 

Hive Connector Changes
* ... 
* ... 

If release note is NOT required, use:

== NO RELEASE NOTE ==

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