John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/9219 )
Change subject: [DOCS] Removed the obsolete Llama options files ...................................................................... Patch Set 4: For structural changes, it's a good idea to run a DITA-OT build in addition to the normal XML validation steps (validating the map in Oxygen, pre-commit hook running xmllint, etc.). Here's what I get when I run 'make' from the docs/ directory (only showing the first few lines): $ make dita -i impala.ditamap -f html5 -o build/html/ -filter impala_html.ditaval [gen-list] [DOTX008E][ERROR] File 'file:/Users/jrussell/Documents/homework/apache_impala/docs/topics/impala_reservation_request_timeout.xml' does not exist or cannot be loaded. [gen-list] [DOTX008E][ERROR] File 'file:/Users/jrussell/Documents/homework/apache_impala/docs/topics/impala_v_cpu_cores.xml' does not exist or cannot be loaded. [move-meta] File /var/folders/4h/fq2d6grs4s5746pc6hh9x2t00000gp/T/temp20180305114138982/topics/impala_v_cpu_cores.xml was not found. [move-meta] File /var/folders/4h/fq2d6grs4s5746pc6hh9x2t00000gp/T/temp20180305114138982/topics/impala_reservation_request_timeout.xml was not found. There must be some more references to these files left behind, in <xref> tags or other DITA maps. I'll dig a little further. -- To view, visit http://gerrit.cloudera.org:8080/9219 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If0c2b8160af9c95ec1e1b744b558d9537dd2550d Gerrit-Change-Number: 9219 Gerrit-PatchSet: 4 Gerrit-Owner: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: John Russell <jruss...@cloudera.com> Gerrit-Reviewer: Michael Brown <mi...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Mon, 05 Mar 2018 19:44:18 +0000 Gerrit-HasComments: No