mbeckerle commented on code in PR #2836:
URL: https://github.com/apache/drill/pull/2836#discussion_r1376993821


##########
contrib/format-daffodil/src/main/java/org/apache/drill/exec/store/daffodil/DaffodilBatchReader.java:
##########
@@ -64,64 +69,97 @@ public DaffodilBatchReader (DaffodilReaderConfig 
readerConfig, EasySubScan scan,
     this.validationMode = formatConfig.getValidationMode();
 
     //
-    // FIXME: Next, a MIRACLE occurs.
+    // FIXME: Where is this config file to be found? And, what is its syntax?

Review Comment:
   No. That didn't help. I'm not looking for user help here. 
   
   A better question is this: where does a junit test put a config file so that 
the test will read it?
   I was trying to test my config file stuff, which was failing in completely 
inscrutable ways. 
   Eventually tracked it down to drills insistence on file extensions. 
   
   My frustration is due to the fact that the entire boilerplate of this plugin 
configuration stuff could be generated from exactly these lines of 
specification:
   ```
   name daffodil
   schemaURI String null
   dataURI String null
   rootNamespace String ""
   rootName String ""
   validationMode Boolean false
   ```
   
   That's it. The name is "daffodil" and it has these 5 parameters. Spending 
time debugging just getting to where my code has its 5 parameters has been 
quite frustrating. 
   
   I have, however, finally got past this and got a first junit test to work 
end to end. Daffodil constructs drill metadata, Daffodil parses actual data and 
Drill row is constructed. 
   
   I could use some help to figure out why I can't access things under 
src/test/resources. I had to use absolute file URLs to get this to work. 
   
   



-- 
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: dev-unsubscr...@drill.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to