Skip to content

feat(hash): initialize field expiration metadata and subkey en/decoding - #3444

Merged
PragmaTwice merged 14 commits into
apache:unstablefrom
PragmaTwice:hfe-task-1
Apr 14, 2026
Merged

PragmaTwice merged 14 commits into
apache:unstablefrom
PragmaTwice:hfe-task-1

Conversation

@PragmaTwice

Copy link
Copy Markdown
Member

Part of #3436. See proposal #3432.

This PR includes:

  • metadata changes for the new encoding
  • subkey encoding/decoding and related logic in redis_hash.cc
  • new config option adding
  • an enhancement to SubKeyScanner so that it can be used for HFE
  • run hash test cases for both modes

Assisted-by: Codex

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@PragmaTwice
PragmaTwice merged commit 37140d6 into apache:unstable Apr 14, 2026
38 of 39 checks passed
@jihuayu

jihuayu commented Apr 15, 2026

Copy link
Copy Markdown
Member

@PragmaTwice SonarCloud reported two blocker issues, which will cause the unstable branch CI failed. Could you please check on this?
image

url: https://sonarcloud.io/project/issues?issueStatuses=OPEN%2CCONFIRMED&types=BUG&severities=BLOCKER%2CCRITICAL%2CMAJOR%2CMINOR&inNewCodePeriod=true&sinceLeakPeriod=true&id=apache_kvrocks

nagisa-kunhah pushed a commit to nagisa-kunhah/kvrocks that referenced this pull request Apr 29, 2026
…ng (apache#3444)

Part of apache#3436. See proposal apache#3432.

This PR includes:
- metadata changes for the new encoding
- subkey encoding/decoding and related logic in `redis_hash.cc`
- new config option adding
- an enhancement to `SubKeyScanner` so that it can be used for HFE
- run hash test cases for both modes

Assisted-by: Codex
nagisa-kunhah pushed a commit to nagisa-kunhah/kvrocks that referenced this pull request Apr 29, 2026
…ng (apache#3444)

Part of apache#3436. See proposal apache#3432.

This PR includes:
- metadata changes for the new encoding
- subkey encoding/decoding and related logic in `redis_hash.cc`
- new config option adding
- an enhancement to `SubKeyScanner` so that it can be used for HFE
- run hash test cases for both modes

Assisted-by: Codex
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants