You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What do you think about making this replacement? ReflectionUtils has a cache that allows you to reduce call to the Java Security Manager (Deprecated in Java 17).
As I understand it, this may allow better optimization for the Spring Native (and other Cloud Nantive).
And java.lang.reflect calls is low-level API.
Current classes having calls to java.lang.reflect:
ad 1: This should be improved: here a ElasticsearchPersistentEntity should be used to get the properties which are annotated with ScriptedField because in Spring Data Elasticsearch this is where all the reflective access to the entity should be implemented (this class is using the Spring methods for reflection). For writing to the field the ElasticsearchPersisntentProperty for that field should be used.
ad 2: this is an old (ugly) implementation to index an object and is only used in test scope. This could be refactored to something that does not use the reflection. But this is not relevant to native code, as it's only in the test
Case 2: I don't know how you can do something with an entity without using reflaction. And you also said that in makes no sense because it is in the test scope.
What do you think about making this replacement?
ReflectionUtils has a cache that allows you to reduce call to the Java Security Manager (Deprecated in Java 17).
As I understand it, this may allow better optimization for the Spring Native (and other Cloud Nantive).
And java.lang.reflect calls is low-level API.
Current classes having calls to java.lang.reflect:
The text was updated successfully, but these errors were encountered: