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

Out of memory error when creating a chart #1727

Closed
wimjongman opened this issue Jun 11, 2024 · 58 comments · Fixed by #1730
Closed

Out of memory error when creating a chart #1727

wimjongman opened this issue Jun 11, 2024 · 58 comments · Fixed by #1730
Assignees
Labels
RELEASE BLOCKER Blocks the nearest release
Milestone

Comments

@wimjongman
Copy link
Contributor

Steps to reproduce

  • Create a simple table with the customer table
  • Drag a chart below the table
  • Fill in X and Y

!ENTRY org.eclipse.ui 4 0 2024-06-11 11:35:50.977
!MESSAGE Unhandled event loop exception
!STACK 0
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.runAsyncMessages(Synchronizer.java:135)
at org.eclipse.swt.widgets.Display.runAsyncMessages(Display.java:4058)
at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3676)
at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$5.run(PartRenderingEngine.java:1151)
at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:339)
at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1042)
at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:152)
at org.eclipse.ui.internal.Workbench.lambda$3(Workbench.java:639)
at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:339)
at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:546)
at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:173)
at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:152)
at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:208)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:143)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:109)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:439)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:271)
at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
at java.base/java.lang.reflect.Method.invoke(Method.java:580)
at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:668)
at org.eclipse.equinox.launcher.Main.basicRun(Main.java:605)
at org.eclipse.equinox.launcher.Main.run(Main.java:1481)
Caused by: java.lang.OutOfMemoryError: Java heap space
at org.eclipse.swt.graphics.ImageData.(ImageData.java:451)
at org.eclipse.swt.graphics.ImageData.(ImageData.java:262)
at org.eclipse.swt.graphics.Image.directToDirect(Image.java:1791)
at org.eclipse.swt.graphics.Image.init(Image.java:1897)
at org.eclipse.swt.graphics.Image.init(Image.java:2111)
at org.eclipse.swt.graphics.Image.(Image.java:497)
at org.eclipse.birt.chart.reportitem.ui.DesignerRepresentation.paintChart(DesignerRepresentation.java:358)
at org.eclipse.birt.chart.reportitem.ui.DesignerRepresentation.paintClientArea(DesignerRepresentation.java:324)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.birt.report.designer.internal.ui.editors.schematic.figures.ReportRootFigure.paintChildren(ReportRootFigure.java:121)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1312)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308)
at org.eclipse.draw2d.ScalableFreeformLayeredPane.paintClientArea(ScalableFreeformLayeredPane.java:73)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1303)
at org.eclipse.draw2d.Viewport.paintClientArea(Viewport.java:161)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)

@wimjongman wimjongman added the RELEASE BLOCKER Blocks the nearest release label Jun 11, 2024
@wimjongman wimjongman added this to the 4.16 milestone Jun 11, 2024
@merks
Copy link
Contributor

merks commented Jun 11, 2024

This is in the all-in-one product? The following seems to be a relatively small value:

image

Does manually changing this in the birt.ini to a larger number help?

@wimjongman
Copy link
Contributor Author

After close Eclipse will not restart.

log.txt

@merks
Copy link
Contributor

merks commented Jun 11, 2024

So it's running out of heap while restarting, trying to re-open the editor you had open.

Please try increasing the -Xmx...

@wimjongman
Copy link
Contributor Author

After changing to 2GB I do not get the OOME but an IOOBE.

