planner: standardize some terminologies in binding pkg (#48957)

ref #48875

pkg/bindinfo/bind_cache.go

@@ -131,10 +131,10 @@ func (c *bindCache) delete(key bindCacheKey) bool {
 	return false
 }
 
-// GetBindRecord gets the BindRecord from the cache.
+// GetBinding gets the BindRecord from the cache.
 // The return value is not read-only, but it shouldn't be changed in the caller functions.
 // The function is thread-safe.
-func (c *bindCache) GetBindRecord(sqlDigest, normalizedSQL, _ string) *BindRecord {
+func (c *bindCache) GetBinding(sqlDigest, normalizedSQL, _ string) *BindRecord {
 	c.lock.Lock()
 	defer c.lock.Unlock()
 	bindRecords := c.get(bindCacheKey(sqlDigest))
@@ -146,10 +146,10 @@ func (c *bindCache) GetBindRecord(sqlDigest, normalizedSQL, _ string) *BindRecor
 	return nil
 }
 
-// GetBindRecordBySQLDigest gets the BindRecord from the cache.
+// GetBindingBySQLDigest gets the BindRecord from the cache.
 // The return value is not read-only, but it shouldn't be changed in the caller functions.
 // The function is thread-safe.
-func (c *bindCache) GetBindRecordBySQLDigest(sqlDigest string) (*BindRecord, error) {
+func (c *bindCache) GetBindingBySQLDigest(sqlDigest string) (*BindRecord, error) {
 	c.lock.Lock()
 	defer c.lock.Unlock()
 	bindings := c.get(bindCacheKey(sqlDigest))
@@ -163,10 +163,10 @@ func (c *bindCache) GetBindRecordBySQLDigest(sqlDigest string) (*BindRecord, err
 	return bindings[0], nil
 }
 
-// GetAllBindRecords return all the bindRecords from the bindCache.
+// GetAllBindings return all the bindRecords from the bindCache.
 // The return value is not read-only, but it shouldn't be changed in the caller functions.
 // The function is thread-safe.
-func (c *bindCache) GetAllBindRecords() []*BindRecord {
+func (c *bindCache) GetAllBindings() []*BindRecord {
 	c.lock.Lock()
 	defer c.lock.Unlock()
 	values := c.cache.Values()
@@ -178,9 +178,9 @@ func (c *bindCache) GetAllBindRecords() []*BindRecord {
 	return bindRecords
 }
 
-// SetBindRecord sets the BindRecord to the cache.
+// SetBinding sets the BindRecord to the cache.
 // The function is thread-safe.
-func (c *bindCache) SetBindRecord(sqlDigest string, meta *BindRecord) (err error) {
+func (c *bindCache) SetBinding(sqlDigest string, meta *BindRecord) (err error) {
 	c.lock.Lock()
 	defer c.lock.Unlock()
 	cacheKey := bindCacheKey(sqlDigest)
@@ -194,9 +194,9 @@ func (c *bindCache) SetBindRecord(sqlDigest string, meta *BindRecord) (err error
 	return
 }
 
-// RemoveBindRecord removes the BindRecord which has same originSQL with specified BindRecord.
+// RemoveBinding removes the BindRecord which has same originSQL with specified BindRecord.
 // The function is thread-safe.
-func (c *bindCache) RemoveBindRecord(sqlDigest string, meta *BindRecord) {
+func (c *bindCache) RemoveBinding(sqlDigest string, meta *BindRecord) {
 	c.lock.Lock()
 	defer c.lock.Unlock()
 	metas := c.getCopiedVal(bindCacheKey(sqlDigest))

pkg/bindinfo/capture_test.go

@@ -329,7 +329,7 @@ func TestBindingSource(t *testing.T) {
 	tk.MustExec("create global binding for select * from t where a > 10 using select * from t ignore index(idx_a) where a > 10")
 	bindHandle := dom.BindHandle()
 	sql, sqlDigest := internal.UtilNormalizeWithDefaultDB(t, "select * from t where a > ?")
-	bindData := bindHandle.GetBindRecord(sqlDigest, sql, "test")
+	bindData := bindHandle.GetGlobalBinding(sqlDigest, sql, "test")
 	require.NotNil(t, bindData)
 	require.Equal(t, "select * from `test` . `t` where `a` > ?", bindData.OriginalSQL)
 	require.Len(t, bindData.Bindings, 1)
@@ -341,7 +341,7 @@ func TestBindingSource(t *testing.T) {
 	tk.MustQuery("select * from t where a > 10")
 	bindHandle.SaveEvolveTasksToStore()
 	sql, sqlDigest = internal.UtilNormalizeWithDefaultDB(t, "select * from t where a > ?")
-	bindData = bindHandle.GetBindRecord(sqlDigest, sql, "test")
+	bindData = bindHandle.GetGlobalBinding(sqlDigest, sql, "test")
 	require.NotNil(t, bindData)
 	require.Equal(t, "select * from `test` . `t` where `a` > ?", bindData.OriginalSQL)
 	require.Len(t, bindData.Bindings, 2)
@@ -362,7 +362,7 @@ func TestBindingSource(t *testing.T) {
 	tk.MustExec("admin capture bindings")
 	bindHandle.CaptureBaselines()
 	sql, sqlDigest = internal.UtilNormalizeWithDefaultDB(t, "select * from t where a < ?")
-	bindData = bindHandle.GetBindRecord(sqlDigest, sql, "test")
+	bindData = bindHandle.GetGlobalBinding(sqlDigest, sql, "test")
 	require.NotNil(t, bindData)
 	require.Equal(t, "select * from `test` . `t` where `a` < ?", bindData.OriginalSQL)
 	require.Len(t, bindData.Bindings, 1)