findDistinctRowByIdOrderByRow()で取得されるrowが同じ値になる

Question

前提・実現したいこと

Spring Bootでselect distinct row from stat where id = ? order by row
同等機能を実装し取得したrowがなぜか同じ値になりました。
どう対処すればよいのか、ご教授お願い致します。

発生している問題・ログメッセージ

2021-12-19 08:50:39.455 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.SQL                        :
    select
        distinct statrow0_.id as id2_0_,
        statrow0_.row as row3_0_
    from
        stat statrow0_
    where
        statrow0_.id=?
    order by
        statrow0_.row asc
Hibernate:
    select
        distinct statrow0_.id as id2_0_,
        statrow0_.row as row3_0_
    from
        stat statrow0_
    where
        statrow0_.id=?
    order by
        statrow0_.row asc
2021-12-19 08:50:39.459 TRACE 32664 --- [nio-8081-exec-1] o.h.type.descriptor.sql.BasicBinder      : binding parameter [1] as [INTEGER] - [1]
2021-12-19 08:50:40.434 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result set row: 0
2021-12-19 08:50:40.447 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result row: EntityKey[com.sct.entity.StatRow#component[id]{id=1}]
2021-12-19 08:50:40.451 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result set row: 1
2021-12-19 08:50:40.451 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result row: EntityKey[com.sct.entity.StatRow#component[id]{id=1}]
2021-12-19 08:50:40.452 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result set row: 2

中略

2021-12-19 08:50:40.456 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result set row: 56
2021-12-19 08:50:40.456 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result row: EntityKey[com.sct.entity.StatRow#component[id]{id=1}]
2021-12-19 08:50:40.456 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result set row: 57
2021-12-19 08:50:40.456 DEBUG 32664 --- [nio-8081-exec-1] org.hibernate.loader.Loader              : Result row: EntityKey[com.sct.entity.StatRow#component[id]{id=1}]
2021-12-19 08:50:40.460 DEBUG 32664 --- [nio-8081-exec-1] o.h.engine.internal.TwoPhaseLoad         : Resolving attributes for [com.sct.entity.StatRow#component[id]{id=1}]
2021-12-19 08:50:40.460 DEBUG 32664 --- [nio-8081-exec-1] o.h.engine.internal.TwoPhaseLoad         : Processing attribute `row` : value = 1964
2021-12-19 08:50:40.460 DEBUG 32664 --- [nio-8081-exec-1] o.h.engine.internal.TwoPhaseLoad         : Attribute (`row`)  - enhanced for lazy-loading? - false
2021-12-19 08:50:40.461 DEBUG 32664 --- [nio-8081-exec-1] o.h.engine.internal.TwoPhaseLoad         : Done materializing entity [com.sct.entity.StatRow#component[id]{id=1}]
2021-12-19 08:50:40.464  INFO 32664 --- [nio-8081-exec-1] com.sct.service.StatRowService           : row=1964
2021-12-19 08:50:40.464  INFO 32664 --- [nio-8081-exec-1] com.sct.service.StatRowService           : row=1964
2021-12-19 08:50:40.464  INFO 32664 --- [nio-8081-exec-1] com.sct.service.StatRowService           : row=1964
2021-12-19 08:50:40.464  INFO 32664 --- [nio-8081-exec-1] com.sct.service.StatRowService           : row=1964

中略

2021-12-19 08:50:40.464  INFO 32664 --- [nio-8081-exec-1] com.sct.service.StatRowService           : row=1964
2021-12-19 08:50:40.464  INFO 32664 --- [nio-8081-exec-1] com.sct.service.StatRowService           : row=1964
2021-12-19 08:50:40.470 DEBUG 32664 --- [nio-8081-exec-1] o.s.w.s.v.ContentNegotiatingViewResolver : Selected 'text/html' given [text/html, application/xhtml+xml, image/avif, image/webp, application/xml;q=0.9, */*;q=0.8]
2021-12-19 08:50:41.470 DEBUG 32664 --- [nio-8081-exec-1] o.s.web.servlet.DispatcherServlet        : Completed 200 OK

該当のソースコード

create table stat(
    id integer
    , row integer
    , col integer
    , data double precision not null
    , constraint stat_pk1 primary key (id, row, col)
)

package com.sct.entity;

import java.io.Serializable;

import javax.persistence.Column;
import javax.persistence.Embeddable;

@Embeddable
public class StatId implements Serializable {

        private static final long serialVersionUID = 1289075301160465252L;

        @Column(name = "id")
        private Integer id;

        public Integer getId() {
                return id;
        }

        public void setId(Integer id) {
                this.id = id;
        }
}

package com.sct.entity;

import java.io.Serializable;

import javax.persistence.Column;
import javax.persistence.EmbeddedId;
import javax.persistence.Entity;
import javax.persistence.Table;

@Entity
@Table(name="stat")
public class StatRow implements Serializable {

        private static final long serialVersionUID = 1289075301160465252L;

        @EmbeddedId
        private StatId id;

        @Column(name = "row")
        private Integer row;

        public StatId getId() {
                return id;
        }

        public void setId(StatId id) {
                this.id = id;
        }

        public Integer getRow() {
                return row;
        }

        public void setRow(Integer row) {
                this.row = row;
        }
}

package com.sct.repository;

import java.util.List;

import org.springframework.data.jpa.repository.JpaRepository;
import org.springframework.stereotype.Repository;

import com.sct.entity.StatId;
import com.sct.entity.StatRow;

@Repository
public interface StatRowRepository extends JpaRepository<StatRow, StatId> {
        List<StatRow> findDistinctRowByIdOrderByRow(StatId Id);
}

package com.sct.service;

import java.util.ArrayList;
import java.util.List;

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;

import com.sct.entity.StatId;
import com.sct.entity.StatRow;
import com.sct.repository.StatRowRepository;

@Service
public class StatRowService {
        static protected final Logger log = LoggerFactory.getLogger(StatRowService.class);

        @Autowired
        private StatRowRepository rowRepo;

        public List<Integer> findAllRows(Integer id) {
                List<Integer> rows = new ArrayList<>();

                StatId statId = new StatId();
                statId.setId(id);

                for (StatRow statRow : rowRepo.findDistinctRowByIdOrderByRow(statId)) {
                        Integer row = statRow.getRow();
                        log.info("row={}", row);
                        rows.add(row);
                }

                return rows;
        }
}

package com.sct.controller;

import java.io.File;
import java.io.IOException;
import java.util.List;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Controller;
import org.springframework.ui.Model;
import org.springframework.web.bind.annotation.GetMapping;

import com.sct.service.StatRowService;

@Controller
public class StatController {
        @Autowired
        private StatRowService rowService;

        @GetMapping(value = "/stat")
        public String displayList(Model model) {
                List<Integer> rowList = rowService.findAllRows(1);
                model.addAttribute("rowList", rowList);
                return "stat";
        }
}

試したこと

Hibernateで生成されたSQL(?を1に置き換えて)をpsqlで実行すると期待した値が取得できました。

$ psql -d 'postgresql://localhost:5432/otnj'
psql (12.8)
Type "help" for help.

otnj=# select
        distinct statrow0_.id as id2_0_,
        statrow0_.row as row3_0_
    from
        stat statrow0_
    where
        statrow0_.id=1
    order by
        statrow0_.row;

 id2_0_ | row3_0_
--------+---------
      1 |    1964
      1 |    1965
      1 |    1966

中略

      1 |    2020
      1 |    2021
(58 rows)

otnj=#

補足情報（FW/ツールのバージョンなど）

sts-4.13.0
PostgreSQL 12.8

Answer 1

findAllRows(1);

固定値渡してますよ。
リクエスト値取りたいならコントローラメソッドででStatIdを受け取る必要があるのでは。

Comments

[david56]

ご回答ありがとうございます。
現在はまだ試作段階なのでまだ固定値です。
今回のrowの値が固定になる問題とは無関係かと思います。
データもid=1の場合しか作っておりません。

otnj=# select * from stat;
id | row | col | data
----+------+-----+----------
1 | 1964 | 1 | 352832
1 | 1965 | 1 | 366649
1 | 1966 | 1 | 432937
中略
1 | 2020 | 1 | 4115828
1 | 2021 | 1 | 173263
(58 rows)

よろしくお願いいたします。

[m.ts10806]

質問内に言及されてなかったもので分かりませんでした。
質問本文に追記してください。

[m.ts10806]

あぁそういうことか。「試したこと」と繋がりませんでした。失礼。

[m.ts10806]

1回変数に受けた場合はどうでしょうか。

List<statRow > list = rowRepo.findDistinctRowByIdOrderByRow(statId);
for (StatRow statRow : list ) {

[david56]

コメントありがとうございます。
やってみましたが変わりませんでした。
取得個数は合っています。
Result Setからのコピーがうまくいっていない気がしますが、Result Setを直接触れないので、どうするのが良いのでしょうか。

[m.ts10806]

>取得個数は合っています。
rowRepo.findDistinctRowByIdOrderByRow(statId)自体の内容はどうなのでしょう。
もし内容が合ってるのでしたら詰め替えるメリットがないようにも思います。
合っていないのでしたらリポジトリへの投げ方も問題があるように思います。

[david56]

コメントありがとうございます。
質問内に書いた関連のコードはすべてありますが、findDistinctRowByIdOrderByRowはインタフェースのみで、実装はResult Setからのコピーも含めHibernateが自動でしています。
Spring Bootは素人なので、「リポジトリへの投げ方も問題」を解説して頂けますか。

[m.ts10806]

>実装はResult Setからのコピーも含めHibernateが自動でしています。
それは存じております。
まずは「なぜ詰め替えてるか」という点と「rowRepo.findDistinctRowByIdOrderByRow(statId)自体の内容」を教えてください。
「rowRepo.findDistinctRowByIdOrderByRow(statId)自体の内容」は「実装を教えて」ではなく、「返却値は想定通りか確認して」です。

[david56]

>「なぜ詰め替えてるか」
StatRowService#findAllRows()の処理内容のことでしょうか？
そうでしたら、レポジトリから直接Integerで戻す方法がわからないため、オブジェクト→Integer変換をしています。

>「rowRepo.findDistinctRowByIdOrderByRow(statId)自体の内容」
質問内の最後にあるpsql実行結果の通りになることを想定しています。
row3_0_列にあるように昇順で別々の値になるべきところが、すべて最初の値「1964」になってしまいます。

よろしくお願いいたします。

[m.ts10806]

> そうでしたら、レポジトリから直接Integerで戻す方法がわからないため、オブジェクト→Integer変換をしています。

利用するときにIDだけ取り出せばよいのでは。

>質問内の最後にあるpsql実行結果の通りになることを想定しています。
いいえ。「想定する結果」ではなく「rowRepo.findDistinctRowByIdOrderByRow(statId)」の実行結果のリストを直接デバッグして内容確認してください。
今は、forで取り出した内容を出力しているので、直接ではないです。

[david56]

: statRows=[com.sct.entity.StatRow@65226ed7, com.sct.entity.StatRow@65226ed7, com.sct.entity.StatRow@65226ed7,　...
同じハッシュ値＝同一インスタンスですね。
Hibernateのバグ？どうすれば良いのでしょうか？

[m.ts10806]

バグではないとは思います。試しにfindDistinctRowByIdOrderByRow以外で同じくIDを検索条件とする機能を利用してみてください。

[david56]

@Queryを用いることで解決に至りました。ありがとうございます。

@Repository
public interface StatRowRepositoryWithJpql extends JpaRepository<StatRow, StatId> {

@Query(value="SELECT DISTINCT s.row FROM StatRow s WHERE id = ?1 ORDER BY s.row")
List<Integer> findDistinctRowByIdOrderByRow(@Param("id") StatId id);
}