Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Question, perhaps bug in 6.3.0 #5565

Open
eduarddrenth opened this issue Nov 20, 2024 · 5 comments
Open

Question, perhaps bug in 6.3.0 #5565

eduarddrenth opened this issue Nov 20, 2024 · 5 comments
Labels
bug issue confirmed as bug regression

Comments

@eduarddrenth
Copy link
Contributor

Running my app, that is working fine in 6.2.0, in 6.3.0 I run into the error below.

Does this ring a bell with anyone?

Caused by: java.lang.NullPointerException
	at org.exist.indexing.lucene.LuceneMatch.getField(LuceneMatch.java:126)
	at org.exist.xquery.modules.lucene.Field.eval(Field.java:123)
	at org.exist.xquery.BasicFunction.eval(BasicFunction.java:73)
	at org.exist.xquery.InternalFunctionCall.eval(InternalFunctionCall.java:62)
	at org.exist.xquery.UserDefinedFunction.eval(UserDefinedFunction.java:161)
	at org.exist.xquery.DynamicCardinalityCheck.eval(DynamicCardinalityCheck.java:73)
	at org.exist.xquery.Atomize.eval(Atomize.java:66)
	at org.exist.xquery.DynamicTypeCheck.eval(DynamicTypeCheck.java:60)
	at org.exist.xquery.FunctionCall.evalFunction(FunctionCall.java:289)
	at org.exist.xquery.FunctionCall.eval(FunctionCall.java:207)
	at org.exist.xquery.PathExpr.eval(PathExpr.java:280)
	at org.exist.xquery.EnclosedExpr.eval(EnclosedExpr.java:80)
	at org.exist.xquery.PathExpr.eval(PathExpr.java:280)
	at org.exist.xquery.ElementConstructor.eval(ElementConstructor.java:330)
	at org.exist.xquery.PathExpr.eval(PathExpr.java:280)
	at org.exist.xquery.ElementConstructor.eval(ElementConstructor.java:330)
	at org.exist.xquery.PathExpr.eval(PathExpr.java:280)
	at org.exist.xquery.ElementConstructor.eval(ElementConstructor.java:330)
	at org.exist.xquery.DebuggableExpression.eval(DebuggableExpression.java:58)
	at org.exist.xquery.LetExpr.eval(LetExpr.java:110)
	at org.exist.xquery.LetExpr.eval(LetExpr.java:110)
	at org.exist.xquery.UserDefinedFunction.eval(UserDefinedFunction.java:161)
	at org.exist.xquery.DynamicCardinalityCheck.eval(DynamicCardinalityCheck.java:73)
	at org.exist.xquery.DynamicTypeCheck.eval(DynamicTypeCheck.java:60)
	at org.exist.xquery.FunctionCall.evalFunction(FunctionCall.java:289)
	at org.exist.xquery.FunctionCall.eval(FunctionCall.java:207)
	at org.exist.xquery.DebuggableExpression.eval(DebuggableExpression.java:58)
	at org.exist.xquery.OrderByClause.eval(OrderByClause.java:81)
	at org.exist.xquery.ForExpr.processItem(ForExpr.java:248)
	at org.exist.xquery.ForExpr.eval(ForExpr.java:187)
	at org.exist.xquery.LetExpr.eval(LetExpr.java:110)
	at org.exist.xquery.PathExpr.eval(PathExpr.java:280)
	at org.exist.xquery.EnclosedExpr.eval(EnclosedExpr.java:80)
	at org.exist.xquery.PathExpr.eval(PathExpr.java:280)
	at org.exist.xquery.ElementConstructor.eval(ElementConstructor.java:330)
	at org.exist.xquery.UserDefinedFunction.eval(UserDefinedFunction.java:161)
	at org.exist.xquery.DynamicCardinalityCheck.eval(DynamicCardinalityCheck.java:73)
@tuurma
Copy link
Contributor

tuurma commented Nov 20, 2024 via email

@adamretter
Copy link
Contributor

adamretter commented Nov 20, 2024

@tuurma Just to make you aware (in-case you missed it) that PR #4541 adds 3 additional new features:

As a consequence, users no longer need to specify the fields to retrieve at query time

an additional configuration attribute is introduced to field definitions in collection.xconf: adding binary="yes"

Adds some missing atomic types to typed lucene fields (xs:decimal, xs:float, xs:short and more)

So I think according to the SemVer 2.0.0 standard that eXist-db follows, that PR would have to go into a Minor release and not a Hotfix release, which would mean an eXist-db 6.4.0 and not a 6.3.1.

@eduarddrenth For background information, it was decided not to include that in the 6.3.0 release due to an as yet unresolved memory leak that has been reported from #4541 in this issue - #4890

@eduarddrenth
Copy link
Contributor Author

Thanks for these details, I'll patiently await the minor

@line-o line-o added bug issue confirmed as bug regression labels Nov 20, 2024
@line-o
Copy link
Member

line-o commented Nov 20, 2024

The fact that the code was running in 6.2.0 and leads to a NPE in 6.3.0 qualifies this issue as a bug and a regression.

@adamretter
Copy link
Contributor

qualifies this issue as a bug and a regression

The issue - yes, but the previous solution - no.

@line-o line-o moved this to In review in v6.3.1 release Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug issue confirmed as bug regression
Projects
Status: In review
Development

No branches or pull requests

4 participants