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.

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

Dev mode report the wrong startup time starting with Quarkus 1.12 #15371

Closed
loicmathieu opened this issue Mar 1, 2021 · 0 comments · Fixed by #15379
Closed

Dev mode report the wrong startup time starting with Quarkus 1.12 #15371

loicmathieu opened this issue Mar 1, 2021 · 0 comments · Fixed by #15379
Labels
area/devmode kind/bug Something isn't working
Milestone

Comments

@loicmathieu
Copy link
Contributor

Describe the bug
Starting with 1.12, dev mode report two times startup improvements ! But looking at the timing of the logs it tells a different story :(

There seems to be an issue with the time reporting by the dev mode since Quarkus 1.12

This is the logs from the run of my application, you can clearly see that the application starts in more than 1.8s by comparing the timestamp but the logs reports 0.969s.

2021-03-01 11:43:36,250 INFO  [io.qua.ely.sec.pro.dep.ElytronPropertiesProcessor] (build-36) Configuring from MPRealmConfig
2021-03-01 11:43:36,518 INFO  [io.qua.fly.FlywayProcessor] (build-7) Adding application migrations in path '/data/git/payment--prepaid-card/target/classes/db/migration/' using protocol 'file'
2021-03-01 11:43:36,668 WARN  [io.qua.arc.pro.BeanArchives] (build-7) Failed to index long: Class does not exist in ClassLoader QuarkusClassLoader:Deployment Class Loader
2021-03-01 11:43:36,671 WARN  [io.qua.arc.pro.BeanArchives] (build-7) Failed to index int: Class does not exist in ClassLoader QuarkusClassLoader:Deployment Class Loader
2021-03-01 11:43:36,672 WARN  [io.qua.arc.pro.BeanArchives] (build-7) Failed to index boolean: Class does not exist in ClassLoader QuarkusClassLoader:Deployment Class Loader
2021-03-01 11:43:36,732 WARN  [io.qua.arc.pro.BeanArchives] (build-7) Failed to index double: Class does not exist in ClassLoader QuarkusClassLoader:Deployment Class Loader
2021-03-01 11:43:37,717 INFO  [org.fly.cor.int.lic.VersionPrinter] (Quarkus Main Thread) Flyway Community Edition 7.5.2 by Redgate
2021-03-01 11:43:37,782 INFO  [org.fly.cor.int.dat.bas.DatabaseType] (Quarkus Main Thread) Database: jdbc:postgresql://localhost:5433/mydatabase (PostgreSQL 11.3)
2021-03-01 11:43:37,806 INFO  [org.fly.cor.int.com.DbMigrate] (Quarkus Main Thread) Current version of schema "public": 4
2021-03-01 11:43:37,806 INFO  [org.fly.cor.int.com.DbMigrate] (Quarkus Main Thread) Schema "public" is up to date. No migration necessary.
2021-03-01 11:43:38,120 INFO  [io.quarkus] (Quarkus Main Thread) prepaid-card-api 0.15.0 on JVM (powered by Quarkus 1.12.0.Final) started in 0.969s. Listening on: http://0.0.0.0:8081

Environment (please complete the following information):

  • Output of uname -a or ver: Linux 5.8.0-43-generic Color log support #49-Ubuntu SMP Fri Feb 5 03:01:28 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
  • Output of java -version: openjdk version "11.0.7" 2020-04-14
  • Quarkus version or git rev: 1.12
  • Build tool (ie. output of mvnw --version or gradlew --version): Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
@loicmathieu loicmathieu added the kind/bug Something isn't working label Mar 1, 2021
geoand added a commit to geoand/quarkus that referenced this issue Mar 1, 2021
This seems to have been (erroneously) removed in
3a6b561

Fixes: quarkusio#15371
geoand added a commit to geoand/quarkus that referenced this issue Mar 1, 2021
This seems to have been (erroneously) removed in
3a6b561

Fixes: quarkusio#15371
gsmet pushed a commit to geoand/quarkus that referenced this issue Mar 1, 2021
This seems to have been (erroneously) removed in
3a6b561

Fixes: quarkusio#15371
geoand added a commit that referenced this issue Mar 2, 2021
Bring back proper timing for quarkus:dev
@quarkus-bot quarkus-bot bot added this to the 1.13 - master milestone Mar 2, 2021
@gsmet gsmet modified the milestones: 1.13 - master, 1.12.1.Final Mar 2, 2021
gsmet pushed a commit to gsmet/quarkus that referenced this issue Mar 2, 2021
This seems to have been (erroneously) removed in
3a6b561

Fixes: quarkusio#15371
(cherry picked from commit a335c6b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devmode kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants