Adam Retter comments

Results 574 comments of


                                            Adam Retter

ConcurrentMap::compute default implementation bug results in NPE when invoked concurrently on NBHM

I just wanted to thank everyone that participated in helping me here - thank you :-)

Restructure pomfiles

@dizzzz you don't need to close it! I think some of the changes are great. The one I think most useful is ordering the dependencies alphabetically

[BUG] Admin Password not set during programmatic installation (JAR installation with -console)

@dariok The Admin Password will not be set if the data dir already exists... could that be the issue you are seeing?

[feature] Relax eXist functions that require parameter of type xs:anyURI to accept xs:string

I think it might be better to discuss adding an addditional Type Promotion rule to XQuery 4 instead to address this

[BUG] Relative URL resolution not working as expected in fn:transform() under eXist-db 6.2

@yamahito @djbpitt Thanks, I have now codified those as Unit Tests here - https://github.com/evolvedbinary/exist/pull/new/hotfix/xslt-relative-imports

Remove the last small dependency on Apache Xalan

> In the unit tests we explicitly test for xalan 😬 @dizzzz Okay no problem. I have updated the PR to address that as well.

Remove the last small dependency on Apache Xalan

There is a runtime dependency on Xalan in the GeoTools libraries used by the Spatial Index. I have opened a PR to fix that here - [GEOT-7284](https://github.com/geotools/geotools/pull/4141). Once GEOT-7284 is...

Remove the last small dependency on Apache Xalan

> it looks the PR combines 2 PRs ? @dizzzz now fixed.

Remove the last small dependency on Apache Xalan

> @adamretter seems there is no support for "namespace" feature: https://github.com/eXist-db/exist/actions/runs/3649983650/jobs/6165379878#step:4:13854 @reinhapa Yup, that is addressed in my PR https://github.com/geotools/geotools/pull/4141

Remove the last small dependency on Apache Xalan

@dizzzz @reinhapa Okay so they released GeoTools 28.1 and I just updated this, so I am hopeful that assuming CI goes through okay, then it is now ready.