Hibernate-reactive: MS SQL: kegagalan saat membuat tabel sementara lokal

Dibuat pada 3 Mei 2021 · 9Komentar · Sumber: hibernate/hibernate-reactive

Menemukan masalah saat menjalankan UnionSubclassInheritanceTest dengan MS SQL.

HR menggunakan ReactiveIdTableSupport untuk menghasilkan SQL untuk membuat tabel sementara lokal.

MS SQL mengharapkan nama tabel sementara lokal dimulai dengan tanda hash tunggal:
https://docs.microsoft.com/en-us/sql/t-sql/statements/create-table-transact-sql#temporary -tables

bug

Sumber

tsegismont

Semua 9 komentar

Ya itu karena kami menggunakan ReactiveBulkIdStrategy alih-alih yang dikembalikan oleh AbstractTransactSQLDialect.getDefaultMultiTableBulkIdStrategy() . Seharusnya cukup mudah untuk diperbaiki. Jangan terlalu khawatir tentang ini untuk saat ini.

gavinking pada 3 Mei 2021

👍1

@tsegismont itu harus cukup untuk mengubah ReactiveIdTableSupport. generateIdTableName () sebagai berikut:

    <strong i="8">@Override</strong>
    public String generateIdTableName(String baseName) {
        return (dialect instanceof SQLServerDialect ?  "#" : "ht_") + baseName;
    }

gavinking pada 3 Mei 2021

@gavinking terima kasih atas tipnya. Saya menggunakan sesuatu yang sedikit berbeda (untuk mempertahankan awalan ht_ ) dan juga harus mengubah metode getCreateIdTableCommand .

Lihat https://github.com/tsegismont/hibernate-reactive/commit/84caabcefb4eb1a47e7e0c2d931584038d162393

tsegismont pada 4 Mei 2021

Saya menggunakan sesuatu yang sedikit berbeda (untuk mempertahankan awalan ht_ )

Yah saya percaya jika Anda memeriksa Hibernate ORM (yang kami coba selaraskan), itu menjatuhkan awalan ht_ dalam kasus SQL Server, karena tabel sementara memiliki namespace mereka sendiri.

gavinking pada 4 Mei 2021

@gavinking :+1: https://github.com/DavideD/hibernate-reactive/pull/13/commits/a52e8e3825dd360a3b3d267fed0b8ec0daed052f

tsegismont pada 4 Mei 2021

👍1

@tsegismont bisakah kita tutup yang ini?

gavinking pada 24 Mei 2021

Saya membagikan beberapa perubahan dalam PR ke garpu @DavideD .

Jika perubahan ini telah dipilih di sini maka ya masalahnya bisa ditutup.

tsegismont pada 25 Mei 2021

Terima kasih @tsegismont , saya akan melihatnya hari ini