[GitHub] paul-rogers commented on issue #1403: DRILL-6634: Add udf module under contrib directory and move some udfs into it

[GitBox <gi...@apache.org>]

paul-rogers commented on issue #1403: DRILL-6634: Add udf module under contrib directory and move some udfs into it
URL: https://github.com/apache/drill/pull/1403#issuecomment-408180876
 
 
   @arina-ielchiieva: question: as you know, Drill functions are quite difficult to debug -- they rely on code generation and code rewriting, one cannot simply step through the code.
   
   UDFs are also hard to test: one needs to set up the inputs to the functions just right to test all cases.
   
   I've found that using the Drill unit test framework is very helpful, especially for setting up input data and verifying output data. However, as some dev list user pointed out, the test classes were not historically available to code outside the `exec` module. Perhaps Tim's recent changes fixed this so that unit test classes are available in `contrib`?
   
   In short, the request is that we ensure that code in `contrib` can be debugged and tested the same way as code in `exec`. Else, people will have to do what I found I had to do recently: develop the code in `exec`, then, once it is done, move it to `contrib`. If I had to make changes, I had to temporarily move it back to `exec`. Let's make sure that this is no longer necessary. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services