stack java.lang.ArrayIndexOutOfBoundsException: Index 466020624 out of bounds for length 466020624 at org.eclipse.swt.graphics.ImageData.blit(ImageData.java:1921) at org.eclipse.swt.graphics.Image.directToDirect(Image.java:1793) at org.eclipse.swt.graphics.Image.init(Image.java:1897) at org.eclipse.swt.graphics.Image.init(Image.java:2111) at org.eclipse.swt.graphics.Image.(Image.java:497) at org.eclipse.birt.chart.reportitem.ui.DesignerRepresentation.paintChart(DesignerRepresentation.java:358) at org.eclipse.birt.chart.reportitem.ui.DesignerRepresentation.paintClientArea(DesignerRepresentation.java:324) at org.eclipse.draw2d.Figure.paint(Figure.java:1229) at org.eclipse.birt.report.designer.internal.ui.editors.schematic.figures.ReportRootFigure.paintChildren(ReportRootFigure.java:121) at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1312) at org.eclipse.draw2d.Figure.paint(Figure.java:1229) at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275) at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308) at org.eclipse.draw2d.Figure.paint(Figure.java:1229) at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275) at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308) at org.eclipse.draw2d.Figure.paint(Figure.java:1229) at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275) at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308) at org.eclipse.draw2d.ScalableFreeformLayeredPane.paintClientArea(ScalableFreeformLayeredPane.java:73) at org.eclipse.draw2d.Figure.paint(Figure.java:1229) at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275) at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308) at org.eclipse.draw2d.Figure.paint(Figure.java:1229) at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275) at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1303) at org.eclipse.draw2d.Viewport.paintClientArea(Viewport.java:161) at org.eclipse.draw2d.Figure.paint(Figure.java:1229) at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275) at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308) at org.eclipse.draw2d.Figure.paint(Figure.java:1229) at org.eclipse.draw2d.DeferredUpdateManager.paint(DeferredUpdateManager.java:168) at org.eclipse.draw2d.LightweightSystem.paint(LightweightSystem.java:202) at org.eclipse.draw2d.LightweightSystem.lambda$0(LightweightSystem.java:110) at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:91) at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4285) at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1160) at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1184) at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1169) at org.eclipse.swt.widgets.Composite.WM_PAINT(Composite.java:1540) at org.eclipse.swt.widgets.Control.windowProc(Control.java:4769) at org.eclipse.swt.widgets.Canvas.windowProc(Canvas.java:340) at org.eclipse.swt.widgets.Display.windowProc(Display.java:5050) at org.eclipse.swt.internal.win32.OS.UpdateWindow(Native Method) at org.eclipse.swt.widgets.Decorations.setVisible(Decorations.java:1293) at org.eclipse.swt.widgets.Shell.setVisible(Shell.java:2097) at org.eclipse.swt.widgets.Shell.open(Shell.java:1306) at org.eclipse.e4.ui.workbench.renderers.swt.WBWRenderer.postProcess(WBWRenderer.java:741) at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.safeCreateGui(PartRenderingEngine.java:676) at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.safeCreateGui(PartRenderingEngine.java:762) at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$2.run(PartRenderingEngine.java:727) at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:47) at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.createGui(PartRenderingEngine.java:711) at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$5.run(PartRenderingEngine.java:1079) at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:339) at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1042) at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:152) at org.eclipse.ui.internal.Workbench.lambda$3(Workbench.java:639) at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:339) at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:546) at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:173) at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:152) at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:208) at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:143) at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:109) at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:439) at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:271) at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103) at java.base/java.lang.reflect.Method.invoke(Method.java:580) at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:668) at org.eclipse.equinox.launcher.Main.basicRun(Main.java:605) at org.eclipse.equinox.launcher.Main.run(Main.java:1481)

@merks
Copy link
Contributor

merks commented Jun 11, 2024

Looking at this stack trace, the birt things it haven't changed for quite a few years, so that leaves draw2d and swt differences and the potential candidates.

I would expect this to be reproducible with a debug launch...

@merks
Copy link
Contributor

merks commented Jun 11, 2024

I'd like to be able to reproduce the problem. Would it be possible to provide "idiot-resistant" steps? Or a zip with the project with the project and what should be opened to see the problem?

@wimjongman
Copy link
Contributor Author

Please drop this report
customerreport.rptdesign.txt in a report project. Then follow this video.

birt_76FtdlC3hW.mp4

@wimjongman
Copy link
Contributor Author

... remove the .txt extension after you dropped it.

@merks
Copy link
Contributor

merks commented Jun 11, 2024

Sorry I was out for a while...

It's unhelpful, but debug launch worked okay:

image

I'll poke at it a bit more, including trying the actual unzipped product.

@merks
Copy link
Contributor

merks commented Jun 11, 2024

It even works in the unzipped product for me.

image

