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

java destination: sequence number support for templates #1628

Merged
merged 1 commit into from Aug 9, 2017

Conversation

furiel
Copy link
Collaborator

@furiel furiel commented Jul 28, 2017

Fixes: #1410

Prior this patch zero was passed as sequence number in java destination templates. This patch propagates the sequence number controlled by the threaded destination driver into LogDestination class, so it will be available in the inherited classes. Thus users can pass it to LogTemplate object when sequence number is used in a template formula.

For example:

LogTemplate logTemplate = new LogTemplate(getConfigHandle());
logTemplate.compile("${SEQNUM}\n");
InternalMessageSender.debug(logTemplate.format(logMessage, 0, logTemplate.LTZ_SEND, getSeqNum()));

@kira-syslogng
Copy link
Contributor

Build SUCCESS, the tests were executed on test branch: master and test suite: functions

@kira-syslogng
Copy link
Contributor

Build SUCCESS, the tests were executed on test branch: master and test suite: functions

@presidento
Copy link
Contributor

Action points:

Signed-off-by: Antal Nemes <antal.nemes@balabit.com>
@kira-syslogng
Copy link
Contributor

Build SUCCESS, the tests were executed on test branch: master and test suite: functions

@pzoleex
Copy link
Collaborator

pzoleex commented Aug 9, 2017

@kira-syslogng test this please test branch=pzolee-java-destination-seqnum;

@kira-syslogng
Copy link
Contributor

Build SUCCESS, the tests were executed on test branch: pzolee-java-destination-seqnum and test suite: functions

@lbudai lbudai merged commit f6b9ba2 into syslog-ng:master Aug 9, 2017
@lbudai lbudai removed the in progress label Aug 9, 2017
@furiel furiel deleted the seqnum_in_java_dest branch May 8, 2018 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants