You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2020/03/14 17:41:59 UTC

[GitHub] [drill] paul-rogers commented on issue #2025: DRILL-7626: Add ability to set HTTP response headers

paul-rogers commented on issue #2025: DRILL-7626: Add ability to set HTTP response headers
URL: https://github.com/apache/drill/pull/2025#issuecomment-599107346
 
 
   Very cool feature, thanks!
   
   I wonder, is it possible to add a unit test for this? In working with the plugin registry, I notice we have few (or no) tests of the UI, and this caused lots of wasted time from reviewers. If we have tests that act as a client to Drill, can we add a quick test for this feature? If not, can we add something so that it is easier to ask others to add unit tests? I think @dobesv, may have added something recently. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services