Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Config parsing method needs to be partitioned #704

Closed
dmlloyd opened this issue Feb 1, 2019 · 1 comment
Closed

Config parsing method needs to be partitioned #704

dmlloyd opened this issue Feb 1, 2019 · 1 comment
Assignees
@dmlloyd
Milestone
0.8.0

Comments

@dmlloyd
Copy link
Member

dmlloyd commented Feb 1, 2019

Many of our integration tests are already generating 30K+ bytecodes in the config parsing method. This will have to be partitioned or we're going to hit method size limits.
@dmlloyd dmlloyd self-assigned this Feb 1, 2019
@dmlloyd dmlloyd mentioned this issue Feb 2, 2019
Merged
Sanne added a commit that referenced this issue Feb 4, 2019
@Sanne
Merge pull request #728 from dmlloyd/config-704
292acef 
[#704] Partition config parsing methods
@dmlloyd dmlloyd added this to the 0.8.0 milestone Feb 6, 2019
@dmlloyd
Copy link
Member Author

dmlloyd commented Feb 6, 2019

Resolved via #728.

@dmlloyd dmlloyd closed this as completed Feb 6, 2019
maxandersen pushed a commit to maxandersen/quarkus that referenced this issue Nov 5, 2022
@quintesse
chore: project restructure: moved some classes to their proper places (…
f61cb8b 
…quarkusio#704)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dmlloyd dmlloyd

Labels
None yet
Projects
None yet
Milestone
0.8.0
Development

No branches or pull requests
1 participant
@dmlloyd