Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-7383

DIH: rewrite XPathEntityProcessor/RSS example as the smallest good demo possible

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 5.0, 6.0
    • Fix Version/s: 6.6, 7.0
    • Labels:
      None

      Description

      The DIH example (solr/example/example-DIH/solr/rss/conf/rss-data-config.xml) is broken again. See associated issues.

      Below is a config that should work.

      This is caused by Slashdot seemingly oscillating between RDF/RSS and pure RSS. Perhaps we should depend upon something more static, rather than an external service that is free to change as it desires.

      <dataConfig>
          <dataSource type="URLDataSource" />
          <document>
              <entity name="slashdot"
                      pk="link"
                      url="http://rss.slashdot.org/Slashdot/slashdot"
                      processor="XPathEntityProcessor"
                      forEach="/RDF/item"
                      transformer="DateFormatTransformer">
      				
                  <field column="source" xpath="/RDF/channel/title" commonField="true" />
                  <field column="source-link" xpath="/RDF/channel/link" commonField="true" />
                  <field column="subject" xpath="/RDF/channel/subject" commonField="true" />
      			
                  <field column="title" xpath="/RDF/item/title" />
                  <field column="link" xpath="/RDF/item/link" />
                  <field column="description" xpath="/RDF/item/description" />
                  <field column="creator" xpath="/RDF/item/creator" />
                  <field column="item-subject" xpath="/RDF/item/subject" />
                  <field column="date" xpath="/RDF/item/date" dateTimeFormat="yyyy-MM-dd'T'HH:mm:ss" />
                  <field column="slash-department" xpath="/RDF/item/department" />
                  <field column="slash-section" xpath="/RDF/item/section" />
                  <field column="slash-comments" xpath="/RDF/item/comments" />
              </entity>
          </document>
      </dataConfig>
      
      1. atom_20170315.tgz
        4 kB
        Alexandre Rafalovitch
      2. rss-data-config.xml
        1 kB
        Jan Høydahl
      3. SOLR-7383.patch
        367 kB
        Alexandre Rafalovitch

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -
          Show
          githubbot ASF GitHub Bot added a comment - Github user nakohdo commented on the pull request: https://github.com/apache/lucene-solr/pull/153#issuecomment-112942155 This would solve https://issues.apache.org/jira/browse/SOLR-7383
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          Looks like this fix never went in. I just tested it with 6.0 and the proposal works apart from the commonField items (/RDF/channel) ones. Which I cannot get to work whatever I do.

          Interestingly, the Reference Guide has a different version of the same file that does not match either current or proposed text. But it does have forEach="/RDF/item|/RDF/channel" which should have made commonField work (but does not seem to).

          Finally, the managed-schema file has some of the fields defined as stored=false. Which makes zero sense in terms of demonstrating that the RSS reader works and should probably be fixed as well as part of any cleanup.

          Show
          arafalov Alexandre Rafalovitch added a comment - Looks like this fix never went in. I just tested it with 6.0 and the proposal works apart from the commonField items (/RDF/channel) ones. Which I cannot get to work whatever I do. Interestingly, the Reference Guide has a different version of the same file that does not match either current or proposed text. But it does have forEach="/RDF/item|/RDF/channel" which should have made commonField work (but does not seem to). Finally, the managed-schema file has some of the fields defined as stored=false. Which makes zero sense in terms of demonstrating that the RSS reader works and should probably be fixed as well as part of any cleanup.
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          This is still broken, so we have a dead example in the distribution.

          I propose to move away from Slashdot as a source and switch to Stack Overflow feed for the Solr tag. And rename the example to ATOM as that's the feed format.

          Show
          arafalov Alexandre Rafalovitch added a comment - This is still broken, so we have a dead example in the distribution. I propose to move away from Slashdot as a source and switch to Stack Overflow feed for the Solr tag . And rename the example to ATOM as that's the feed format.
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          After some testing, even with corrected (RDF) field definition, the commonField mapping is a problem for several reasons:

          • commonFields are set in postTransform, which is not called if there are no (normal) fields. So, the /RDF/channel record which does not match any normal fields is skipped
          • even if normal fields are matched for some reason, they will still be skipped unless the primary key record is matched

          the commonField is not really described anywhere and is not tested.

          In summary, we have a dead example on our hands.

          Show
          arafalov Alexandre Rafalovitch added a comment - After some testing, even with corrected (RDF) field definition, the commonField mapping is a problem for several reasons: commonFields are set in postTransform, which is not called if there are no (normal) fields. So, the /RDF/channel record which does not match any normal fields is skipped even if normal fields are matched for some reason, they will still be skipped unless the primary key record is matched the commonField is not really described anywhere and is not tested. In summary, we have a dead example on our hands.
          Hide
          janhoy Jan Høydahl added a comment -

          +1 to go with http://stackoverflow.com/feeds/tag/solr. Do you have a patch?

          Show
          janhoy Jan Høydahl added a comment - +1 to go with http://stackoverflow.com/feeds/tag/solr . Do you have a patch?
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          Don't have a patch yet. Just preliminary tests a while ago. I'll look into this again soon.

          Show
          arafalov Alexandre Rafalovitch added a comment - Don't have a patch yet. Just preliminary tests a while ago. I'll look into this again soon.
          Hide
          janhoy Jan Høydahl added a comment -

          In the meantime I fixed the SlashDot paths (mainly replacing s/rss/RDF/g). Attached.

          Show
          janhoy Jan Høydahl added a comment - In the meantime I fixed the SlashDot paths (mainly replacing s/rss/RDF/g ). Attached.
          Hide
          janhoy Jan Høydahl added a comment -

          Also I find it strange that the RSS example's managed-schema has stored=false for the explicitly mapped fields

          • item-subject
          • date
          • slash-department
          • slash-section
          • slash-comments

          It would not have been a problem if the schema was updated with docValues=true for string, int and date, then we would have pulled stored fields from DV, but that is not the case here.

          We should have some way of auto-including all the standard primitive types, something like:

          <initPrimitiveTypes types="all|none|int|tint|float|tfloat|long|tlong|date|tdate|boolean...."/>
          

          Perhaps such a setting could be made default for the new schema version with PointType, and that you had to say <initPrimitiveTypes types="none"/> to disable? And if the schema explicitly redefines one of them, it could take precedence but print warning in logs.

          Show
          janhoy Jan Høydahl added a comment - Also I find it strange that the RSS example's managed-schema has stored=false for the explicitly mapped fields item-subject date slash-department slash-section slash-comments It would not have been a problem if the schema was updated with docValues=true for string, int and date, then we would have pulled stored fields from DV, but that is not the case here. We should have some way of auto-including all the standard primitive types, something like: <initPrimitiveTypes types= "all|none|int|tint|float|tfloat|long|tlong|date|tdate|boolean...." /> Perhaps such a setting could be made default for the new schema version with PointType, and that you had to say <initPrimitiveTypes types="none"/> to disable? And if the schema explicitly redefines one of them, it could take precedence but print warning in logs.
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          Attached is a replacement example that uses StackOverflow ATOM feed and demonstrates ALL and more features than the original RSS example (as far as I can tell). And some features (e.g. commonField) now actually work.

          It has a different directory name, so can be decompressed alongside other DIH examples.

          It is not cleaned up, as I need to double-check camelCases vs dashes vs underscores, spaces vs tabs and maybe another comment or two (and removing checkist comment at the top of DIH definition file)

          But it should work and demonstrate a nice example. The solrconfig.xml file is super-minimal similar to work in SOLR-9601. It also uses new updateProcessors syntax.

          If this looks good, then RSS example will just be deleted and this will be the new one.

          I will appreciate the reviews and comments, as this example is 15! times smaller than the RSS one.

          Show
          arafalov Alexandre Rafalovitch added a comment - Attached is a replacement example that uses StackOverflow ATOM feed and demonstrates ALL and more features than the original RSS example (as far as I can tell). And some features (e.g. commonField) now actually work. It has a different directory name, so can be decompressed alongside other DIH examples. It is not cleaned up, as I need to double-check camelCases vs dashes vs underscores, spaces vs tabs and maybe another comment or two (and removing checkist comment at the top of DIH definition file) But it should work and demonstrate a nice example. The solrconfig.xml file is super-minimal similar to work in SOLR-9601 . It also uses new updateProcessors syntax. If this looks good, then RSS example will just be deleted and this will be the new one. I will appreciate the reviews and comments, as this example is 15! times smaller than the RSS one.
          Hide
          janhoy Jan Høydahl added a comment -

          Nice and small! Did not test it though

          Show
          janhoy Jan Høydahl added a comment - Nice and small! Did not test it though
          Hide
          varunthacker Varun Thacker added a comment -

          Hi Alaxandre,

          This is great! Thanks for taking this up!

          I'm curious as to why the core.properties file is empty in the tar that you uploaded. Even the existing rss example is has an empty core.properties . Maybe I am missing something here?

          I personally don't like the concept of these catch all fields. I understand that this is helpful as "/select" can then use "df=text"
          The alternate solution is : remove all the copy all fields and in the "/select" handler use edismax and define "qf" with the list of fields. I t personally would like this better but if you like the current solution then lets stick with that.

          I would change these three fieldTypes

              <fieldType name="int" class="solr.TrieIntField" precisionStep="0" positionIncrementGap="0"/>
              <fieldType name="tdate" class="solr.TrieDateField" precisionStep="6" positionIncrementGap="0"/>
              <fieldType name="string" class="solr.StrField" sortMissingLast="true" />
          
          • In the string fieldType we should add docValues
          • For the int and tdate we can change them to using points
          • Maybe remove "*_tdt" or change it to points?
              <fieldType name="pint" class="solr.IntPointField" docValues="true"/>
              <fieldType name="pdate" class="solr.DatePointField" docValues="true"/>
              <fieldType name="string" class="solr.StrField" sortMissingLast="true" docValues="true"/>
          

          Last thing I can think of is simplifying text_en_splitting

          • Can we remove KeywordMarkerFilterFactory and thereby the protwords.txt
          • Also I'd imagine that porter stemmer in technical post summary search would be bad?
          • I haven't actually used the example so this might not apply . Do we need to strip out html ? When I see a sample summary on http://stackoverflow.com/feeds/tag/solr I see html chars in there.
          Show
          varunthacker Varun Thacker added a comment - Hi Alaxandre, This is great! Thanks for taking this up! I'm curious as to why the core.properties file is empty in the tar that you uploaded. Even the existing rss example is has an empty core.properties . Maybe I am missing something here? I personally don't like the concept of these catch all fields. I understand that this is helpful as "/select" can then use "df=text" The alternate solution is : remove all the copy all fields and in the "/select" handler use edismax and define "qf" with the list of fields. I t personally would like this better but if you like the current solution then lets stick with that. I would change these three fieldTypes <fieldType name= " int " class= "solr.TrieIntField" precisionStep= "0" positionIncrementGap= "0" /> <fieldType name= "tdate" class= "solr.TrieDateField" precisionStep= "6" positionIncrementGap= "0" /> <fieldType name= "string" class= "solr.StrField" sortMissingLast= " true " /> In the string fieldType we should add docValues For the int and tdate we can change them to using points Maybe remove "*_tdt" or change it to points? <fieldType name= "pint" class= "solr.IntPointField" docValues= " true " /> <fieldType name= "pdate" class= "solr.DatePointField" docValues= " true " /> <fieldType name= "string" class= "solr.StrField" sortMissingLast= " true " docValues= " true " /> Last thing I can think of is simplifying text_en_splitting Can we remove KeywordMarkerFilterFactory and thereby the protwords.txt Also I'd imagine that porter stemmer in technical post summary search would be bad? I haven't actually used the example so this might not apply . Do we need to strip out html ? When I see a sample summary on http://stackoverflow.com/feeds/tag/solr I see html chars in there.
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          Varun, thank you for the comments.

          I'm curious as to why the core.properties file is empty in the tar that you uploaded. Even the existing rss example is has an empty core.properties . Maybe I am missing something here?

          What would you expect in that file? The core name is by default the same as directory name. File is present, so Solr autodiscovers the core on startup, but there is no need for any extra configuration.

          I personally don't like the concept of these catch all fields. I understand that this is helpful as "/select" can then use "df=text"

          If we switch to eDisMax to search the original fields, then the string fields such as author will not be easily searchable and/or will require a secondary copy into a text field to be searched properly. As it is, one could facet on string field and search on catch-all text field.

          I would change these three fieldTypes

          I will look into that. I don't know much about points for now, so this is definitely a good suggestion to check.

          simplifying text_en_splitting

          I did not want to create another type unless needed (that was my big problem with Tika example), so instead I have kept the protwords.txt and put 'lucene' in there. However, if other type is better I have no objections.

          Do we need to strip out html ? When I see a sample summary on http://stackoverflow.com/feeds/tag/solr I see html chars in there.

          The HTML is stripped by using two DIH transformers, so the text ends up without any HTML. There is also a new-style URP in solrconfig.xml to trim the post-DIH whitespace and - importantly in my opinion - to show that it is possible to have URPs with DIH. The stored summary field content at the end looks quite presentable.

          Show
          arafalov Alexandre Rafalovitch added a comment - Varun, thank you for the comments. I'm curious as to why the core.properties file is empty in the tar that you uploaded. Even the existing rss example is has an empty core.properties . Maybe I am missing something here? What would you expect in that file? The core name is by default the same as directory name. File is present, so Solr autodiscovers the core on startup, but there is no need for any extra configuration. I personally don't like the concept of these catch all fields. I understand that this is helpful as "/select" can then use "df=text" If we switch to eDisMax to search the original fields, then the string fields such as author will not be easily searchable and/or will require a secondary copy into a text field to be searched properly. As it is, one could facet on string field and search on catch-all text field. I would change these three fieldTypes I will look into that. I don't know much about points for now, so this is definitely a good suggestion to check. simplifying text_en_splitting I did not want to create another type unless needed (that was my big problem with Tika example), so instead I have kept the protwords.txt and put 'lucene' in there. However, if other type is better I have no objections. Do we need to strip out html ? When I see a sample summary on http://stackoverflow.com/feeds/tag/solr I see html chars in there. The HTML is stripped by using two DIH transformers, so the text ends up without any HTML. There is also a new-style URP in solrconfig.xml to trim the post-DIH whitespace and - importantly in my opinion - to show that it is possible to have URPs with DIH. The stored summary field content at the end looks quite presentable.
          Hide
          varunthacker Varun Thacker added a comment -

          What would you expect in that file? The core name is by default the same as directory name.

          Yeah i wasn't sure if that logic still holds true . Like when we start bin/solr -e techproducts the core.properties files writes out the core name. I used to think that today that's the only way solr discovers core names but I am probably wrong. Anyways not an issue here.

          If we switch to eDisMax to search the original fields, then the string fields such as author will not be easily searchable and/or will require a secondary copy into a text field to be searched properly.

          Fair point. Let's stick to the current approach only then. The edismax idea will bring more complexity than good here.

          Can we also update the README under example-DIH ? More specifically this section

          See also README.txt in the solr subdirectory, and check
          http://wiki.apache.org/solr/DataImportHandler for detailed
          usage guide and tutorial.
          

          I'm not sure what README it's referring to ? And instead of the wiki maybe point users to this : https://cwiki.apache.org/confluence/display/solr/Uploading+Structured+Data+Store+Data+with+the+Data+Import+Handler ?

          Show
          varunthacker Varun Thacker added a comment - What would you expect in that file? The core name is by default the same as directory name. Yeah i wasn't sure if that logic still holds true . Like when we start bin/solr -e techproducts the core.properties files writes out the core name. I used to think that today that's the only way solr discovers core names but I am probably wrong. Anyways not an issue here. If we switch to eDisMax to search the original fields, then the string fields such as author will not be easily searchable and/or will require a secondary copy into a text field to be searched properly. Fair point. Let's stick to the current approach only then. The edismax idea will bring more complexity than good here. Can we also update the README under example-DIH ? More specifically this section See also README.txt in the solr subdirectory, and check http: //wiki.apache.org/solr/DataImportHandler for detailed usage guide and tutorial. I'm not sure what README it's referring to ? And instead of the wiki maybe point users to this : https://cwiki.apache.org/confluence/display/solr/Uploading+Structured+Data+Store+Data+with+the+Data+Import+Handler ?
          Hide
          erickerickson Erick Erickson added a comment -

          The name property in the core.properties file overrides the default, which is to use the same name as the parent directory.

          I think the techproducts example creates a core programmatically, then persists core.properties. Since the core has a name, the name property gets written.

          But the name property is certainly not necessary.

          FWIW.

          Show
          erickerickson Erick Erickson added a comment - The name property in the core.properties file overrides the default, which is to use the same name as the parent directory. I think the techproducts example creates a core programmatically, then persists core.properties. Since the core has a name, the name property gets written. But the name property is certainly not necessary. FWIW.
          Hide
          varunthacker Varun Thacker added a comment -

          Hi Alexandre,

          From my perspective this example looms great. We should just change the Trie field to Point variant and then commit

          Did you have anything in mind apart from this that needs to be addressed?

          Show
          varunthacker Varun Thacker added a comment - Hi Alexandre, From my perspective this example looms great. We should just change the Trie field to Point variant and then commit Did you have anything in mind apart from this that needs to be addressed?
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          Nothing else. Just change type, retest, cleanup the spaces and README file reference (RSS->ATOM), Changes.txt note and I think it is good to go. I am hoping to commit it this weekend.

          Show
          arafalov Alexandre Rafalovitch added a comment - Nothing else. Just change type, retest, cleanup the spaces and README file reference (RSS->ATOM), Changes.txt note and I think it is good to go. I am hoping to commit it this weekend.
          Hide
          varunthacker Varun Thacker added a comment -

          Great!

          Show
          varunthacker Varun Thacker added a comment - Great!
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          Final patch is attached. It switches to point types, fixes recopies text_en_splitting definition to take into account Graph filters (SOLR-10344) and reorders fields in schema and DIH definition to match the order.

          The rss example directory is deleted. The new directory is atom. All that's left to do is CHANGES file text, which I can commit directly.

          Show
          arafalov Alexandre Rafalovitch added a comment - Final patch is attached. It switches to point types, fixes recopies text_en_splitting definition to take into account Graph filters ( SOLR-10344 ) and reorders fields in schema and DIH definition to match the order. The rss example directory is deleted. The new directory is atom . All that's left to do is CHANGES file text, which I can commit directly.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 580f6e98fb033dbbb8e0921fc3175021714ce956 in lucene-solr's branch refs/heads/master from Alexandre Rafalovitch
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=580f6e9 ]

          SOLR-7383: Replace DIH 'rss' example with 'atom'
          rss example was broken for multiple reasons.
          atom example showcases the same - and more - features
          and uses the smallest config file needed to make it work.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 580f6e98fb033dbbb8e0921fc3175021714ce956 in lucene-solr's branch refs/heads/master from Alexandre Rafalovitch [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=580f6e9 ] SOLR-7383 : Replace DIH 'rss' example with 'atom' rss example was broken for multiple reasons. atom example showcases the same - and more - features and uses the smallest config file needed to make it work.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit e987654aa31554fd27f3110d7def3eb782e5c199 in lucene-solr's branch refs/heads/branch_6x from Alexandre Rafalovitch
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=e987654 ]

          SOLR-7383: Replace DIH 'rss' example with 'atom'
          rss example was broken for multiple reasons.
          atom example showcases the same - and more - features
          and uses the smallest config file needed to make it work.

          Show
          jira-bot ASF subversion and git services added a comment - Commit e987654aa31554fd27f3110d7def3eb782e5c199 in lucene-solr's branch refs/heads/branch_6x from Alexandre Rafalovitch [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=e987654 ] SOLR-7383 : Replace DIH 'rss' example with 'atom' rss example was broken for multiple reasons. atom example showcases the same - and more - features and uses the smallest config file needed to make it work.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 2319d69fd3d5b67729f31b5796cc1eb68220b664 in lucene-solr's branch refs/heads/master from Cassandra Targett
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=2319d69 ]

          Ref Guide: update DIH docs for SOLR-7383; SOLR-9601; plus major surgery on page layout

          Show
          jira-bot ASF subversion and git services added a comment - Commit 2319d69fd3d5b67729f31b5796cc1eb68220b664 in lucene-solr's branch refs/heads/master from Cassandra Targett [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=2319d69 ] Ref Guide: update DIH docs for SOLR-7383 ; SOLR-9601 ; plus major surgery on page layout
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 2d054965a5c5313a486540c79ef29b0dbf05bc70 in lucene-solr's branch refs/heads/branch_6x from Cassandra Targett
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=2d05496 ]

          Ref Guide: update DIH docs for SOLR-7383; SOLR-9601; plus major surgery on page layout

          Show
          jira-bot ASF subversion and git services added a comment - Commit 2d054965a5c5313a486540c79ef29b0dbf05bc70 in lucene-solr's branch refs/heads/branch_6x from Cassandra Targett [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=2d05496 ] Ref Guide: update DIH docs for SOLR-7383 ; SOLR-9601 ; plus major surgery on page layout
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit fd8ac5b959f26c8a979752c9bf61bb8a545b2e3a in lucene-solr's branch refs/heads/branch_6_6 from Cassandra Targett
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=fd8ac5b ]

          Ref Guide: update DIH docs for SOLR-7383; SOLR-9601; plus major surgery on page layout

          Show
          jira-bot ASF subversion and git services added a comment - Commit fd8ac5b959f26c8a979752c9bf61bb8a545b2e3a in lucene-solr's branch refs/heads/branch_6_6 from Cassandra Targett [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=fd8ac5b ] Ref Guide: update DIH docs for SOLR-7383 ; SOLR-9601 ; plus major surgery on page layout
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 10d3aa23fe3d12c652de8c8e3f434e8c83bdbe0d in lucene-solr's branch refs/heads/master from Alexandre Rafalovitch
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=10d3aa2 ]

          SOLR-7383: Switch to https for StackOverflow
          Using http no longer works

          Show
          jira-bot ASF subversion and git services added a comment - Commit 10d3aa23fe3d12c652de8c8e3f434e8c83bdbe0d in lucene-solr's branch refs/heads/master from Alexandre Rafalovitch [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=10d3aa2 ] SOLR-7383 : Switch to https for StackOverflow Using http no longer works
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 8efaefacc2d21fbeadcefa07577d765e19b21f04 in lucene-solr's branch refs/heads/branch_6x from Alexandre Rafalovitch
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=8efaefa ]

          SOLR-7383: Switch to https for StackOverflow
          Using http no longer works

          Show
          jira-bot ASF subversion and git services added a comment - Commit 8efaefacc2d21fbeadcefa07577d765e19b21f04 in lucene-solr's branch refs/heads/branch_6x from Alexandre Rafalovitch [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=8efaefa ] SOLR-7383 : Switch to https for StackOverflow Using http no longer works
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 15e2567b0f78aba341e2608ea5265820ba4daa46 in lucene-solr's branch refs/heads/branch_6_6 from Alexandre Rafalovitch
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=15e2567 ]

          SOLR-7383: Switch to https for StackOverflow
          Using http no longer works

          Show
          jira-bot ASF subversion and git services added a comment - Commit 15e2567b0f78aba341e2608ea5265820ba4daa46 in lucene-solr's branch refs/heads/branch_6_6 from Alexandre Rafalovitch [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=15e2567 ] SOLR-7383 : Switch to https for StackOverflow Using http no longer works
          Hide
          ctargett Cassandra Targett added a comment -

          Alexandre Rafalovitch - Do you intend to update the example in the Ref Guide for the most recent change? I updated the DIH page extensively the other day, and included the new example for atom (see earlier commit notifications right before yours), so now that needs to be updated also.

          Show
          ctargett Cassandra Targett added a comment - Alexandre Rafalovitch - Do you intend to update the example in the Ref Guide for the most recent change? I updated the DIH page extensively the other day, and included the new example for atom (see earlier commit notifications right before yours), so now that needs to be updated also.
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          I will not have time until the weekend, unfortunately. Yesterday's work was an emergency 1am intervention to prevent a DOA example going out.

          Show
          arafalov Alexandre Rafalovitch added a comment - I will not have time until the weekend, unfortunately. Yesterday's work was an emergency 1am intervention to prevent a DOA example going out.
          Hide
          ctargett Cassandra Targett added a comment -

          OK, I'll fix it this time. But please accept a gentle reminder that it's now dead-simple to include documentation changes with code changes, even in an emergency.

          Show
          ctargett Cassandra Targett added a comment - OK, I'll fix it this time. But please accept a gentle reminder that it's now dead-simple to include documentation changes with code changes, even in an emergency.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit b3024d67cae0f2c9bbfb9bdf897c9b43d6ab8926 in lucene-solr's branch refs/heads/master from Cassandra Targett
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=b3024d6 ]

          Ref Guide: fix atom example for SOLR-7383

          Show
          jira-bot ASF subversion and git services added a comment - Commit b3024d67cae0f2c9bbfb9bdf897c9b43d6ab8926 in lucene-solr's branch refs/heads/master from Cassandra Targett [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=b3024d6 ] Ref Guide: fix atom example for SOLR-7383
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 17f565c71af875d95a47c81894a816159ba5a981 in lucene-solr's branch refs/heads/branch_6x from Cassandra Targett
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=17f565c ]

          Ref Guide: fix atom example for SOLR-7383

          Show
          jira-bot ASF subversion and git services added a comment - Commit 17f565c71af875d95a47c81894a816159ba5a981 in lucene-solr's branch refs/heads/branch_6x from Cassandra Targett [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=17f565c ] Ref Guide: fix atom example for SOLR-7383
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 2bc88b3df20f3367b13055aafe64da42e467790b in lucene-solr's branch refs/heads/branch_6_6 from Cassandra Targett
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=2bc88b3 ]

          Ref Guide: fix atom example for SOLR-7383

          Show
          jira-bot ASF subversion and git services added a comment - Commit 2bc88b3df20f3367b13055aafe64da42e467790b in lucene-solr's branch refs/heads/branch_6_6 from Cassandra Targett [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=2bc88b3 ] Ref Guide: fix atom example for SOLR-7383
          Hide
          arafalov Alexandre Rafalovitch added a comment -

          Gentle reminder accepted. Thank you for stepping in for now.

          Show
          arafalov Alexandre Rafalovitch added a comment - Gentle reminder accepted. Thank you for stepping in for now.

            People

            • Assignee:
              arafalov Alexandre Rafalovitch
              Reporter:
              upayavira Upayavira
            • Votes:
              1 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development