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

MacroConfiguration doesn't implement the comparator for macros correctly #544

Closed
beikov opened this issue Mar 29, 2018 · 0 comments

Comments

@beikov
Copy link
Member

commented Mar 29, 2018

@JWGmeligMeyling found out that MacroConfiguration is actually buggy because doing a length comparison is not enough for the macros. Right now, it doesn't affect us but this should be fixed. Care has to be taken since this is performance critial code.

@beikov beikov added bug core labels Mar 29, 2018

@beikov beikov added this to the 1.2.0 milestone Mar 29, 2018

@beikov beikov self-assigned this Mar 29, 2018

beikov added a commit to beikov/blaze-persistence that referenced this issue Apr 12, 2018

@beikov beikov closed this in 571c1fb Apr 12, 2018

beikov added a commit to beikov/blaze-persistence that referenced this issue Jun 21, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Jun 21, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Jun 22, 2018
[Blazebit#544] Use proper join alias for correlation result in join f…
…etch correlated mappings. Fixes Blazebit#544. Replace injection plugin
beikov added a commit to beikov/blaze-persistence that referenced this issue Jun 22, 2018
[Blazebit#554] Use proper join alias for correlation result in join f…
…etch correlated mappings. Fixes Blazebit#544. Replace injection plugin
beikov added a commit that referenced this issue Jun 26, 2018
[#554] Use proper join alias for correlation result in join fetch cor…
…related mappings. Fixes #544. Replace injection plugin
beikov added a commit to beikov/blaze-persistence that referenced this issue Jun 26, 2018
[Blazebit#554] Use proper join alias for correlation result in join f…
…etch correlated mappings. Fixes Blazebit#544. Replace injection plugin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.