Skip to content

CLIENT-4143 Restructure to support the new Java client#204

Draft
agrgr wants to merge 8 commits intomainfrom
CLIENT-4143-to-multi-module
Draft

CLIENT-4143 Restructure to support the new Java client#204
agrgr wants to merge 8 commits intomainfrom
CLIENT-4143-to-multi-module

Conversation

@agrgr
Copy link

@agrgr agrgr commented Mar 10, 2026

No description provided.

agrgr and others added 5 commits February 20, 2026 18:38
…ceMapper, ObjectEmbedMapper to use IObjectMapper/IRecordConverter

- Replace IBaseAeroMapper with IObjectMapper in ClassCache, ClassCacheEntry,
  MappingConverter, ObjectReferenceMapper, ObjectEmbedMapper
- Remove policy storage (defaultPolicies, childrenPolicies, specificPolicies,
  determinePolicy, setDefaultPolicies, setReactiveDefaultPolicies) from ClassCache;
  moved to new PolicyCache singleton in legacy
- Remove policy constructor params and getter methods (getReadPolicy, getWritePolicy,
  getBatchPolicy, getQueryPolicy, getScanPolicy) from ClassCacheEntry
- Remove getBins, constructAndHydrate(Key,Record), hydrateFromRecord from ClassCacheEntry;
  add getKeyFieldName(), isKeyFieldStoredAsBin(), setGenerationValue() helpers
- Replace IAerospikeClient with RecordLoader in MappingConverter constructor;
  new AerospikeRecordLoader implements RecordLoader via IAerospikeClient
- Remove Key/Record convertToObject overloads from MappingConverter; keep
  map/list-based overloads and add resolveDependencies=false variant
- ObjectReferenceMapper uses mapper.getRecordLoader() for record fetches
- Add IRecordConverter core interface; IObjectMapper returns IRecordConverter
  (MappingConverter implements IRecordConverter)
- IBaseAeroMapper extends IObjectMapper with covariant MappingConverter return
- AeroMapper and ReactiveAeroMapper implement getRecordLoader(); use PolicyCache
  for policy lookups instead of ClassCacheEntry policy getters
- Also update TypeUtils, ListMapper, MapMapper, PropertyDefinition, MapperUtils
  and virtuallist classes to use IObjectMapper; fix BaseVirtualList write policy

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@agrgr agrgr requested a review from tim-aero March 10, 2026 16:03
@agrgr agrgr added the enhancement New feature or request label Mar 10, 2026
@github-advanced-security
Copy link

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@agrgr
Copy link
Author

agrgr commented Mar 10, 2026

The build is stable and only missing fluent client dependency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant