search

internetitem / logback-elasticsearch-appender

Logback Elasticsearch Appender
Other
233 stars 137 forks source link

Memory leak in appender #44

Open huksley opened 6 years ago

huksley commented 6 years ago

Added appender to spring boot application. Attaching config and screenshot of Eclipse MAT report.

image

logback-config.zip

huksley commented 6 years ago

com.internetitem.logback.elasticsearch.AbstractElasticsearchPublisher.events contains 71140 objects and occupies 136 mb. Is there any way to limit maximum events stored?

maczikasz commented 6 years ago

Would be really awesome to have this fixed

huksley commented 6 years ago

Please test my version If everything ok vote for this to be merged in.

https://jitpack.io/#huksley/logback-elasticsearch-appender

        <!-- https://github.com/internetitem/logback-elasticsearch-appender -->
        <dependency>
            <groupId>com.github.huksley</groupId>
            <artifactId>logback-elasticsearch-appender</artifactId>
            <version>v1.8</version>
        </dependency>

    <repositories>
        <repository>
            <id>jitpack.io</id>
            <url>https://jitpack.io</url>
        </repository>
    </repositories
maczikasz commented 6 years ago

Okay I will look at that, thanks