pingcap/tidb

Use a new struct to store the props generated by `preparePossibleProperties`

Open

#10,749 opened on 2019年6月10日

GitHub で見る
 (0 comments) (1 reaction) (2 assignees)Go (40,090 stars) (6,186 forks)batch import
help wantedsig/plannertype/performance

説明

after https://github.com/pingcap/tidb/pull/10548 There'll be multiple props which share the same suffix. We can implement a struct which can store the eqPrefixLen to cut off the possible props then reduce the search space.

コントリビューターガイド

技術スタック
go
領域
databaseperformance
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
4
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Go programmingTiDB internals
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
25
調査方針
Review the referenced PR #10548 to understand the background about multiple props sharing suffixes. Look at the current implementation of `preparePossibleProperties` in the TiDB source code to identify where the new struct should be introduced. The goal is to encapsulate `eqPrefixLen` and the props in a struct to reduce search space. There are no comments or additional guidance, so the contributor may need to propose a design in the issue first.
Use a new struct to store the props generated by `preparePossibleProperties` · pingcap/tidb#10749 | Good First Issue