Skip to content

Sync Java SDK models from atlanhq/models#2415

Merged
cmgrote merged 2 commits intomainfrom
sync-models-20260416-112222
Apr 16, 2026
Merged

Sync Java SDK models from atlanhq/models#2415
cmgrote merged 2 commits intomainfrom
sync-models-20260416-112222

Conversation

@atlan-ci
Copy link
Copy Markdown
Collaborator

Summary

This PR syncs the generated Java SDK models from the atlanhq/models repository.

Changes

Auto-generated Java model files from Pkl type definitions.

Test Plan

  • Verify generated models compile successfully
  • Run existing tests against updated models

🤖 This PR was automatically created by the sync-java-sdk workflow.

Source commit: a86c1e8c2bb1e1f05f981deeb8fba6535865c0e7
Source ref: refs/heads/master

Co-Authored-By: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@atlan-ci atlan-ci requested a review from cmgrote as a code owner April 16, 2026 11:31
cmgrote added a commit that referenced this pull request Apr 16, 2026
The old ModelGeneratorV2 accidentally placed copies of the DbtTest
concrete class and IDbtTest interface into src/test/java alongside
the authoritative versions in src/main/java. Because the test
compilation source set includes both trees, the stale test-tree
copies shadow the correctly-maintained main-tree versions.

When the Pkl sync (PR #2415) added catalogDatasetGuid to DbtTest.java
in src/main/java, the stale src/test/java/DbtTest.java (without the
field) continued to be compiled by compileTestJava, producing:

  DbtTest.java:42: error: DbtTest is not abstract and does not
  override abstract method getCatalogDatasetGuid() in IDbt

The canonical files in src/main/java are generated and maintained by
the Pkl renderer; the test-tree copies serve no purpose.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Chris (He/Him) <cgrote@gmail.com>
@cmgrote cmgrote enabled auto-merge April 16, 2026 12:42
@cmgrote cmgrote merged commit 3ba453e into main Apr 16, 2026
7 checks passed
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