github-mirrors/logstash
1
0
Fork 0
mirror of https://github.com/elastic/logstash.git synced 2025-04-24 14:47:19 -04:00
Code Issues Projects Releases Packages Wiki Activity

improve test for cache difference

Browse source
This commit is contained in:
Joao Duarte 2020-06-26 16:41:41 +01:00 • committed by João Duarte
parent 27117a345a
commit 838c54879b
3 changed files with 16 additions and 29 deletions
Download patch file Download diff file Expand all files Collapse all files

12
logstash-core/src/test/java/org/logstash/config/ir/CompiledPipelineTest.java
View file

@ -570,7 +570,7 @@ public final class CompiledPipelineTest extends RubyEnvTestCase {
@Test
@SuppressWarnings({"unchecked"})
public void testCompilerCacheCompiledClasses() throws IOException, InvalidIRException {
public void testCacheCompiledClassesWithDifferentId() throws IOException, InvalidIRException {
final FixedPluginFactory pluginFactory = new FixedPluginFactory(
() -> null,
() -> IDENTITY_FILTER,
@ -582,20 +582,20 @@ public final class CompiledPipelineTest extends RubyEnvTestCase {
false);
final CompiledPipeline cBaselinePipeline = new CompiledPipeline(baselinePipeline, pluginFactory);
final PipelineIR pipelineWithExtraFilter = ConfigCompiler.configToPipelineIR(
final PipelineIR pipelineWithDifferentId = ConfigCompiler.configToPipelineIR(
IRHelpers.toSourceWithMetadataFromPath("org/logstash/config/ir/cache/pipeline2.conf"),
false);
final CompiledPipeline cPipelineWithExtraFilter = new CompiledPipeline(pipelineWithExtraFilter, pluginFactory);
final CompiledPipeline cPipelineWithDifferentId = new CompiledPipeline(pipelineWithDifferentId, pluginFactory);
// actual test: compiling a pipeline with an extra filter should only create 1 extra class
ComputeStepSyntaxElement.cleanClassCache();
cBaselinePipeline.buildExecution();
final int cachedBefore = ComputeStepSyntaxElement.classCacheSize();
cPipelineWithExtraFilter.buildExecution();
cPipelineWithDifferentId.buildExecution();
final int cachedAfter = ComputeStepSyntaxElement.classCacheSize();
final String message = String.format("unexpected cache size, cachedAfter: %d, cachedBefore: %d", cachedAfter, cachedBefore);
assertEquals(message, 1, cachedAfter - cachedBefore);
assertEquals(message, 0, cachedAfter - cachedBefore);
}
@Test

13
logstash-core/src/test/resources/org/logstash/config/ir/cache/pipeline1.conf vendored
View file

@ -1,19 +1,14 @@
input {
stdin {}
stdin { id => "stdin_id_1" }
}
filter {
mutate {
id => "ppl1_1"
id => "mutate_id_1"
rename => ["a_field", "into_another"]
}
mutate {
id => "ppl1_2"
rename => ["a_field", "into_another"]
}
}
output {
stdout {}
}
stdout { id => "stdout_id_1" }
}

20
logstash-core/src/test/resources/org/logstash/config/ir/cache/pipeline2.conf vendored
View file

@ -1,24 +1,16 @@
# pipeline similar to pipeline 1 but with different ids
# no extra classes should be generated because of this
input {
stdin {}
stdin { id => "stdin_id_1" }
}
filter {
mutate {
id => "ppl2_1"
id => "mutate_id_1"
rename => ["a_field", "into_another"]
}
mutate {
id => "ppl2_2"
rename => ["a_field", "into_another"]
}
mutate {
id => "ppl2_3"
rename => ["a_field", "into_another"]
}
}
output {
stdout {}
}
stdout { id => "stdout_id_1" }
}