I'm on Windows 10 still.

I'm not sure what to make of this. 😞

@wimjongman
Copy link
Contributor Author

Mmm. It works in the debug launch for me as well. I do have a deep path for my runtime C:\Users\jongw\Downloads\birt-report-designer-all-in-one-4.16.0-202406071258-win32.win32.x86_64\eclipse

@merks
Copy link
Contributor

merks commented Jun 11, 2024

I tried a really long name too but no problems.

D:/Users/test36/birt-Downloads-birt-report-designer-all-in-one-4.16.0-202406071258-win32.win32.x86_64/eclipse/birt.exe

Maybe you can try not having it in the Downloads folder.

Unless I can reproduce the problem somewhere somehow, I'm not sure what to do.

BTW, here is the saved result. I wonder if you have problems when you open that?

customerreport-with-chart.rptdesign.txt

@merks
Copy link
Contributor

merks commented Jun 11, 2024

BTW, I have this local modified version of BIRT.setup for generating the *.target I've been committing, but I have not commit the setup itself. So you can use this locally to ensure that the target platform is exactly like the one used by the actual product built by Tycho:

BIRT.setup.txt

You of course need to replace the existing one with this copy, and do Help -> Perform Setup Tasks.

You only need to run this task:

image

@merks
Copy link
Contributor

merks commented Jun 12, 2024

Of course I will hold of the release for this, but I am at a complete loss for how to help with a problem that I cannot reproduce...

I'll be a car soon until later in the afternoon so that also will limit my helpfulness today.

@speckyspooky
Copy link
Contributor

Hi together, I have retested the latest milestone "BIRT 4.16.0 Milestone - S202406071258" with the "all in one designer" from my side with Windows 10.

I used the default download-link "birt-report-designer-all-in-one-4.16.0-202406071258-win32.win32.x86_64.zip"

And changed nothing from my side and used your report @wimjongman
According to your video I added the chart to the report and the result is:

grafik

I didn't got the "event loop exception". I got only an information.
The preview could be created also the PDF & Excel.

Have you used your local dev-environment (with the latest changes) or our official download link?
If I have time I can test it this evening with my Windows 11 system.

@wimjongman
Copy link
Contributor Author

Hi, thanks for testing. I have Windows 11. I downloaded the milestone again. This time I extracted it outside Downloads with a shorter path

I opened Ed's report without any issues but as soon as I added a chart it failed again.

firefox_PTYuQXt3dE.mp4

@wimjongman
Copy link
Contributor Author

After increasing the memory I get the

java.lang.ArrayIndexOutOfBoundsException: Index 466020624 out of bounds for length 466020624
at org.eclipse.swt.graphics.ImageData.blit(ImageData.java:1921)
at org.eclipse.swt.graphics.Image.directToDirect(Image.java:1793)
at org.eclipse.swt.graphics.Image.init(Image.java:1897)
at org.eclipse.swt.graphics.Image.init(Image.java:2111)
at org.eclipse.swt.graphics.Image.(Image.java:497)
at org.eclipse.birt.chart.reportitem.ui.DesignerRepresentation.paintChart(DesignerRepresentation.java:358)
at org.eclipse.birt.chart.reportitem.ui.DesignerRepresentation.paintClientArea(DesignerRepresentation.java:324)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.birt.report.designer.internal.ui.editors.schematic.figures.ReportRootFigure.paintChildren(ReportRootFigure.java:121)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1312)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308)
at org.eclipse.draw2d.Figure.paint(Figure.java:1229)
at org.eclipse.draw2d.Figure.paintChildren(Figure.java:1275)
at org.eclipse.draw2d.Figure.paintClientArea(Figure.java:1308)

@claesrosell
Copy link
Contributor

1:24 into the movie the target size for chart is briefly visible. It is for some reason set to: 57225 x 286125 pt
What happens if you lower that a bit?

bild

@hvbtup
Copy link
Contributor

hvbtup commented Jun 12, 2024

Just for info: In countries which use a decimal comma instead of a decimal point, editing sizes manually in the dialogs is often a matter of luck: Do I have to enter a point or a comma, and if so, is it interpreted as a decimal separator or as a thousands separator?
The huge width and height values might be a result of such a confusion.

