Github user joewitt commented on the issue:
https://github.com/apache/nifi/pull/2365
Option1: Change from the old impl/dep which had 4 spaces to JDK built-in
classes which is now 2.
Pro: Removes a problematic dependency. Con: Impacts users doing pure
diffs of templates.
Option2: Set the com.sun.indent property on the marshaller. This is not
doc'd or guaranteed and could change. If it does change it could break the app
since it would throw an exception if it gets a property it doesn't understand.
Option3: Use the 2007/java.net binary you found. Given that is from a now
defunct location it is hard to really trust it from an L&N pov and it appears
to have a transitive dep based on looking at its pom. In general i dont love
this option.
So my preference is in order 1, 3, 2.
What do you think @mcgilman
---
