lucenenet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Van Den Berghe, Vincent" <Vincent.VanDenBer...@bvdinfo.com>
Subject RE: [GitHub] lucenenet pull request #208: LUCENE-5644: switch to simpler LIFO thread to T...
Date Tue, 13 Jun 2017 07:09:16 GMT
Yes. I don't know why this appeared together with my pull request. 
I don't understand this system.

Vincent

-----Original Message-----
From: synhershko [mailto:git@git.apache.org] 
Sent: Tuesday, June 13, 2017 9:06 AM
To: dev@lucenenet.apache.org
Subject: [GitHub] lucenenet pull request #208: LUCENE-5644: switch to simpler LIFO thread
to T...

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

    https://github.com/apache/lucenenet/pull/208#discussion_r121596530
  
    --- Diff: src/Lucene.Net/Document/Document.cs ---
    @@ -278,9 +279,17 @@ public string Get(string name)
                 return null;
             }
     
    -        /// <summary>
    -        /// Prints the fields of a document for human consumption. </summary>
    -        public override string ToString()
    +		/// <summary>
    +		/// Remove all fields from a document
    +		/// </summary>
    +		public void Clear()
    +		{
    +			fields.Clear();
    +		}
    --- End diff --
    
    this seems unrelated?


---
If your project is set up for it, you can reply to this email and have your reply appear on
GitHub as well. If your project does not have this feature enabled and wishes so, or if the
feature is enabled but not working, please contact infrastructure at infrastructure@apache.org
or file a JIRA ticket with INFRA.
---

Mime
View raw message