pingcap/tidb

Proposal to use atomic.Pointer[T] instead of atomic.Value

Open

#44,736 opened on Jun 16, 2023

View on GitHub
 (6 comments) (0 reactions) (1 assignee)Go (6,186 forks)batch import
good first issuetype/enhancement

Repository metrics

Stars
 (40,090 stars)
PR merge metrics
 (Avg merge 14d 4h) (346 merged PRs in 30d)

Description

Enhancement

Currently, the usage of atomic.Value is prevalent in our codebase. However, I would like to propose using atomic.Pointer[T] instead. The reason being that atomic.Pointer[T] provides a clearer expression of the type, which can help in reducing type conversions.

Therefore, I suggest that we consider using atomic.Pointer[T] in our codebase wherever possible. This change will not only result in better code clarity but also improve the performance of our code.

Contributor guide

Research direction
Identify all usages of atomic.Value in the codebase, determine the concrete type T for each, and replace with atomic.Pointer[T] while ensuring no functional changes.
Tech stack
go
Domain
backend
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Go
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50