@wimjongman
Copy link
Contributor Author

@claesrosell When I change the size manually I can open the report without issues.

image

@wimjongman
Copy link
Contributor Author

@claesrosell Thanks for finding the root cause. Now we need to figure out why it resizes to that format for me and not for the others. I have windows 11, but when I run it from a Launch configuration everything works as expected.

I will try the launch config with the new target platform Ed gave me.

@wimjongman
Copy link
Contributor Author

@merks your new setup runs in an error

image

@merks
Copy link
Contributor

merks commented Jun 12, 2024

The updated catalogs were promoted a few hours ago so I will need to make adjustments to the setup now. Which is hard to do from a speeding car. I’ll be home in about 2 hours. Perhaps try to fiddle with an explicit -nl argument in the launch or in the unpackaged application to test that theory.

@ptziegler maybe this rings a bell?

@merks
Copy link
Contributor

merks commented Jun 12, 2024

This version should work with the latest catalogs:

BIRT.setup.txt

@merks
Copy link
Contributor

merks commented Jun 12, 2024

When I look at sizes, they look the same as yours except yours are missing the '.':

image

@merks
Copy link
Contributor

merks commented Jun 12, 2024

With -nl nl it looks like this with ',':

image

@wimjongman
Copy link
Contributor Author

That's it. @speckyspooky did you not change some things in the dimension calculations?

@merks
Copy link
Contributor

merks commented Jun 13, 2024

But, all that being said, it doesn't actually crash for me...

@hvbtup
Copy link
Contributor

hvbtup commented Jun 13, 2024

This is what I mean with "confusion".
The root cause of this may be the fact that the value contains a string which should be parsed as a number and a unit.
IMHO, such a string should always use a decimal point and no thousands separator, because basically it is a CSS value and there is no "locale" concept for such values.
Thus the correct value should be 572.25pt. Can you find out where the value is set and why it uses a comma?

@merks
Copy link
Contributor

merks commented Jun 13, 2024

Later it is doing things properly:

image

This is where it's using the locale-specific fomatter:

image

But later parses with the locale as well while validating:

image

@merks
Copy link
Contributor

merks commented Jun 13, 2024

Even here it looks okay still:

image

I.e., that it properly sets a value that uses a ".".

Somehow in Wim's case I think this is kind of removing the comma rather than parsing it correctly, but I can't replicate that.

@merks
Copy link
Contributor

merks commented Jun 13, 2024

@hvbtup

I agree with the locale-specific concerns. It's a nightmare, but not easily unraveled at this point in the game. 😱

@merks
Copy link
Contributor

merks commented Jun 13, 2024

@wimjongman

BTW, I was trying -nl nl_NL to be more like en_US.

@merks
Copy link
Contributor

merks commented Jun 13, 2024

@wimjongman

FYI, I also tried to launch like this to maybe replicate what happens when the locale is determine directly from the environment but still I can't replicate the problem even though I see it using "," values:

image

I don't think I can help further. 😭

@hvbtup
Copy link
Contributor

hvbtup commented Jun 13, 2024

Maybe Wim can use the debugger and look at the same steps like Ed, but in his environment?

@wimjongman
Copy link
Contributor Author

Any -nl value that I set is fine -nl nl -nl en_US

When I don't set -nl is where it bombs

@merks
Copy link
Contributor

merks commented Jun 13, 2024

What is the value of the LC_CTYPE environment variable in your environment? Mine is LC_CTYPE=en_US.UTF-8...

@merks
Copy link
Contributor

merks commented Jun 13, 2024

Set a conditional breakpoint in org.eclipse.birt.report.model.api.DesignElementHandle.setProperty(String, Object)

image

It should be hit stepping through this:

image

Like this:

image

At some point this processing goes badly wrong. I expect it to end up here with the correct value:

image

@wimjongman
Copy link
Contributor Author

What is the value of the LC_CTYPE environment variable in your environment? Mine is LC_CTYPE=en_US.UTF-8...

I do not have this property.

Suggested workaround

