diff options
author | Harald Musum <musum@verizonmedia.com> | 2020-11-12 13:05:58 +0100 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2020-11-12 13:05:58 +0100 |
commit | a93e4b804a578a8fd7b0f990b161e488026f8eb9 (patch) | |
tree | 4bda1ccb2ba240bbba3d1f5d604cba9b2a101e6b /config-application-package/src/main/java | |
parent | f2e6bfa53874950c594d541944e085ad12e727f5 (diff) |
Create transformer factory only once
Diffstat (limited to 'config-application-package/src/main/java')
-rw-r--r-- | config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java b/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java index 9c68fb95cce..ffc975d8668 100644 --- a/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java +++ b/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java @@ -34,7 +34,6 @@ import org.w3c.dom.NodeList; import org.xml.sax.SAXException; import javax.xml.parsers.ParserConfigurationException; -import javax.xml.transform.Transformer; import javax.xml.transform.TransformerException; import javax.xml.transform.TransformerFactory; import javax.xml.transform.dom.DOMSource; @@ -98,6 +97,7 @@ public class FilesApplicationPackage implements ApplicationPackage { private final List<String> userIncludeDirs = new ArrayList<>(); private final ApplicationMetaData metaData; private final boolean includeSourceFiles; + private final TransformerFactory transformerFactory; /** Creates from a directory with source files included */ public static FilesApplicationPackage fromFile(File appDir) { @@ -160,6 +160,7 @@ public class FilesApplicationPackage implements ApplicationPackage { configDefsDir = new File(appDir, ApplicationPackage.CONFIG_DEFINITIONS_DIR); addUserIncludeDirs(); this.metaData = metaData; + transformerFactory = TransformerFactory.newInstance(); } @Override @@ -591,12 +592,13 @@ public class FilesApplicationPackage implements ApplicationPackage { inputXml, metaData.getApplicationId().instance(), zone.environment(), - zone.region()).run(); - Transformer transformer = TransformerFactory.newInstance().newTransformer(); + zone.region()) + .run(); + try (FileOutputStream outputStream = new FileOutputStream(destination)) { - transformer.transform(new DOMSource(document), new StreamResult(outputStream)); + transformerFactory.newTransformer().transform(new DOMSource(document), new StreamResult(outputStream)); } - } catch (TransformerException |ParserConfigurationException | SAXException e) { + } catch (TransformerException | ParserConfigurationException | SAXException e) { // TODO: 2020-11-09, debugging what seems like missing or empty file while preprocessing. Remove afterwards if (inputXml.canRead()) { log.log(Level.WARNING, "Error preprocessing " + inputXml.getAbsolutePath() + ", file content: " + IOUtils.readFile(inputXml)); |