Today, LocalExporter removes its licenseChanged() method from the licence state listener when closed:
|
licenseState.removeListener(this::licenseChanged); |
However in fact this does nothing, because it involves a new Runnable that is different from the one we added earlier. The fix is fairly trivial, but writing the test for the fix has overflowed my stack of yaks so I'm spilling it out here.
Today,
LocalExporterremoves itslicenseChanged()method from the licence state listener when closed:elasticsearch/x-pack/plugin/monitoring/src/main/java/org/elasticsearch/xpack/monitoring/exporter/local/LocalExporter.java
Line 155 in 816e6f6
However in fact this does nothing, because it involves a new
Runnablethat is different from the one we added earlier. The fix is fairly trivial, but writing the test for the fix has overflowed my stack of yaks so I'm spilling it out here.