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

Bugfix patch #544

Merged
merged 19 commits into from
Apr 29, 2024
Merged

Bugfix patch #544

merged 19 commits into from
Apr 29, 2024

Conversation

novoj
Copy link
Collaborator

@novoj novoj commented Apr 29, 2024

No description provided.

novoj and others added 19 commits April 22, 2024 10:28
…d-and-gql-query-doesnt-contain-query-name

fix(#532): NPE when GQL query is sent without a name and tracing is enabled
…an better work with the multiline text inside"

This reverts commit 9eb443e.
…n-json-layout

 fix(#534): log exception with log message so that consumers can better work with the multiline text inside
…es into account

Equal date times:

- `2024-04-24T11:07:11.677467736Z`
- `2024-04-24T13:07:11.677467736+02:00`

were considered different. Also, the filter index didn't take string collations into account, so less than & greater than didn't work properly for strings containing national characters.

The fix changed the format of the stored data, so I had to include a backward compatible deserializer so that existing data is loaded correctly. When the data is saved again, it has the correct new format. The old format should be automatically removed by compaction.
…version-on-the-way-from-the-client-to-the-server

fix (#536): Invalid OffsetDateTime comparison taking offset difference into an account
When UserFilterFormula was not part of the computational tree, the facet impact computation didn't work well.
…-difference-when-userfilter-is-used

fix(#537): Incorrect facet statistics difference when userFilter is used
There's a transactional boolean `dirty` in many places in transactional data structures, but it's not taken into account in the `io.evitadb.core.transaction.memory.TransactionalLayerProducer#createCopyWithMergedTransactionalMemory` method, which is called when a new state is created. If we had taken advantage of this flag and just returned "this" reference in cases where it's not dirty, we could save a lot of memory allocations. If nothing has changed, we can reuse the exact same instance of the object for the next generation.
…changes-in-indexes-occur

refactor(#542): Limit allocations when no changes in indexes occur
@novoj novoj merged commit f53594f into master Apr 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants