apache / apache/pulsar-connectors

[improve][test] JDBC SQLite sink: optionally align embedded test with shared JDBC harness

Open
#113 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
26
Forks
25
Avg merge
17h 7m
Merged PRs (30d)
1

Description

## Current state (correction)

`jdbc/sqlite` **already has integration coverage**: `SqliteJdbcSinkTest` / `SqliteJdbcSinkBatchTest` run `SqliteJdbcAutoSchemaSink` end-to-end against a real embedded SQLite database (in-process `jdbc:sqlite:` via `SqliteUtils`), covering insert/update/batch paths. This is genuine functional coverage — an earlier triage note incorrectly listed this as a gap.

## Enhancement (optional, not a gap)

Optionally fold the SQLite variant into the shared JDBC integration approach used by the clickhouse/mariadb/postgres modules so all JDBC sinks are exercised through one consistent harness. Embedded SQLite is a perfectly valid backend, so this is low priority and may reasonably be declined.

This is a test-consistency improvement, not a coverage gap.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with SqliteJdbcSinkTest, SqliteJdbcSinkBatchTest, SqliteJdbcAutoSchemaSink, and SqliteUtils, then compare the shared JDBC integration approach used by the clickhouse, mariadb, and postgres modules. Done means the SQLite variant is optionally exercised through the same consistent harness while retaining its existing embedded integration coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, sqlite
Domain
databases, testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.