lucenenet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From NightOwl888 <...@git.apache.org>
Subject [GitHub] lucenenet issue #176: Ported QueryParser
Date Mon, 08 Aug 2016 09:28:50 GMT
Github user NightOwl888 commented on the issue:

    https://github.com/apache/lucenenet/pull/176
  
    Ok, I have moved the test to Lucene.Net.Tests/core/Support. Apparently there *was* a directory
with this name, but it had been removed from the project. Not sure what the reasoning was
for this, I suppose this one doesn't belong in the project either if none of those do. But,
at least it is now with the rest of the .NET compatibility tests, rather than buried in the
Java-ported code.
    
    Feel free to remove it from the project if that is your prerogative. To me it seems like
those tests to ensure the parts of Java framework that are ported over are working are a good
idea. For example, projects that derive from this one (such as BoboBrowse.Net) might need
them, too. But if the implementation is only complete/correct enough to make Lucene.Net work,
but not work right for derived libraries, it could mean a lot of copy-paste-edit for those
derived libraries.


---
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