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

Improvements and maintenance #75

Open
MkCons opened this issue Jul 7, 2020 · 7 comments
Open

Improvements and maintenance #75

MkCons opened this issue Jul 7, 2020 · 7 comments

Comments

@MkCons
Copy link

MkCons commented Jul 7, 2020

Hi,

I have made a few improvements on the appender:

  • Clear the log buffer so that when the program is about to finish pending messages are sent to Elasticsearch.
  • Add the capability of transmitting compressed messages.
  • Add the capability of logging Java POJOs as JSON.
  • Whenever a key-value parameter is logged as part of a message “serialized document with docOid=1234”, we log it outside as well, as other appenders do “serialized document with docOid=1234” + “docOid=1234”.
  • Allow the users to add a keyPrefix, so we would have prefix.docOid=1234.

I have not seen much activity in the repo lately though. To the original maintainers, are you guys still there? And if so, do you have plans for a 1.7 release?

Best,

@abatkin
Copy link
Member

abatkin commented Jul 7, 2020

Still here...just distracted with other things (sorry).

I haven't done a release in a while because I don't (currently) have an ES cluster to test anything against, so while there a bunch of PRs outstanding, I feel weird about merging and releasing without being able to test anything.

@MkCons
Copy link
Author

MkCons commented Jul 7, 2020

Heya,

No apologies necessary.

I've been testing my changes on a single ES + Kibana on my laptop. Is there any reason we would need to do that in a cluster?

@Lucas3oo
Copy link

It would be really great to see those PR merged into main. Especially issues around memory leaks and setup/ configuration. There are many forks so your lib is really used a lot too :-)
/Lucas

@veenarm
Copy link

veenarm commented Oct 15, 2021

Guessing the project is now not maintained?

@abatkin
Copy link
Member

abatkin commented Oct 17, 2021

I would be open to doing a release (and potentially adding maintainers), but I would need some help (since I don't actively use this regularly anymore, and I don't have a convenient ElasticSearch cluster to test with).

If a few interested folks want to propose the list of PRs that should go into the next release, I can push a test branch with all of those in it. If a couple folks vouch that it all still works (and no one else has objections), I will merge and push that out as a release.

@jonahbenton
Copy link

Wanted to chime in, I'm on a team that just started using this appender with OpenSearch, eager to see some enhancements, and would be happy to test a release.

@agido-malter
Copy link

Wanted to chime in, I'm on a team that just started using this appender with OpenSearch, eager to see some enhancements, and would be happy to test a release.

It looks like this project has been abandoned. We and some others use this library and we decided some time ago to continue it. The libs used are up to date and a number of pull requests are merged. This isn't to steal a project, we're just keeping it up to date.
See agido-malter/logback-elasticsearch-appender

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants