Skip to content
This repository has been archived by the owner on Feb 23, 2023. It is now read-only.

Add support for Log4j2 #115

Closed
sdeleuze opened this issue Apr 27, 2020 · 6 comments
Closed

Add support for Log4j2 #115

sdeleuze opened this issue Apr 27, 2020 · 6 comments
Labels
for: external-project For an external project and not something we can fix type: compatibility Native image compatibility issue

Comments

@sdeleuze
Copy link
Contributor

Related to #84.
@sdeleuze sdeleuze added this to the v0.8.0 milestone Apr 27, 2020
@sdeleuze
Copy link
Contributor Author

See related oracle/graal#808 closed issue.

@sdeleuze sdeleuze self-assigned this Jun 4, 2020
@sdeleuze sdeleuze modified the milestones: v0.8.0, v0.9.0 Jul 21, 2020
@sdeleuze sdeleuze removed their assignment Nov 3, 2020
@sdeleuze sdeleuze modified the milestones: 0.9.0, 0.8.5 Dec 15, 2020
@sdeleuze
Copy link
Contributor Author

Blocked on oracle/graal#2991.

@sdeleuze sdeleuze modified the milestones: 0.8.5, 0.9.1 Dec 18, 2020
@sdeleuze sdeleuze added the status: blocked An issue that's blocked on an external project change or another issue label Dec 18, 2020
sdeleuze added a commit that referenced this issue Dec 18, 2020
@sdeleuze
Initialize LogAdapter$Log4jLog at build time
2598256 
See gh-115
@sdeleuze sdeleuze modified the milestones: 0.9.1, 0.9.2 Feb 26, 2021
@sdeleuze sdeleuze mentioned this issue Mar 19, 2021
Closed
@sdeleuze sdeleuze modified the milestones: 0.11.0, 0.10.0 Mar 29, 2021
@sdeleuze sdeleuze added type: compatibility Native image compatibility issue and removed status: blocked An issue that's blocked on an external project change or another issue labels Mar 29, 2021
@sdeleuze
Copy link
Contributor Author

Should be doable with GraalVM 21.1, see this comment for more details.

@sdeleuze sdeleuze self-assigned this Apr 26, 2021
@sdeleuze sdeleuze reopened this Apr 26, 2021
@sdeleuze
Copy link
Contributor Author

Some hints are still missing so I reopen this issue.

sdeleuze added a commit that referenced this issue Apr 27, 2021
@sdeleuze
Mention that Log4j2 hints are partial.
146e3bf 
They are just designed to avoid crashing during compilation.

Reopens gh-115
See gh-398
@sdeleuze
Copy link
Contributor Author

Ok this one is more involved than expected since Log4j2 is really using a log of reflection. I have tried to craft custom hints but even with that this is not super maintainable/complete and would require native support on Log4j2 side (preferred solution) or a dedicated TypeProcessor or NativeConfiguration. This would be doable but I think we have other priorities tight now, so I move this issue to the backlog, just keeping Log4jCoreHints to avoid a hard breakage during compilation with some dependencies like Elasticsearch (see #398).

@sdeleuze sdeleuze modified the milestones: 0.10.0, Backlog Apr 27, 2021
@sdeleuze sdeleuze mentioned this issue May 7, 2021
Closed
@ttddyy ttddyy mentioned this issue Jul 10, 2021
Closed
@ttddyy ttddyy mentioned this issue Sep 1, 2021
Closed
@ttddyy ttddyy mentioned this issue Feb 1, 2022
Closed
@sdeleuze sdeleuze removed their assignment Feb 3, 2022
@sdeleuze sdeleuze added the jvm-reachability-metadata Should be supported via the GraalVM JVM Reachability Metadata Repository once available label Mar 17, 2022
@sdeleuze
Copy link
Contributor Author

Superseded by https://issues.apache.org/jira/browse/LOG4J2-2649?focusedCommentId=17610488&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-17610488.

@sdeleuze sdeleuze closed this as not planned Won't fix, can't repro, duplicate, stale Sep 28, 2022
@sdeleuze sdeleuze added for: external-project For an external project and not something we can fix and removed jvm-reachability-metadata Should be supported via the GraalVM JVM Reachability Metadata Repository once available labels Sep 28, 2022
@sdeleuze sdeleuze removed this from the Backlog milestone Jan 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
for: external-project For an external project and not something we can fix type: compatibility Native image compatibility issue
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sdeleuze