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.

Sign up for GitHub

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

Jump to bottom

Accidental leak of the min. json dependency #13883

Closed
jerrinot opened this issue Oct 2, 2018 · 2 comments
Closed

Accidental leak of the min. json dependency #13883

jerrinot opened this issue Oct 2, 2018 · 2 comments
Assignees
@kwart
Labels
[OLD]Team: Integration Source: Internal PR or issue was opened by an employee Team: Core Type: Defect
Milestone
3.10.6

Comments

@jerrinot
Copy link
Contributor

jerrinot commented Oct 2, 2018

there is an accidental leak of the min. json dependency in the hazelcast-all artefact.
compare:
https://mvnrepository.com/artifact/com.hazelcast/hazelcast-all/3.10.4 <-- compile time dependency
https://mvnrepository.com/artifact/com.hazelcast/hazelcast/3.10.4 <-- no dependency
https://mvnrepository.com/artifact/com.hazelcast/hazelcast-all/3.9.3 <-- no dependency
https://mvnrepository.com/artifact/com.hazelcast/hazelcast-all/3.9.3 <-- compile time dependency
@jerrinot jerrinot added this to the 3.11 milestone Oct 2, 2018
@kwart kwart self-assigned this Oct 2, 2018
@kwart kwart modified the milestones: 3.11, 3.10.6 Oct 2, 2018
@kwart
Copy link
Member

kwart commented Oct 2, 2018

This issue duplicates #11524. Let's use it to resolve the issue in the maintenance branch.

kwart added a commit to kwart/hazelcast that referenced this issue Oct 2, 2018
@kwart
Fixes hazelcast#13883. Duplicate minimal-json relocation configuratio…
e941811 
…n in hazelcast-all to workaround Maven dependencies resolution in multimodule projects.
@kwart kwart mentioned this issue Oct 2, 2018
Merged
kwart added a commit that referenced this issue Oct 3, 2018
@kwart
Fixes #13883. Duplicate minimal-json relocation configuration in haze…
ecd6af7 
…lcast-all to workaround Maven dependencies resolution in multimodule projects.
@mmedenjak
Copy link
Contributor

Closing as fixed by #13885

@mmedenjak mmedenjak added the Source: Internal PR or issue was opened by an employee label Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kwart kwart

Labels
[OLD]Team: Integration Source: Internal PR or issue was opened by an employee Team: Core Type: Defect
Projects
None yet
Milestone
3.10.6
Development

No branches or pull requests
3 participants
@jerrinot @kwart @mmedenjak