[jira] [Updated] (LUCENE-10251) Add the compliant "Direct" doc-value format

2021-12-31 Thread spike liu (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10251?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] spike liu updated LUCENE-10251: --- Attachment: LUCENE-10251-20211231.patch > Add the compliant "Direct" doc

[GitHub] [lucene] mocobeta edited a comment on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
mocobeta edited a comment on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003258524 > There are some remaining issues with code doing: Class.getClassLoader().getResource This code does not even work from inside the same module and always returns null.

[GitHub] [lucene] dweiss commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
dweiss commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003310361 Yeah - we came up with the same explanation independently, Uwe. In my opinion a simpler API that does not require special module-info clauses is a preference (if it can be done in

[GitHub] [lucene] mocobeta edited a comment on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
mocobeta edited a comment on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003258524 > There are some remaining issues with code doing: Class.getClassLoader().getResource This code does not even work from inside the same module and always returns null.

[GitHub] [lucene] mocobeta edited a comment on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
mocobeta edited a comment on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003258524 > There are some remaining issues with code doing: Class.getClassLoader().getResource This code does not even work from inside the same module and always returns null.

[GitHub] [lucene-solr] cpoerschke closed pull request #1286: SOLR-14279: remove CSVStrategy's deprecated setters

2021-12-31 Thread GitBox
cpoerschke closed pull request #1286: URL: https://github.com/apache/lucene-solr/pull/1286 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-

[GitHub] [lucene-solr] cpoerschke commented on pull request #1286: SOLR-14279: remove CSVStrategy's deprecated setters

2021-12-31 Thread GitBox
cpoerschke commented on pull request #1286: URL: https://github.com/apache/lucene-solr/pull/1286#issuecomment-1003327759 Replaced by https://github.com/apache/solr/pull/480 PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [lucene] uschindler commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003328772 > Yeah - we came up with the same explanation independently, Uwe. In my opinion a simpler API that does not require special module-info clauses is a preference (if it can be do

[GitHub] [lucene] uschindler commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003330092 Hi @mocobeta, > > There are some remaining issues with code doing: Class.getClassLoader().getResource This code does not even work from inside the same module and al

[GitHub] [lucene] uschindler commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003330472 I changed back to "draft", once I am done, I switch back! We can then merge to main and or your branch and solve conflicts. -- This is an automated message from the Apa

[GitHub] [lucene] mocobeta commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
mocobeta commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003341163 > The problem with your patch is that you change public API to suddenly use relative resource names in public ctors. Instead we should deprecate those and add InputStream taking

[GitHub] [lucene] mocobeta edited a comment on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
mocobeta edited a comment on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003341163 > The problem with your patch is that you change public API to suddenly use relative resource names in public ctors. Instead we should deprecate those and add InputStream

[GitHub] [lucene] uschindler commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003343605 > > The problem with your patch is that you change public API to suddenly use relative resource names in public ctors. Instead we should deprecate those and add InputStream tak

[GitHub] [lucene] uschindler commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003345402 > > > The problem with your patch is that you change public API to suddenly use relative resource names in public ctors. Instead we should deprecate those and add InputStream t

[GitHub] [lucene] uschindler commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003358980 Hi @mocobeta, I cherrypicked your commit and modified it a bit: To still support full names, it just prefixes all paths with "/". After doing this I was able to revert the

[GitHub] [lucene] uschindler commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003365974 > In general a simple hotfix rule for `Class#getClassLoader().getResource(xxx)` is: Replace by `Class#getResource("/".concat(xxx))` An alternative is `Class#getModule().

[GitHub] [lucene] uschindler edited a comment on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler edited a comment on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003365974 > In general a simple hotfix rule for `Class#getClassLoader().getResource(xxx)` is: Replace by `Class#getResource("/".concat(xxx))` An alternative is `Class#getMo

[GitHub] [lucene] uschindler edited a comment on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler edited a comment on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003365974 > In general a simple hotfix rule for `Class#getClassLoader().getResource(xxx)` is: Replace by `Class#getResource("/".concat(xxx))` An alternative is `Class#getMo

[GitHub] [lucene] mocobeta commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
mocobeta commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003369484 @uschindler Thank you. I looked through the changes. I can't look closely at or review the whole change soon since I have to leave for a while (my father just passed away due

[GitHub] [lucene] uschindler commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
uschindler commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003380156 Hi, > @uschindler Thank you. I looked through the changes. I can't look closely at or review the whole change soon since I have to leave for a while (my father just pass

[GitHub] [lucene] mocobeta commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
mocobeta commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003386091 Thank you, Uwe. I am grateful for your kind concern. Wish you all a Happy New Year! -- This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [lucene] rmuir commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
rmuir commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003404788 > @uschindler Thank you. I looked through the changes. > I can't look closely at or review the whole change soon since I have to leave for a while (my father just passed away due

[jira] [Created] (LUCENE-10346) Specially treat SingletonSortedNumericDocValues in FastTaxonomyFacetCounts#countAll()

2021-12-31 Thread Feng Guo (Jira)
Feng Guo created LUCENE-10346: - Summary: Specially treat SingletonSortedNumericDocValues in FastTaxonomyFacetCounts#countAll() Key: LUCENE-10346 URL: https://issues.apache.org/jira/browse/LUCENE-10346 Pro

[GitHub] [lucene] gf2121 opened a new pull request #574: LUCENE-10346: unwrap

2021-12-31 Thread GitBox
gf2121 opened a new pull request #574: URL: https://github.com/apache/lucene/pull/574 CPU profile often tells SingletonSortedNumericDocValues#nextDoc() is using a high percentage of CPU when running luceneutil, but the nextDoc() of dense cases should be rather simple. So I suspect that it

[jira] [Updated] (LUCENE-10346) Specially treat SingletonSortedNumericDocValues in FastTaxonomyFacetCounts#countAll()

2021-12-31 Thread Feng Guo (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10346?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Feng Guo updated LUCENE-10346: -- Description: CPU profile often tells {{SingletonSortedNumericDocValues#nextDoc()}} is using a high p

[GitHub] [lucene] mayya-sharipova commented on pull request #536: Don't store graph offsets for HNSW graph

2021-12-31 Thread GitBox
mayya-sharipova commented on pull request #536: URL: https://github.com/apache/lucene/pull/536#issuecomment-1003458911 @msokolov > I seem to remember that when I checked (you can use -fanout parameter to KnnGraphTester IIRC) most nodes were not fully populated; ie they had fewer th

[GitHub] [lucene] mayya-sharipova commented on pull request #536: Don't store graph offsets for HNSW graph

2021-12-31 Thread GitBox
mayya-sharipova commented on pull request #536: URL: https://github.com/apache/lucene/pull/536#issuecomment-1003460979 > I think it would be prudent to check the size increase/decrease from this change for some dataset/parameter choices I've checked the index sizes and the size actu

[GitHub] [lucene] mayya-sharipova edited a comment on pull request #536: Don't store graph offsets for HNSW graph

2021-12-31 Thread GitBox
mayya-sharipova edited a comment on pull request #536: URL: https://github.com/apache/lucene/pull/536#issuecomment-1003460979 > I think it would be prudent to check the size increase/decrease from this change for some dataset/parameter choices I've checked the index sizes and the si

[GitHub] [lucene] rmuir commented on pull request #574: LUCENE-10346: Specially treat SingletonSortedNumericDocValues in FastTaxonomyFacetCounts#countAll()

2021-12-31 Thread GitBox
rmuir commented on pull request #574: URL: https://github.com/apache/lucene/pull/574#issuecomment-1003466563 very interesting (and simple contained) find. cc @gsmiller @mikemccand , I think this is an easy win? Thank you @gf2121 ! -- This is an automated message from the Apache Git Servi

[GitHub] [lucene] rmuir commented on pull request #574: LUCENE-10346: Specially treat SingletonSortedNumericDocValues in FastTaxonomyFacetCounts#countAll()

2021-12-31 Thread GitBox
rmuir commented on pull request #574: URL: https://github.com/apache/lucene/pull/574#issuecomment-1003467123 This actually puts the taxo facets as running faster than SSDV facets? I wonder if the same optimization is missing in SSDV facets countall? Or perhaps there is a more complex expla

[GitHub] [lucene] mocobeta commented on pull request #567: LUCENE-10335: Deprecate broken helper methods not compatible with module system; add ModuleResourceLoader and add integration test

2021-12-31 Thread GitBox
mocobeta commented on pull request #567: URL: https://github.com/apache/lucene/pull/567#issuecomment-1003475498 @rmuir Thank you, for your concern. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [lucene] gf2121 commented on pull request #574: LUCENE-10346: Specially treat SingletonSortedNumericDocValues in FastTaxonomyFacetCounts#countAll()

2021-12-31 Thread GitBox
gf2121 commented on pull request #574: URL: https://github.com/apache/lucene/pull/574#issuecomment-1003514413 Thanks @rmuir for so quick reply! Happy New Year :) > just wondering if we miss to optimize the single-valued situation for SSDV facets? Thanks for remiding! I find we