engine: Java Integration tests
This commit adds Java integration tests. They are currently not integrated with cmake. Happy to discuss the proper integration as part of this PR.
I wrote them to support tracking down https://github.com/fluent/fluent-bit/issues/3014
Enter [N/A] in the box, if an item is not applicable to your change.
Testing Before we can approve your change; please submit the following in a comment:
- [N/A] Example configuration file for the change
- [N/A] Debug log output from testing the change
- [N/A] Attached Valgrind output that shows no leaks or memory corruption was found
Documentation
- [X] Documentation required for this feature
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.
NOTE: It uses a 3rd party library which is also under Apache 2.0:
Apache 2.0: https://github.com/komamitsu/fluency/blob/master/LICENSE.txt
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
Is there any interest in this?
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
Still waiting for feedback, related issue is still open
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
@edsiper @patrick-stephens @nokute78 could you please take a look?
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
@edsiper @patrick-stephens @nokute78 could you please take a look?
@pierluigilenoci I can't see any changes since I last reviewed... see my comments above. I think you need to rebase as well, looks like it may have a dodgy version around the unit tests.
@elruwen could you please implement/review the @patrick-stephens suggestions in your PR? And rebase from master your branch. Thanks.
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
@elruwen could you please implement the suggestions in your PR?
@patrick-stephens finally something moved. Can you take a look again?
@patrick-stephens the ball has returned to your side. 😜
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
@patrick-stephens, could you please revisit this?
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
It's a shame that no one is interested in this PR.
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
😮💨
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
😿
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.