[ 
https://issues.apache.org/jira/browse/BAHIR-75?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15672143#comment-15672143
 ] 

ASF GitHub Bot commented on BAHIR-75:
-------------------------------------

Github user ckadner commented on a diff in the pull request:

    https://github.com/apache/bahir/pull/27#discussion_r88360001
  
    --- Diff: pom.xml ---
    @@ -446,6 +447,7 @@
                   <exclude>.classpath</exclude>
                   <exclude>.project</exclude>
                   <exclude>**/dependency-reduced-pom.xml</exclude>
    +              
<exclude>**/resources/META-INF/services/org.apache.spark.sql.sources.DataSourceRegister</exclude>
    --- End diff --
    
    @lresende even though 
[java.util.ServiceLoader](http://docs.oracle.com/javase/7/docs/api/java/util/ServiceLoader.html)
 ignores `#`-comment lines I did not see any examples of it being used in 
DataSourceRegister files of other projects &mdash; i.e. Spark does not include 
license comments in its 3 DataSourceRegister service provider configuration 
files. What's wrong with excluding it from the RAT license checks?


> Initial Code Delivery
> ---------------------
>
>                 Key: BAHIR-75
>                 URL: https://issues.apache.org/jira/browse/BAHIR-75
>             Project: Bahir
>          Issue Type: Sub-task
>          Components: Spark SQL Data Sources
>            Reporter: Sourav Mazumder
>   Original Estimate: 504h
>  Remaining Estimate: 504h
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to