[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[
https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15940697#comment-15940697
]
Steve Rowe commented on SOLR-9221:
--
the git bot failed to post the branch_6x commit - here it is from the commit
notification email:
{noformat}
SOLR-9221: Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
Conflicts:
solr/contrib/map-reduce/src/java/org/apache/solr/hadoop/TreeMergeOutputFormat.java
solr/contrib/map-reduce/src/test/org/apache/solr/hadoop/MorphlineMapperTest.java
solr/contrib/map-reduce/src/test/org/apache/solr/hadoop/MorphlineReducerTest.java
solr/contrib/morphlines-core/src/java/org/apache/solr/morphlines/solr/LoadSolrBuilder.java
solr/contrib/morphlines-core/src/test/org/apache/solr/morphlines/solr/AbstractSolrMorphlineTestBase.java
solr/contrib/morphlines-core/src/test/org/apache/solr/morphlines/solr/AbstractSolrMorphlineZkTestBase.java
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/ac221b96
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/ac221b96
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/ac221b96
Branch: refs/heads/branch_6x
Commit: ac221b96b6d16569ca17e37cbebe717f7b86484c
Parents: 2adbd76
Author: Steve Rowe
Authored: Fri Mar 24 12:31:16 2017 -0400
Committer: Steve Rowe
Committed: Fri Mar 24 12:35:46 2017 -0400
{noformat}
> Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
> -
>
> Key: SOLR-9221
> URL: https://issues.apache.org/jira/browse/SOLR-9221
> Project: Solr
> Issue Type: Task
> Components: contrib - MapReduce, contrib - morphlines-cell, contrib
> - morphlines-core
>Reporter: Steve Rowe
>Assignee: Steve Rowe
>Priority: Minor
> Fix For: master (7.0), 6.6
>
> Attachments: SOLR-9221.patch, SOLR-9221.patch, SOLR-9221.patch,
> SOLR-9221-review.patch
>
>
> The Solr contribs map-reduce, morphlines-cell and morphlines-core contain
> tests that are not being fixed: SOLR-6489 and SOLR-9220.
> (Some subset of?) these components live in the Kite SDK: http://kitesdk.org -
> why are they also hosted in Solr?
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15940676#comment-15940676 ] ASF subversion and git services commented on SOLR-9221: --- Commit 53e5f34f66d264c8f0ea2861e77389902b2a36c4 in lucene-solr's branch refs/heads/master from [~steve_rowe] [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=53e5f34 ] SOLR-9221: Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch, SOLR-9221.patch, SOLR-9221.patch, > SOLR-9221-review.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15940639#comment-15940639 ] Steve Rowe commented on SOLR-9221: -- No objections => committing removal patch shortly. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch, SOLR-9221.patch, SOLR-9221.patch, > SOLR-9221-review.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15935668#comment-15935668 ] Mark Miller commented on SOLR-9221: --- Looks like no progress on this, so I am removing my veto to this issue. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15379535#comment-15379535 ] Mark Miller commented on SOLR-9221: --- If you go look at all the JIRA's I've worked on, I don't think you can say it's not actively maintained. Every issue may not get addressed or addressed on your timeline, but that is perfectly fine and it's BS that it's not maintained. I have worked a bit on a couple of the latest issues. We have discussed what we want to do with the contrib and it seems most likely we will either pull the tika integration contrib (morphlines-cell) or pull both morphlines contribs and have a more generic plugin point. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15341162#comment-15341162 ] Noble Paul commented on SOLR-9221: -- bq. a tool to build indexes on hdfs is a perfect Solr contrib. As long as at least one committer owns up maintenance of a component we can keep it around. It is a problem when it is not actively maintained. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338775#comment-15338775 ] Mark Miller commented on SOLR-9221: --- Sorry, a tool to build indexes on hdfs is a perfect Solr contrib. Won't remove my veto for flimsy arguments. I will if we are not going to put effort into the current issues. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338674#comment-15338674 ] David Smiley commented on SOLR-9221: bq. 3. Remove the contrib and ship the (full) Solr support with Kitesdk. I'd prefer this. The three contribs are just some client for solr, so why does it need to be inside Solr's repository? +1. There are plenty of systems out there that integrate with Solr... why should Morphlines in particular have this here? > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338458#comment-15338458 ] Uwe Schindler commented on SOLR-9221: - Hi, I see the following possibilities: # Include the whole source code of morphlines into Solr (not just a part of it). - I don't like that option, because we have missing expertise. I am also sure that Cloudera is not happy to donate all code. # Fix morphlines to not depend on Solr and TIKA directly. We have no control about this, but I am sure Mark might be able to open issues about that on kitesdk. If we would need to add the TIKA adaptors into Solr then, I would be fine. But the current dependency-hell is a no-go. # Remove the contrib and ship the (full) Solr support with Kitesdk. I'd prefer this. The three contribs are just some client for solr, so why does it need to be inside Solr's repository? I still think that #3 is the only option to go. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338430#comment-15338430 ] Noble Paul commented on SOLR-9221: -- I'm not sure morphlines stuff should be a part of Solr. It should be a part of morphlines and not Solr. There are so many tools/frameworks that provide Solr integration. If we put that code in Solr contribs it is too painful for us to manage. Most of the Solr devs have no idea about the correctness of that code because we don't know those frameworks. +1 to remove these > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338327#comment-15338327 ] Mark Miller commented on SOLR-9221: --- I willl remove my veto of we decide not to put the effort in to fix the issues, but it remains until that's determined. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338324#comment-15338324 ] Mark Miller commented on SOLR-9221: --- I'm still -1 on removing. People email me and use it, if you have an issue you want resolved and it's not ping me, but I remain a veto on removal. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (SOLR-9221) Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell
[ https://issues.apache.org/jira/browse/SOLR-9221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15338234#comment-15338234 ] Uwe Schindler commented on SOLR-9221: - +1 to remove it. I think [~noble.paul] already said in another issue that KiteSDK includes the whole Solr support, so we don't need the contribs. It just brings problems: - We cannot change some Solr APIs (removing deprecations), because the code in morphlines uses those Solr APIs (circular dependency). - We cannot update TIKA, because it depends on a really antique version of it. - There seems to be no maintainer of this contrib anymore. There are tons of issues open, and none of them are fixed. I have no idea how to fix them, as the whole thing (kite moprphlines) is a black box. > Remove Solr contribs: map-reduce, morphlines-core and morphlines-cell > - > > Key: SOLR-9221 > URL: https://issues.apache.org/jira/browse/SOLR-9221 > Project: Solr > Issue Type: Task > Components: contrib - MapReduce, contrib - morphlines-cell, contrib > - morphlines-core >Reporter: Steve Rowe >Priority: Minor > Attachments: SOLR-9221.patch > > > The Solr contribs map-reduce, morphlines-cell and morphlines-core contain > tests that are not being fixed: SOLR-6489 and SOLR-9220. > (Some subset of?) these components live in the Kite SDK: http://kitesdk.org - > why are they also hosted in Solr? -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
