dweiss commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-180556
> I still cannot convince myself it is the best or only one solution for
that, but maybe I'd need to recognize the strict dependency management of jms
as a necessary thing for embr
Dawid Weiss created LUCENE-10337:
Summary: xercesImpl has split packages with the JDK (required
transitively by benchmarks)
Key: LUCENE-10337
URL: https://issues.apache.org/jira/browse/LUCENE-10337
Pr
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dawid Weiss updated LUCENE-10335:
-
Attachment: LUCENE-10335.patch
> IOUtils.getDecodingReader(Class, String) is broken with module
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464331#comment-17464331
]
Dawid Weiss commented on LUCENE-10335:
--
You may be right that the lambda version m
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dawid Weiss updated LUCENE-10335:
-
Attachment: LUCENE-10335-1.patch
> IOUtils.getDecodingReader(Class, String) is broken with modu
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464334#comment-17464334
]
Dawid Weiss commented on LUCENE-10335:
--
The attached seems to fix all of them and
mocobeta commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-196889
I started to feel that we may need to check if the "runtime" dependency
graph in the binary distribution is complete (or, does not lack any node).
While it can be seen as a colle
Dawid Weiss created LUCENE-10338:
Summary: Scan for tests only by convention file name pattern
(Test*)
Key: LUCENE-10338
URL: https://issues.apache.org/jira/browse/LUCENE-10338
Project: Lucene - Core
[
https://issues.apache.org/jira/browse/LUCENE-10338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464341#comment-17464341
]
Dawid Weiss commented on LUCENE-10338:
--
Hey, Uwe - this is the thing that also aff
[
https://issues.apache.org/jira/browse/LUCENE-10338?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dawid Weiss reassigned LUCENE-10338:
Assignee: Dawid Weiss
> Scan for tests only by convention file name pattern (Test*)
> --
[
https://issues.apache.org/jira/browse/LUCENE-10307?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464345#comment-17464345
]
Tomoko Uchida commented on LUCENE-10307:
Just for filing - according to the pac
uschindler commented on pull request #518:
URL: https://github.com/apache/lucene/pull/518#issuecomment-999371293
Hi, that's sorry. I just merged yesterday but didn't run tests. The test is
obsolete on branch.
--
This is an automated message from the Apache Git Service.
To respond to the
uschindler edited a comment on pull request #518:
URL: https://github.com/apache/lucene/pull/518#issuecomment-999371293
Hi @dweiss, I'm so sorry. I just merged yesterday but didn't run tests. The
test is obsolete on branch.
--
This is an automated message from the Apache Git Service.
To
gf2121 commented on pull request #557:
URL: https://github.com/apache/lucene/pull/557#issuecomment-999377609
@jpountz Thanks a lot for the suggestion!
> Let's update this pull request to limit the impact to the API to
DirectMonotonicReader, since it's the only LongValues impl where
dweiss commented on pull request #518:
URL: https://github.com/apache/lucene/pull/518#issuecomment-999377984
No worries - just tweak whatever you want. I just didn't want jenkins to
complain over and over.
--
This is an automated message from the Apache Git Service.
To respond to the mes
mocobeta commented on a change in pull request #550:
URL: https://github.com/apache/lucene/pull/550#discussion_r773693085
##
File path:
lucene/distribution.tests/src/test/org/apache/lucene/distribution/TestScripts.java
##
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the Apache Softwa
mocobeta commented on a change in pull request #550:
URL: https://github.com/apache/lucene/pull/550#discussion_r773693085
##
File path:
lucene/distribution.tests/src/test/org/apache/lucene/distribution/TestScripts.java
##
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the Apache Softwa
gf2121 commented on a change in pull request #557:
URL: https://github.com/apache/lucene/pull/557#discussion_r773692776
##
File path:
lucene/core/src/java/org/apache/lucene/util/packed/DirectMonotonicReader.java
##
@@ -157,11 +159,29 @@ private DirectMonotonicReader(
@Overr
uschindler commented on pull request #518:
URL: https://github.com/apache/lucene/pull/518#issuecomment-999391791
I corrected the test to assert the `jdk.unsupported.foreign` module. But as
this won't compile without it, it is in fact obsolete. But for easier merging
in future I left it. I
dweiss commented on a change in pull request #550:
URL: https://github.com/apache/lucene/pull/550#discussion_r773758512
##
File path:
lucene/distribution.tests/src/test/org/apache/lucene/distribution/TestScripts.java
##
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the Apache Software
uschindler opened a new pull request #561:
URL: https://github.com/apache/lucene/pull/561
Followup to cleanup TestSecrets after
https://issues.apache.org/jira/browse/LUCENE-10301
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
uschindler commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999455025
This is done in similar way by the JDK SharedSecrets. The reason is mainly
to not produce a stack inspection on every access. It is also more readable
because code is smaller.
mocobeta commented on a change in pull request #550:
URL: https://github.com/apache/lucene/pull/550#discussion_r773771220
##
File path:
lucene/distribution.tests/src/test/org/apache/lucene/distribution/TestScripts.java
##
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the Apache Softwa
dweiss commented on a change in pull request #550:
URL: https://github.com/apache/lucene/pull/550#discussion_r773776646
##
File path:
lucene/distribution.tests/src/test/org/apache/lucene/distribution/TestScripts.java
##
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the Apache Software
dweiss commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999463820
To me the explicit version is more readable because you see the
"testsecrets" class and not the private constant but I really don't care or
mind either way - looks fine to me.
--
uschindler commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999466216
This is why I asked to rename the constants. I think `INDEX_PACKAGE_ACCESS`
=> `INDEX_PACKAGE_SECRETS` or `indexPackageSecrets` (although this does not
conform to coding standar
[
https://issues.apache.org/jira/browse/LUCENE-10301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463706#comment-17463706
]
Uwe Schindler commented on LUCENE-10301:
Hi, I created a PR with some followup
uschindler commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999469874
I think both names are fine. No matter if its "secret" or "access". Both
mean the same in this context. I will merge this later and backport (should
work).
--
This is an auto
dweiss commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999470748
I modelled this after the JDK, actually. So Test"Secrets" but inside they
have "access" objects. Makes sense to me.
--
This is an automated message from the Apache Git Service.
To
uschindler commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999471360
Here is an example in the JDK:
https://github.com/openjdk/jdk/blob/739769c8fc4b496f08a92225a12d07414537b6c0/src/java.base/windows/classes/sun/nio/ch/SocketDispatcher.java#L40-L41
uschindler edited a comment on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999471360
Here is an example in the JDK:
https://github.com/openjdk/jdk/blob/739769c8fc4b496f08a92225a12d07414537b6c0/src/java.base/windows/classes/sun/nio/ch/SocketDispatcher.java#
uschindler commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999479096
Hi, have you seen this method in Java 15, this should be used, maybe we
should add a TODO:
[Lookup#ensureInitialized(Clazz)](https://docs.oracle.com/en/java/javase/17/docs/api/j
rmuir commented on pull request #557:
URL: https://github.com/apache/lucene/pull/557#issuecomment-999482656
> I like this change, it's a net win that is not only going to speed up
binary doc values but also sorted set and sorted numeric doc values, which also
use a direct monotonic reader
dweiss commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999485297
No, I didn't see that. But it's an equivalent of what class for name
currently does (and it doesn't require a lookup). Up to you, this would only
apply on main though if it requires
rmuir merged pull request #553:
URL: https://github.com/apache/lucene/pull/553
--
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-unsubscr...@
[
https://issues.apache.org/jira/browse/LUCENE-10329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463741#comment-17463741
]
ASF subversion and git services commented on LUCENE-10329:
--
Co
rmuir commented on pull request #553:
URL: https://github.com/apache/lucene/pull/553#issuecomment-999486098
Thank you @gf2121 !
--
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 comme
dweiss commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999486042
I'd stick to normal java conventions with respect to naming (so upper case,
final constant).
--
This is an automated message from the Apache Git Service.
To respond to the message
uschindler commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999486380
I just left a TODO. The code you did is fine, I just like to add TODOs for
all stuff where we may need to work after a JDK upgrade. Maybe we should mark
those with a separate "t
uschindler commented on pull request #561:
URL: https://github.com/apache/lucene/pull/561#issuecomment-999487238
Main is still JDK 11, but may change soon.
--
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
uschindler merged pull request #561:
URL: https://github.com/apache/lucene/pull/561
--
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-unsubsc
[
https://issues.apache.org/jira/browse/LUCENE-10301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463749#comment-17463749
]
ASF subversion and git services commented on LUCENE-10301:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463750#comment-17463750
]
ASF subversion and git services commented on LUCENE-10301:
--
Co
mocobeta commented on a change in pull request #550:
URL: https://github.com/apache/lucene/pull/550#discussion_r773810546
##
File path:
lucene/distribution.tests/src/test/org/apache/lucene/distribution/TestScripts.java
##
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the Apache Softwa
Feng Guo created LUCENE-10334:
-
Summary: Introduce a BlockReader based on ForUtil and use it for
NumericDocValues
Key: LUCENE-10334
URL: https://issues.apache.org/jira/browse/LUCENE-10334
Project: Lucene
[
https://issues.apache.org/jira/browse/LUCENE-10329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463755#comment-17463755
]
ASF subversion and git services commented on LUCENE-10329:
--
Co
gf2121 opened a new pull request #562:
URL: https://github.com/apache/lucene/pull/562
see https://issues.apache.org/jira/browse/LUCENE-10334
--
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 s
[
https://issues.apache.org/jira/browse/LUCENE-10329?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robert Muir resolved LUCENE-10329.
--
Fix Version/s: 9.1
Resolution: Fixed
Thanks [~gf2121]!
> Use Computed Mask For Direct
gf2121 commented on pull request #557:
URL: https://github.com/apache/lucene/pull/557#issuecomment-999516790
Hi @rmuir @jpountz , Thanks a lot for all talking about this! I think i
**probably** find out a better way there:
> Actually, what I thought at first was to only change the st
gf2121 edited a comment on pull request #557:
URL: https://github.com/apache/lucene/pull/557#issuecomment-999516790
Hi @rmuir @jpountz , Thanks a lot for all talking about this! I think i
**probably** find out a better way there:
> Actually, what I thought at first was to only change
[
https://issues.apache.org/jira/browse/LUCENE-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463764#comment-17463764
]
Robert Muir commented on LUCENE-10334:
--
Wow! very promising results :)
> Introduc
gf2121 edited a comment on pull request #557:
URL: https://github.com/apache/lucene/pull/557#issuecomment-999516790
Hi @rmuir @jpountz , Thanks a lot for all talking about this! I think we
**probably** find out a better way there:
> Actually, what I thought at first was to only chang
mocobeta commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999607132
The test failure cannot be reproduced for me.
https://github.com/apache/lucene/runs/4606721258?check_suite_focus=true
```
gradlew :lucene:distribution.tests:test --tests
"
mocobeta edited a comment on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999607132
The test failure cannot be reproduced for me.
https://github.com/apache/lucene/runs/4606721258?check_suite_focus=true
```
gradlew :lucene:distribution.tests:test --t
dweiss commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999609699
I'm not sure what it is but I'm ears deep into rewriting these path-related
things - there are bugs and quirks there that I'm trying to overcome. Once I
finalize that, I'll return t
mocobeta commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999624305
Thanks for your quick response as always. Sorry, I just tend to add
uncoordinated memos in pull requests but they are not questions that need
responses.
--
This is an automated
mocobeta edited a comment on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999607132
The test failure cannot be reproduced for me.
https://github.com/apache/lucene/runs/4606721258?check_suite_focus=true
```
gradlew :lucene:distribution.tests:test --t
[
https://issues.apache.org/jira/browse/LUCENE-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463868#comment-17463868
]
Greg Miller commented on LUCENE-10334:
--
Cool! I haven't looked at the PR yet but t
mocobeta commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999637256
It seems that this can be reproduced on the Github Actions container. As a
test, let me revert the recently merged change...
--
This is an automated message from the Apache Git
gf2121 edited a comment on pull request #557:
URL: https://github.com/apache/lucene/pull/557#issuecomment-999516790
Hi @rmuir @jpountz , Thanks a lot for all talking about this! I think we
**probably** find out a better way there:
> Actually, what I thought at first was to only chang
mocobeta commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999650568
It looks like the failure is not related to
https://github.com/dweiss/lucene/pull/15. I will revert the revert commit;
sorry for the noise again.
--
This is an automated messag
gf2121 edited a comment on pull request #557:
URL: https://github.com/apache/lucene/pull/557#issuecomment-999516790
Hi @rmuir @jpountz , Thanks a lot for all talking about this! I think we
**probably** find out a better way there:
> Actually, what I thought at first was to only chang
cpoerschke commented on pull request #1994:
URL: https://github.com/apache/lucene-solr/pull/1994#issuecomment-999663008
Closing as stale and probably now less useful with Gradle familiarity having
increased over time and Ant habits having faded over time.
--
This is an automated message
cpoerschke closed pull request #1994:
URL: https://github.com/apache/lucene-solr/pull/1994
--
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-
mocobeta opened a new pull request #563:
URL: https://github.com/apache/lucene/pull/563
This is a clone PR of https://github.com/apache/lucene/pull/550 to explore
the test failure on Github Actions.
--
This is an automated message from the Apache Git Service.
To respond to the messag
[
https://issues.apache.org/jira/browse/LUCENE-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463911#comment-17463911
]
Feng Guo commented on LUCENE-10334:
---
Thanks [~gsmiller] ! Yes I do thought i would ge
[
https://issues.apache.org/jira/browse/LUCENE-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463911#comment-17463911
]
Feng Guo edited comment on LUCENE-10334 at 12/22/21, 4:34 PM:
---
mocobeta closed pull request #563:
URL: https://github.com/apache/lucene/pull/563
--
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-unsubscr.
mocobeta commented on pull request #563:
URL: https://github.com/apache/lucene/pull/563#issuecomment-999744354
It would be better to wait for LUCENE-10301
--
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
mocobeta commented on pull request #564:
URL: https://github.com/apache/lucene/pull/564#issuecomment-999767130
Actually, `lucene-test-framework-10.0.0-SNAPSHOT.jar` is on the module path.
```
1> --- [forked subprocess output: /usr/bin/dash] ---
1> files in
/home/runner/work/luc
[
https://issues.apache.org/jira/browse/LUCENE-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463911#comment-17463911
]
Feng Guo edited comment on LUCENE-10334 at 12/22/21, 6:05 PM:
---
[
https://issues.apache.org/jira/browse/LUCENE-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463911#comment-17463911
]
Feng Guo edited comment on LUCENE-10334 at 12/22/21, 6:10 PM:
---
[
https://issues.apache.org/jira/browse/LUCENE-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463911#comment-17463911
]
Feng Guo edited comment on LUCENE-10334 at 12/22/21, 6:24 PM:
---
[
https://issues.apache.org/jira/browse/LUCENE-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463911#comment-17463911
]
Feng Guo edited comment on LUCENE-10334 at 12/22/21, 6:25 PM:
---
dweiss commented on pull request #564:
URL: https://github.com/apache/lucene/pull/564#issuecomment-999781303
This may be a problem with how paths are placed for tests - it's not correct
on main. I'm working on it, but it's not trivial at all. I think this will have
to wait until LUCENE-103
[
https://issues.apache.org/jira/browse/LUCENE-10328?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464002#comment-17464002
]
Dawid Weiss commented on LUCENE-10328:
--
Right. Once you actually fix the module pa
mocobeta commented on pull request #564:
URL: https://github.com/apache/lucene/pull/564#issuecomment-999793043
It seems there is a difference in the binary distribution structure.
On my local machine:
```
lucene/distribution/build/packages/lucene-10.0.0-SNAPSHOT-itests/
├── R
mocobeta commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999814200
I thought [something strange was happening on the
Actions](https://github.com/apache/lucene/pull/564#issuecomment-999793043), but
after merging the main branch (I did that to fetc
dweiss commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999817741
This means module resolution picks up all the modules, not just those
requested and fails there because it can't find dependencies of the test
framework... If this indeed is the cas
mocobeta commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999818035
Besides the test failure, Luke fails to start on the latest main as I
speculated.
```
lucene $ tar xzf lucene/distribution/build/packages/lucene-10.0.0-SNAPSHOT
lucene-10.
dweiss commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999819547
Well, this confirms the test is working... ;) I didn't realize the
resolution is eager for all modules on module path - I thought it'd only
include the actual graph of included modu
dweiss commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999823044
I think the resolution may be actually lazy but the test framework provides
services consumed by lucene-core - that's why the resolver tries to load the
test framework module (and f
dweiss commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999823838
If you'd like to provide a minor patch reverting just this
distribution-assembly bit, @mocobeta , it'd be great (the tests in
distribution.tests would have to be updated too).
--
mocobeta commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999825830
Thanks.
The problem is now clear to me. I will try to look at this tomorrow (or
today; it's 5 am in my timezone).
--
This is an automated message from the Apache Git Service
mocobeta closed pull request #564:
URL: https://github.com/apache/lucene/pull/564
--
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-unsubscr.
dweiss commented on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999833490
bq. 5 am in my timezone
Nobody should not be awake at this ungodly hour.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
dweiss edited a comment on pull request #550:
URL: https://github.com/apache/lucene/pull/550#issuecomment-999833490
bq. 5 am in my timezone
Nobody should be awake at this ungodly hour.
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
Dawid Weiss created LUCENE-10335:
Summary: IOUtils.getDecodingReader(Class, String) is broken
with modules
Key: LUCENE-10335
URL: https://issues.apache.org/jira/browse/LUCENE-10335
Project: Lucene - C
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464033#comment-17464033
]
Uwe Schindler commented on LUCENE-10335:
This should work inside the same modul
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464033#comment-17464033
]
Uwe Schindler edited comment on LUCENE-10335 at 12/22/21, 8:23 PM:
--
[
https://issues.apache.org/jira/browse/LUCENE-10328?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464034#comment-17464034
]
Dawid Weiss commented on LUCENE-10328:
--
There are more problems that surfaced when
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464035#comment-17464035
]
Dawid Weiss commented on LUCENE-10335:
--
take a look at JavascriptCompiler - it cal
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464036#comment-17464036
]
Dawid Weiss commented on LUCENE-10335:
--
btw - I know it's broken because I have it
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464040#comment-17464040
]
Uwe Schindler commented on LUCENE-10335:
I wonder why it is broken. Because you
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464045#comment-17464045
]
Uwe Schindler commented on LUCENE-10335:
It is not caller sensitive. I have the
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464048#comment-17464048
]
Dawid Weiss commented on LUCENE-10335:
--
And it's exactly how it works. So unless y
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464046#comment-17464046
]
Dawid Weiss commented on LUCENE-10335:
--
Resources in named modules are subject to
[ https://issues.apache.org/jira/browse/LUCENE-10335 ]
Uwe Schindler deleted comment on LUCENE-10335:
was (Author: thetaphi):
It is not caller sensitive. I have the feeling your packaging is wrong. If the
resource is in same package and is
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464049#comment-17464049
]
Dawid Weiss commented on LUCENE-10335:
--
If you don't believe me, write a snippet o
[
https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464050#comment-17464050
]
Uwe Schindler commented on LUCENE-10335:
Yes I figured, it is caller sensitive:
1 - 100 of 125 matches
Mail list logo
