Skip to content

Conversation

@iloveeclipse
Copy link
Member

Regression from commit 9bd3284 / #2400

Also updated version range on bundle with increased version.

Fixes #2430

Regression from commit 9bd3284 / eclipse-platform#2400

Also updated version range on bundle with increased version.

Fixes eclipse-platform#2430
@iloveeclipse
Copy link
Member Author

Exception in thread "Active Thread: Equinox Container: aa943991-8bd5-44f7-a55a-1a7bc7647820" java.lang.OutOfMemoryError: Java heap space
Exception in thread "WorkbenchTestable" org.eclipse.swt.SWTException: Failed to execute runnable (java.lang.OutOfMemoryError: Java heap space)
	at org.eclipse.swt.SWT.error(SWT.java:4922)
	at org.eclipse.swt.SWT.error(SWT.java:4837)
	at org.eclipse.swt.widgets.Synchronizer.syncExec(Synchronizer.java:209)
	at org.eclipse.ui.internal.UISynchronizer.syncExec(UISynchronizer.java:133)
	at org.eclipse.swt.widgets.Display.syncExec(Unknown Source)
	at org.eclipse.e4.ui.internal.workbench.swt.E4Testable.runTest(E4Testable.java:118)
	at org.eclipse.tycho.surefire.osgibooter.AbstractUITestApplication.runTests(AbstractUITestApplication.java:38)
	at org.eclipse.e4.ui.internal.workbench.swt.E4Testable.lambda$0(E4Testable.java:79)
	at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: java.lang.OutOfMemoryError: Java heap space

Where is that coming from? Is this code regression or infrastructure change?

@iloveeclipse
Copy link
Member Author

@laeubi : Looking at the tycho command line output, I wonder why do we have no summary report about number of API warnings/errors for org.eclipse.ui.editors (compared to org.eclipse.ui.workbench.texteditor)?

What causes this difference (see log https://ci.eclipse.org/platform/job/eclipse.platform.ui/job/PR-2431/1/consoleFull)?

07:47:52.107 [INFO] --- tycho-apitools:4.0.8:verify (verify) @ org.eclipse.ui.editors ---
07:47:52.108 [INFO] Resolve API baseline for org.eclipse.platform:org.eclipse.ui.editors:eclipse-plugin:3.19.0-SNAPSHOT with linux/gtk/x86_64
07:48:11.819 [INFO] API Analysis finished in 19 s.
07:47:33.181 [INFO] --- tycho-apitools:4.0.8:verify (verify) @ org.eclipse.ui.workbench.texteditor ---
07:47:33.182 [INFO] Resolve API baseline for org.eclipse.platform:org.eclipse.ui.workbench.texteditor:eclipse-plugin:3.19.0-SNAPSHOT with linux/gtk/x86_64
07:47:43.699 [INFO] API Analysis finished in 10 s.
07:47:43.699 [INFO] 0 API ERRORS
07:47:43.699 [INFO] 0 API warnings

@iloveeclipse
Copy link
Member Author

Where is that coming from? Is this code regression or infrastructure change?

Created #2432 fur further investigation.

@github-actions
Copy link
Contributor

Test Results

 1 818 files  +  483   1 818 suites  +483   1h 54m 25s ⏱️ + 52m 3s
 7 711 tests ±    0   7 482 ✅  -     1  228 💤 ±  0  1 ❌ +1 
24 294 runs  +4 427  23 546 ✅ +4 194  747 💤 +232  1 ❌ +1 

For more details on these failures, see this check.

Results for commit ca6d1df. ± Comparison against base commit 54479d7.

@iloveeclipse
Copy link
Member Author

For the record, test failures are unrelated (see eclipse-platform/eclipse.platform#1592):

image

Merging.

@iloveeclipse iloveeclipse merged commit c9c9ec7 into eclipse-platform:master Oct 21, 2024
12 of 17 checks passed
@iloveeclipse iloveeclipse deleted the issue_2430 branch October 21, 2024 10:11
@BeckerWdf
Copy link
Member

thanks for fixing this.

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.

API errors on org.eclipse.ui.editors and org.eclipse.ui.workbench.texteditor

2 participants