DanielCarter-stack commented on PR #10503:
URL: https://github.com/apache/seatunnel/pull/10503#issuecomment-3939851201
<!-- code-pr-reviewer -->
<!-- cpr:pr_reply_v2_parts {"group": "apache/seatunnel#10503", "part": 1,
"total": 1} -->
### Issue 1: Incomplete Documentation
**Location**: `docs/en/connectors/changelog/connector-qdrant.md`
**Problem Description**:
The PR description mentions that Qdrant v1.17.0+ has breaking changes, but
version compatibility information is not documented. Users cannot determine
from the documentation which version of Qdrant should be used.
**Potential Risks**:
- Users may use incompatible Qdrant versions in production environments
- Lack of reference information during troubleshooting
**Scope of Impact**:
- Direct Impact: Qdrant connector users
- Indirect Impact: CI/CD pipeline configuration
- Affected Area: Single Connector
**Severity**: MINOR
**Improvement Suggestions**:
Add version compatibility notes in `docs/en/connectors/sink/Qdrant.md`:
```markdown
## Compatibility
This connector has been tested with:
- Qdrant Server: v1.15.0
- Qdrant Java Client: v1.11.0
**Note**: Qdrant v1.17.0+ introduced breaking changes in gRPC and storage
layers
that are currently not compatible with this connector. Please use v1.15.x
for now.
```
Add in `docs/en/connectors/changelog/connector-qdrant.md`:
```markdown
| [Hotfix] Fixed Qdrant E2E test by pinning Docker image version |
https://github.com/apache/seatunnel/commit/778ac12f1 | 2.3.x |
```
**Rationale**: Documentation is an important reference for users. Clear
version specifications can prevent compatibility issues in production
environments.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
