Skip to content

Refactor - Consolidate mvn codebase#109

Closed
andy-k-improving wants to merge 2 commits intoft-hf-databricks-connectorfrom
ft-hf-mvn-modules-refactor
Closed

Refactor - Consolidate mvn codebase#109
andy-k-improving wants to merge 2 commits intoft-hf-databricks-connectorfrom
ft-hf-mvn-modules-refactor

Conversation

@andy-k-improving
Copy link
Copy Markdown
Contributor

Summary 📝

Move athena-s3vector-connector into the connectors/ directory to co-locate all Athena
federated query connectors under a single parent Maven module.

Changes:

  • Moved athena-s3vector-connector from repo root into connectors/
  • Added athena-s3vector-connector as a module in the parent connectors/pom.xml
  • Updated connectors/README.md with new structure diagram and s3vector build command
  • Updated athena-s3vector-connector/README.md build, deploy, and JAR paths to reflect new
    location

No code changes — this is a structural refactor only.

Test plan:

  • Verified mvn -f connectors/pom.xml package -DskipTests -pl :athena-s3vector-connector builds
    successfully
  • Verified full build mvn -f connectors/pom.xml clean package -DskipTests builds all modules (
    athena-jdbc, databricks, s3vector)
  • No functional changes — connector behavior is unchanged

Permissions

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute
this contribution, under the terms of your choice.

@andy-k-improving andy-k-improving deleted the branch ft-hf-databricks-connector April 8, 2026 23:18
@andy-k-improving andy-k-improving deleted the ft-hf-mvn-modules-refactor branch April 8, 2026 23:21
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.

1 participant