Skip to content

Add Jackson-based model generation for Trino 483 (prototyping) - #170

Draft
p623 wants to merge 7 commits into
treasure-data:masterfrom
p623:update-modelgen-using-jackson
Draft

p623 wants to merge 7 commits into
treasure-data:masterfrom
p623:update-modelgen-using-jackson

Conversation

@p623

@p623 p623 commented Sep 29, 2026

Copy link
Copy Markdown

Purpose

Briefly describe the purpose of the pull request.

Overview

Describe the list of work you have done in this pull request.

Example:

  • Add something
  • Remove other things

Checklist

  • Code compiles correctly
  • Created tests which fail without the change (if possible)
  • All tests passing
  • Extended the README / documentation, if necessary
% cd modelgen/java-extractor

% ./gradlew extractModels \
  -PtrinoVersion=351 \
  -PjavaVersion=25 \
  --rerun-tasks

% ./gradlew extractModels \
  -PtrinoVersion=483 \
  -PjavaVersion=25 \
  --rerun-tasks

% cd ../../
% bundle exec ruby \
  modelgen/generate_from_schema.rb \
  483 \
  modelgen/schemas/483.json \
  build/modelgen/483.rb

# for compatibility test
% bundle exec ruby \
  modelgen/generate_from_schema.rb \
  351 \
  modelgen/schemas/351.json \             
  build/modelgen/351-new.rb

% bundle exec ruby \
  modelgen/compare_model_shapes.rb \
  351 \
  lib/trino/client/model_versions/351.rb \
  build/modelgen/351-new.rb

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