olabusayoT commented on code in PR #1479: URL: https://github.com/apache/daffodil/pull/1479#discussion_r2070457648
########## daffodil-runtime1/src/main/scala/org/apache/daffodil/core/compiler/Compiler.scala: ########## @@ -31,17 +31,21 @@ import scala.collection.mutable.ArrayBuffer import scala.util.Try import scala.xml.Node +import org.apache.daffodil.api.compiler.{ ProcessorFactory => JProcessorFactory } +import org.apache.daffodil.api.exceptions.{ InvalidParserException => JInvalidParserException } +import org.apache.daffodil.api.{ CodeGenerator => JCodeGenerator } +import org.apache.daffodil.api.{ DataProcessor => JDataProcessor } Review Comment: Re 2, The J prefix is because alot of the classes share the same name as their Scala counterpart ex InfosetInputter, DataProcessor, so we needed to rename them and J seemed like the best option. Any suggestions? Re 1, we are also integrating the new JAVA API into our current backend for compatibility @stevedlawrence is that right? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@daffodil.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org