Skip to content

DAOS-18487 utils: refine ddb_iov_to_printable_buf#17594

Open
liuxuezhao wants to merge 1 commit intomasterfrom
lxz/ddb_print_fix
Open

DAOS-18487 utils: refine ddb_iov_to_printable_buf#17594
liuxuezhao wants to merge 1 commit intomasterfrom
lxz/ddb_print_fix

Conversation

@liuxuezhao
Copy link
Contributor

@liuxuezhao liuxuezhao commented Feb 25, 2026

Current code may cause some integer keys' output not-readable, for example dkey 10, 60 printed by ddb as -
DKEY: (/[4]/[2]/[0]) /b2b37b1a-2894-4970-b51c-1a964526bb02/948289433254841843.256.535.2/ {8}
DKEY: (/[4]/[2]/[2]) /b2b37b1a-2894-4970-b51c-1a964526bb02/948289433254841843.256.535.2/<{8}

That cause confuse and misunderstanding of ddb output.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

Current code may cause some keys' output not-readable, for example
dkey 10, 60 printed by ddb as -
DKEY: (/[4]/[2]/[0]) /b2b37b1a-2894-4970-b51c-1a964526bb02/948289433254841843.256.535.2/
{8}
DKEY: (/[4]/[2]/[2]) /b2b37b1a-2894-4970-b51c-1a964526bb02/948289433254841843.256.535.2/<{8}

That cause confuse and misunderstanding of ddb output.

Signed-off-by: Xuezhao Liu <xuezhao.liu@hpe.com>
@github-actions
Copy link

Errors are Unable to load ticket data
https://daosio.atlassian.net/browse/DAOS-18487

@daosbuild3
Copy link
Collaborator

@daosbuild3
Copy link
Collaborator

Test stage Unit Test with memcheck on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17594/1/testReport/

Copy link
Contributor

@Nasf-Fan Nasf-Fan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This patch is workable for integer key, but maybe generate wrong output for sting formatted key. I will make more suitable solution via DAOS-18625.

@Nasf-Fan
Copy link
Contributor

Nasf-Fan commented Mar 2, 2026

This patch is workable for integer key, but maybe generate wrong output for sting formatted key. I will make more suitable solution via DAOS-18625.

Here is the patch:
#17617

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants