[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-24 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15981108#comment-15981108 ] Steve Loughran commented on HADOOP-14138: - Actually, doing it in code would work with some

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-22 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15979880#comment-15979880 ] Steve Loughran commented on HADOOP-14138: - bq. A lot of people consider the core-default files

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-21 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15979369#comment-15979369 ] Siddharth Seth commented on HADOOP-14138: - bq. those JIRAs are so old they are implicitly dead.

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-21 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15978670#comment-15978670 ] Steve Loughran commented on HADOOP-14138: - +use XSD to define schema, allow for type of defval

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-20 Thread Aaron Fabbri (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15977816#comment-15977816 ] Aaron Fabbri commented on HADOOP-14138: --- [~ste...@apache.org]'s original proposal here seems like

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-20 Thread John Zhuge (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15977071#comment-15977071 ] John Zhuge commented on HADOOP-14138: - I'd like source code with annotation as the source of truth

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-20 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15976768#comment-15976768 ] Steve Loughran commented on HADOOP-14138: - those JIRAs are so old they are implicitly dead.

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-19 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15975477#comment-15975477 ] Siddharth Seth commented on HADOOP-14138: - Not reading core-default was a bug, and is fixed.

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-18 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15972796#comment-15972796 ] Steve Loughran commented on HADOOP-14138: - The problem with the current mechanism is that it

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-17 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15971800#comment-15971800 ] Siddharth Seth commented on HADOOP-14138: - bq. I do not want to replicate such ugliness

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-07 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15960913#comment-15960913 ] Steve Loughran commented on HADOOP-14138: - It doesn't get special treatment: all the filesystems

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-06 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15960141#comment-15960141 ] Siddharth Seth commented on HADOOP-14138: - [~steve_l] - I understand the mechanics behind

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-06 Thread Mingliang Liu (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15959622#comment-15959622 ] Mingliang Liu commented on HADOOP-14138: Steve gave us very good reasons behind the fact. I

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-06 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15958744#comment-15958744 ] Steve Loughran commented on HADOOP-14138: - bq. why should s3a entries exist in core-default.xml?

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-05 Thread Siddharth Seth (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15957261#comment-15957261 ] Siddharth Seth commented on HADOOP-14138: - [~steve_l] - why should s3a entries exist in

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-04-05 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15956528#comment-15956528 ] Steve Loughran commented on HADOOP-14138: - Just flagging that this change broke some downstream

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-03-02 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15893132#comment-15893132 ] Hudson commented on HADOOP-14138: - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11333 (See

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-03-02 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15893065#comment-15893065 ] Hadoop QA commented on HADOOP-14138: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-03-02 Thread Mingliang Liu (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15892955#comment-15892955 ] Mingliang Liu commented on HADOOP-14138: +1 > Remove S3A ref from META-INF service discovery,

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-03-02 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15892850#comment-15892850 ] Steve Loughran commented on HADOOP-14138: - Thanks why against branch-2? I went for branch-2 is

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-03-02 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15892569#comment-15892569 ] Jason Lowe commented on HADOOP-14138: - +1 patch looks good to me. I agree that this would be a good

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-03-02 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15892196#comment-15892196 ] Steve Loughran commented on HADOOP-14138: - Did run one test with Filesystem log to DEBUG, so

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-03-02 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15892115#comment-15892115 ] Steve Loughran commented on HADOOP-14138: - No tests needed, because removing the service entry

[jira] [Commented] (HADOOP-14138) Remove S3A ref from META-INF service discovery, rely on existing core-default entry

2017-03-02 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HADOOP-14138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15892100#comment-15892100 ] Hadoop QA commented on HADOOP-14138: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||