Elasticsearch導入 #224
Merged
Elasticsearch導入 #224
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Sakeのカラムの内、string型とtext型のものを対象に検索できるうようにした - kuromojiとICU Analysisプラグインによって日本語の検索精度をよくした - https://www.elastic.co/guide/en/elasticsearch/plugins/current/analysis-kuromoji.html - https://www.elastic.co/guide/en/elasticsearch/plugins/current/analysis-icu.html - カスタマイズした辞書と同義語を登録した
適切な場所を決めるまでの仮置きとして、ElasticSearch用の検索ボックスをheaderに配置
- 列挙型の書式を統一 - portsの指定を文字列型に変更。60番以下のポート指定でおかしくなる可能性があるため。 - https://docs.docker.com/compose/compose-file/compose-file-v3/#ports Co-authored-by: SAITOU Keita <[email protected]>
Co-authored-by: SAITOU Keita <[email protected]>
ELASTICSEARCH_HOSTとELASTICSEARCH_HOSTSが似ていてわかりづらかった。 kibanaのための環境変数ELASTICSEARCH_HOSTSは名前を変えられないため、 ELASTICSEARCH_HOSTをELASTICSEARCH_HOSTNAME に変更した。
原料米や掛け米はただ列挙するだけだと不自然な文章になるため、原料米: XXと表示するようにした
- 改行を修正 - Elasticsearchとユーザーメニューの間に水平線を挿入
- 掛け米・原料米表記を削除 - 検索結果のテキストを<span>でくくった
Elasticsearch導入その2
Elasticsearchのhostとportを別々の環境変数にしていたが、まとめられることがわかったためまとめた
- Enumを使うカラムを検索できるようにするためElasticSearchのインデックスに追加 - ソートするためにdate型とintegerのカラムをElasticSearchのインデックスに追加
masterにtestが追加されたため、GitHub Action実行時にElasticsearchのコンテナを動かさないといけなくなった。 （動かしておかないと、酒の登録のテストとかでFailする）
step名を修正 Co-authored-by: SAITOU Keita <[email protected]>
ロングオプションを使うようにした Co-authored-by: SAITOU Keita <[email protected]>
Elasticsearch導入その3
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#150 #159 #169 にてElasticsearchがある程度かたちになってきたので本番環境で動かしてみよう。
herokuにはbonsai-elasticsearchの設定をしておいたので多分動くはず。