-
Notifications
You must be signed in to change notification settings - Fork 653
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
postgresql利用時のtranslateのFROMの修正 #5472
postgresql利用時のtranslateのFROMの修正 #5472
Conversation
Codecov Report
@@ Coverage Diff @@
## 4.2 #5472 +/- ##
=========================================
Coverage 78.74% 78.75%
Complexity 6251 6251
=========================================
Files 468 468
Lines 21007 21010 +3
=========================================
+ Hits 16543 16547 +4
+ Misses 4464 4463 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@kazuyoshikakihara ありがとうございます。 |
@kiy0taka なるほど了解しました。 |
PostgreSQL固有の問題なので、PostgreSQLの場合のみ動作する形でテストケース追加しました。 |
@kazuyoshikakihara |
概要(Overview・Refs Issue)
PostgreSQL使用時に使われるtranslate用のFROM文字列が一部間違っており、意図どおりにtranslateされず、「ウヰスキー」を「うゐすきー」で検索してもヒットしなかったりするので修正
方針(Policy)
最短で修正
実装に関する補足(Appendix)
特になし
テスト(Test)
PostgreSQL固有の問題のため自動テストなし。
ローカルの環境で商品名に「ウヰスキー」を登録し、「うゐすきー」で検索した場合、修正前はヒットしないが、修正後はヒットすることを確認。
相談(Discussion)
特になし
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目