DanielLeens commented on PR #10988: URL: https://github.com/apache/seatunnel/pull/10988#issuecomment-5120270005
Hi @CosmosNi, thank you again for the pgvector work here. This PR had already gone through several careful review rounds, and it would be a shame for the useful context and tests to be lost just because the branch became stale. I updated this original PR branch directly: - Merged the latest `apache/dev` into `feature-pg-vector`. - Kept the PR scope focused on the PostgreSQL JDBC pgvector path: schema inference, type mapping, row conversion, and the pgvector E2E coverage. - Fixed a small compile-level issue in `PostgresTypeMapper`: the new `LOG.debug(...)` path now has the required logger field. - Ran scoped formatting: `./mvnw -nsu -Dmaven.gitcommitid.skip=true -pl seatunnel-connectors-v2/connector-jdbc,seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-part-3 spotless:apply`. The PR now points to head `8dcf4f646440e18fd92580f1225ec833d8557c36`, and GitHub reports it as mergeable again. Current checks have just been queued, so the remaining gate is the normal GitHub CI/review result. Thanks for pushing this feature forward — continuing on the same PR keeps the discussion and review history intact. 🙏 -- 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]
