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

    https://github.com/apache/lucene-solr/pull/435#discussion_r209442210
  
    --- Diff: 
solr/core/src/test-files/solr/collection1/conf/solrconfig-parsing-update-processor-chains.xml
 ---
    @@ -109,6 +109,21 @@
         </processor>
       </updateRequestProcessorChain>
     
    +  <updateRequestProcessorChain name="parse-date-extraction-util-formats">
    +    <processor class="solr.ParseDateFieldUpdateProcessorFactory">
    +      <str name="defaultTimeZone">UTC</str>
    +      <str name="locale">en</str>
    +      <arr name="format">
    +        <str>yyyy-MM-dd['T'[HH:mm:ss['.'SSS]['Z'</str>
    --- End diff --
    
    After working with you on the previous issue, I now know that `'Z'` isn't 
appropriate if the defaultTimeZone is configurable, since as-such it's a 
literal that is ignored -- and the defaultTimeZone would take effect which 
might not be UTC.  We know 'z' is right now.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to