influxdata / influxdata/influxdb-java

Support using package-private classes as POJOs for InfluxDBResultMapper

Abierto
#457 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

difficulty/low kind/enhancement
Lenguaje dominante
Java
Estrellas
1.2k
Forks
469
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

I'm using org.influxdb:influxdb-java:2.10.
I'd like to limit my @Measurement classes visibility to package-private, but currently it doesn't work because of the following exception:

org.influxdb.InfluxDBMapperException: java.lang.IllegalAccessException: Class org.influxdb.impl.InfluxDBResultMapper can not access a member of class com.example.MyMeasurement with modifiers "public"

The line which causes this is https://github.com/influxdata/influxdb-java/blob/influxdb-java-2.10/src/main/java/org/influxdb/impl/InfluxDBResultMapper.java#L160.
Looks like it will be enough to replace

object = clazz.newInstance();

with

Constructor<T> defaultConstructor = clazz.getDeclaredConstructor();
defaultConstructor.setAccessible(true);
object = declaredConstructor.newInstance();

If there are no objections, I'll provide a PR.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza en src/main/java/org/influxdb/impl/InfluxDBResultMapper.java en la línea 160 y sigue cómo se instancian los measurement objects. Verifica el comportamiento con una clase @Measurement package-private y considera una prueba de regresión que muestre que el mapeo se realiza correctamente para este nivel de visibilidad.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
databases
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
38/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.