What i see today is that it fails in my situation when there is no -nl flag in the ini file. When I put the flag the situation is resolved.

I think we can just add the -nl flag to the generated ini and then release.

@merks
Copy link
Contributor

merks commented Jun 13, 2024

The -nl is followed by the locale. I don't think setting it to en_US is a proper solution.

What does sysinfo tell you?

image

It just seems to me if you can reproduce it under debug control we should be able to find out where things go wrong. 😢

Is the local that does the formatting really nl_NL and not something else/strange in the case it break?

image

@wimjongman
Copy link
Contributor Author

So it only crashes if I leave out the -nl flag.

image

@merks
Copy link
Contributor

merks commented Jun 13, 2024

We cannot just add an -nl to the product to work around a problem that appears to happen on only your machine.

@wimjongman
Copy link
Contributor Author

wimjongman commented Jun 13, 2024

We cannot just add an -nl to the product to work around a problem that appears to happen on only your machine.

Yes, I did not think this through. We do not have an installer that can set it.

@merks
Copy link
Contributor

merks commented Jun 13, 2024

Certainly one can add arguments to the product definition but the -nl requires to specify a locale:

image

We must get to the bottom of this....

@wimjongman
Copy link
Contributor Author

wimjongman commented Jun 13, 2024

Here are the props in my failing setting

no -nl

osgi.nl = en_US
osgi.os = win32
osgi.ws = win32
path.separator = ;
stderr.encoding = UTF-8
stdout.encoding = UTF-8
sun.arch.data.model = 64
sun.cpu.endian = little
sun.cpu.isalist = amd64
sun.io.unicode.encoding = UnicodeLittle
sun.jnu.encoding = UTF-8
sun.management.compiler = HotSpot 64-Bit Tiered Compilers
sun.os.patch.level =
user.country = US
user.country.format = NL
user.language = en
user.name = jongw
user.script =
user.timezone =

@wimjongman
Copy link
Contributor Author

image

@wimjongman
Copy link
Contributor Author

wimjongman commented Jun 13, 2024

if I do Locale.getLocales() then en_NL is not in that list.

@wimjongman
Copy link
Contributor Author

If I add nl_PP or nl_EN or en_NL explicitly then everything is ok.

@merks
Copy link
Contributor

merks commented Jun 13, 2024

The property combination is really odd!

user.country = US
user.country.format = NL
user.language = en

@wimjongman
Copy link
Contributor Author

Ok, I found a solution.

wimjongman added a commit that referenced this issue Jun 13, 2024
Sometimes NumberFormat is used to handle the decimal point
and in other cases there is a dependency on ULocale.

A small fix was made to use the default ULocale in the
Numberformat calculations instead of the default NumberFormat

The whole operation is not fully understood so more issues
could arise.

See the issue for a discussion.
@wimjongman wimjongman linked a pull request Jun 13, 2024 that will close this issue
wimjongman added a commit that referenced this issue Jun 13, 2024
1. Sometimes NumberFormat is used to handle the decimal point
and in other cases there is a dependency on ULocale.

A small fix was made to use the default ULocale in the
Numberformat calculations instead of the default NumberFormat

The whole operation is not fully understood so more issues
could arise.

2. To avoid memory cramps, an additional 1GB was added to the
maximum memory.

See the issue for a discussion.
merks pushed a commit that referenced this issue Jun 13, 2024
Sometimes NumberFormat is used to handle the decimal point
and in other cases there is a dependency on ULocale.

A small fix was made to use the default ULocale in the
Numberformat calculations instead of the default NumberFormat

The whole operation is not fully understood so more issues
could arise.

See the issue for a discussion.
merks pushed a commit that referenced this issue Jun 13, 2024
1. Sometimes NumberFormat is used to handle the decimal point
and in other cases there is a dependency on ULocale.

A small fix was made to use the default ULocale in the
Numberformat calculations instead of the default NumberFormat

The whole operation is not fully understood so more issues
could arise.

2. To avoid memory cramps, an additional 1GB was added to the
maximum memory.

See the issue for a discussion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RELEASE BLOCKER Blocks the nearest release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants