Jdbi: 버전 3.5.1 이후 상당한 성능 저하

안녕하세요 @Alexey1Gavrilov , 신고해 주셔서 감사합니다! 확실히 약간의 회귀가 있었다는 데 동의합니다.

상황 개선 작업을 시작했습니다. https://github.com/jdbi/jdbi/pull/1607
현재, 우리가 모든 것을 깨뜨리지 않도록 검토와 몇 가지 테스트를 기다리고 있습니다. :)

위에 링크된 수정 사항이 사용 사례에 도움이 되는지 독립적으로 확인할 시간이 있습니까?
감사합니다. 곧 수정 사항이 병합되기를 바랍니다.

이미 개선 작업을 하고 있다는 아주 좋은 소식입니다! #1607이 성능을 크게 향상시키는 것을 확인할 수 있습니다. 내 마이크로 벤치마크 프로그램은 해당 PR로 실행한 다음 3.5.1에서 약 10-15% 더 빠르게 실행됩니다. 훌륭한 일!

또한 성능을 더욱 향상시키는 열 이름 일치자에 대한 사용자 지정 캐시 계층을 구현했습니다. 아이디어는 기본적으로 Jdbi 인스턴스가 처리하는 모든 필드에 대해 columnName 및 javaName 의 비교를 캐시하는 것이었습니다. 다음은 코드 스니펫입니다.

class CachingColumnNameMatcher implements ColumnNameMatcher {
  private static class Column {
    final String columnName;
    final String javaName;

    private Column(String columnName, String javaName) {
      this.columnName = columnName;
      this.javaName = javaName;
    }

    <strong i="10">@Override</strong>
    public boolean equals(Object o) {
      if (this == o) {
        return true;
      }
      if (o == null || getClass() != o.getClass()) {
        return false;
      }
      Column column = (Column) o;
      return Objects.equals(columnName, column.columnName) &&
          Objects.equals(javaName, column.javaName);
    }

    <strong i="11">@Override</strong>
    public int hashCode() {
      return Objects.hash(columnName, javaName);
    }
  }

  private final ColumnNameMatcher delegate;
  private final ConcurrentMap<Column, Boolean> cache;

  CachingColumnNameMatcher(final ColumnNameMatcher delegate) {
    this.delegate = delegate;
    this.cache = new ConcurrentHashMap<>();
  }

  <strong i="12">@Override</strong>
  public boolean columnNameMatches(final String columnName, final String javaName) {
    final Column key = new Column(columnName, javaName);

    return cache.computeIfAbsent(key, (c) -> delegate.columnNameMatches(columnName, javaName));
  }
}

...
    final ReflectionMappers reflectionMappers = jdbi.getConfig().get(ReflectionMappers.class);
    reflectionMappers.setColumnNameMatchers(
        reflectionMappers.getColumnNameMatchers().stream()
            .map(CachingColumnNameMatcher::new)
            .collect(Collectors.toList()));
...

캐싱 열 이름 비교가 의미가 있는지(일부 가비지 수집 비용이 포함됨) PR을 열 수 있는지 알려주십시오.

@Alexey1Gavrilov , 방금 #1607이 포함된 3.11.0을 출시했습니다.

모든 일치를 캐싱하는 것이 아마도 무거운 솔루션일 수 있지만 열 이름 일치의 성능을 개선하게 되어 기쁩니다. 특정 매처(SnakeCase 매처)에 문제가 있습니까? 당신에게 영향을 미치는 문제에 대한 더 나은 설명부터 시작하여 캐싱일 수 있는 솔루션을 고안해 보겠습니다. :)

테스트 3.11.1 - 3.5.1에 ​​비해 20% 개선! 훌륭한 일!
SnakeCase 매처에 대해 별도의 문제를 만들고 거기에서 가능한 솔루션에 대해 논의하겠습니다.

좋아 고마워!