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

Add constraints for protocol java generation #3119

Merged
merged 10 commits into from
Sep 25, 2024

Conversation

pierredebelen
Copy link
Contributor

Add constraints for protocol java generation

@pierredebelen pierredebelen requested a review from a team as a code owner September 24, 2024 15:59
Copy link

Workflow Telemetry - Build CI / Build

Workflow telemetry for commit 4870eba33be8bfb836d55fb2b444558f5419035d
You can access workflow job details here

Step Trace

gantt
	title Build
	dateFormat x
	axisFormat %H:%M:%S
	Set up job : milestone, 1727193609000, 1727193612000
	Checkout repo : 1727193612000, 1727193616000
	Cache Maven dependencies : 1727193616000, 1727193655000
	Set up JDK : 1727193655000, 1727193681000
	Check Java version : 1727193681000, 1727193681000
	Configure git : 1727193681000, 1727193681000
	Download deps and plugins : 1727193681000, 1727193708000
	Collect Workflow Telemetry : 1727193708000, 1727193708000

Loading

CPU Metrics

chart_stacked_area_time_1a7059a4-c419-42b1-9135-6fba567ee0ca

Memory Metrics

chart_stacked_area_time_863bcf43-8c79-457c-8cdc-4149d83f544f

IO Metrics

Read Write
Network I/O chart_line_time_8ca03e02-98a7-4b34-8c0b-34080246c7f5 chart_line_time_2d5698f9-04b3-48b5-86ba-824357dfd448
Disk I/O chart_line_time_be38f17e-b2fe-4439-9536-b8c9a98bafdc chart_line_time_297eba81-75ec-4d8d-ae8f-8ac2ff3f6951

Process Trace

Top 100 processes with highest duration

gantt
	title Build
	dateFormat x
	axisFormat %H:%M:%S

Loading

Copy link

Workflow Telemetry - Build CI / Build

Workflow telemetry for commit 5e54629ddc482d920b1fc561e41ff7e6b41c2af8
You can access workflow job details here

Step Trace

gantt
	title Build
	dateFormat x
	axisFormat %H:%M:%S
	Set up job : milestone, 1727194265000, 1727194267000
	Checkout repo : 1727194267000, 1727194271000
	Cache Maven dependencies : 1727194271000, 1727194310000
	Set up JDK : 1727194310000, 1727194336000
	Check Java version : 1727194336000, 1727194336000
	Configure git : 1727194336000, 1727194336000
	Download deps and plugins : 1727194336000, 1727194355000
	Collect Workflow Telemetry : 1727194355000, 1727194355000
	Build (PR) : 1727194355000, 1727199424000
	Build (with Maven Deploy + Docker Snapshot) : done, 1727199424000, 1727199424000
	Test : 1727199424000, 1727205048000
	Upload Test Results : 1727205048000, 1727205079000

Loading

CPU Metrics

chart_stacked_area_time_597e9544-20b8-4a22-bd1e-7d81f4617bfa

Memory Metrics

chart_stacked_area_time_488fcadd-2be2-41b7-9456-e48eaa636e49

IO Metrics

Read Write
Network I/O chart_line_time_6cbc1ad6-3730-4a21-9868-5b7d04834fc4 chart_line_time_623c4458-64e1-4097-a426-24b5dac86327
Disk I/O chart_line_time_0b58885d-a69d-4e81-82ac-4bef7adc1dfa chart_line_time_2b8078db-05fd-4b12-8dc6-dcf945e0aa8c

Process Trace

Top 100 processes with highest duration

gantt
	title Build
	dateFormat x
	axisFormat %H:%M:%S

Loading

Copy link

github-actions bot commented Sep 24, 2024

Test Results

  1 036 files  ±0    1 036 suites  ±0   1h 26m 57s ⏱️ - 1m 31s
12 298 tests +2  12 207 ✔️ +2  91 💤 ±0  0 ±0 
17 988 runs  +2  17 897 ✔️ +2  91 💤 ±0  0 ±0 

Results for commit a1accd9. ± Comparison against base commit 2f3c6ce.

♻️ This comment has been updated with latest results.

Copy link

Workflow Telemetry - Build CI / Build

Workflow telemetry for commit a1accd99fae34b0c74321c1f09702cc98dd3ea11
You can access workflow job details here

Step Trace

gantt
	title Build
	dateFormat x
	axisFormat %H:%M:%S
	Set up job : milestone, 1727213230000, 1727213232000
	Checkout repo : 1727213232000, 1727213237000
	Cache Maven dependencies : 1727213237000, 1727213278000
	Set up JDK : 1727213278000, 1727213302000
	Check Java version : 1727213302000, 1727213302000
	Configure git : 1727213302000, 1727213302000
	Download deps and plugins : 1727213302000, 1727213327000
	Collect Workflow Telemetry : 1727213327000, 1727213327000
	Build (PR) : 1727213327000, 1727218373000
	Build (with Maven Deploy + Docker Snapshot) : done, 1727218373000, 1727218373000
	Test : 1727218373000, 1727223892000

Loading

CPU Metrics

chart_stacked_area_time_15da0d19-ffad-4135-8ec8-890ce54ecba4

Memory Metrics

chart_stacked_area_time_9aeab0b9-c3c8-493c-9e0c-da460d66fa74

IO Metrics

Read Write
Network I/O chart_line_time_350e0634-16e9-4fac-ac72-7c22fddaf523 chart_line_time_e7caa882-7628-432c-ba12-053c2522f54b
Disk I/O chart_line_time_6ef1efea-8f56-4cd8-808e-8f6e80a87efc chart_line_time_c02cdedf-e1f0-471a-9ec9-122621d16cfe

Process Trace

Top 100 processes with highest duration

gantt
	title Build
	dateFormat x
	axisFormat %H:%M:%S

Loading

@pierredebelen pierredebelen merged commit b2e6bc8 into master Sep 25, 2024
4 of 5 checks passed
@pierredebelen pierredebelen deleted the addConstraintsForProtocolJavaGen branch September 25, 2024 12:16
AFine-gs pushed a commit to AFine-gs/legend-engine that referenced this pull request Sep 27, 2024
* Add constraint validation for Protocol Java generation

* Add tests for Enum constraints

* Always Box primitives

* Fix boolean type

* Fix tests

* Add default to Protocol

* Fix validation

* Fix impacted tests

* Fix one validation edge case

* Flip test for constraints
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