Branch: refs/heads/master
Home: https://github.com/jenkinsci/logstash-plugin
Commit: dadd1e930d666f6418b69231007088b01e4c7aa5
https://github.com/jenkinsci/logstash-plugin/commit/dadd1e930d666f6418b69231007088b01e4c7aa5
Author: René Scheibe <[email protected]>
Date: 2018-11-22 (Thu, 22 Nov 2018)
Changed paths:
M src/main/java/jenkins/plugins/logstash/LogstashConfiguration.java
M src/main/java/jenkins/plugins/logstash/LogstashConsoleLogFilter.java
M src/main/java/jenkins/plugins/logstash/LogstashItemListener.java
M src/main/java/jenkins/plugins/logstash/LogstashNotifier.java
M src/main/java/jenkins/plugins/logstash/LogstashWriter.java
M src/main/java/jenkins/plugins/logstash/configuration/ElasticSearch.java
M src/main/java/jenkins/plugins/logstash/configuration/LogstashIndexer.java
M src/main/java/jenkins/plugins/logstash/persistence/BuildData.java
M src/main/java/jenkins/plugins/logstash/persistence/ElasticSearchDao.java
M
src/main/java/jenkins/plugins/logstash/persistence/HostBasedLogstashIndexerDao.java
M src/main/java/jenkins/plugins/logstash/persistence/LogstashIndexerDao.java
M src/main/java/jenkins/plugins/logstash/persistence/RabbitMqDao.java
M src/main/java/jenkins/plugins/logstash/persistence/RedisDao.java
M
src/main/resources/jenkins/plugins/logstash/configuration/ElasticSearch/help-mimeType.html
M src/test/java/jenkins/plugins/logstash/LogstashConfigurationTest.java
A src/test/java/jenkins/plugins/logstash/LogstashConsoleLogFilterTest.java
R src/test/java/jenkins/plugins/logstash/LogstashConsoloLogFilterTest.java
M src/test/java/jenkins/plugins/logstash/LogstashOutputStreamTest.java
M src/test/java/jenkins/plugins/logstash/LogstashWriterTest.java
M src/test/java/jenkins/plugins/logstash/configuration/MemoryIndexer.java
M
src/test/java/jenkins/plugins/logstash/persistence/AbstractLogstashIndexerDaoTest.java
M src/test/java/jenkins/plugins/logstash/persistence/BuildDataTest.java
M
src/test/java/jenkins/plugins/logstash/persistence/ElasticSearchSSLCertsTest.java
M src/test/java/jenkins/plugins/logstash/persistence/LogstashDaoTest.java
M src/test/java/jenkins/plugins/logstash/persistence/MemoryDao.java
M src/test/java/jenkins/plugins/logstash/persistence/RabbitMqDaoTest.java
M src/test/java/jenkins/plugins/logstash/persistence/SyslogDaoTestIT.java
Log Message:
-----------
Code Improvements (#82)
* Remove duplicated code
The exact same buildPayload() implementation is already contained in the
base class AbstractLogstashIndexerDao.
* Fix typos
* Remove redundant enum modifier
Inner enums are per default static.
* Remove redundant interface modifiers
Interface methods are per default public.
* Remove unnecessary semicolon
* Fix raw type warning
* Remove unused imports
* Use try-with-resources statements for automatic resource cleanup
* Fix Javadoc issues
* Not using octal number
* Use type inference to increase readability
* Use diamond operator to reduce type duplication
* Make fields final where possible
This reduces the probability of mutability issues.
* Replace deprecated Mockito methods
* Inline redundant variable
**NOTE:** This service has been marked for deprecation:
https://developer.github.com/changes/2018-04-25-github-services-deprecation/
Functionality will be removed from GitHub.com on January 31st, 2019.
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
