Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

added next_key_bytes #9765

Merged
merged 1 commit into from
Dec 9, 2020
Merged

added next_key_bytes #9765

merged 1 commit into from
Dec 9, 2020

Conversation

kimjh2005
Copy link
Contributor

@kimjh2005 kimjh2005 commented Dec 9, 2020

Change Description

Added next_key_bytes field to get_table_rows_result structure to indicate there are more rows.
As the value is encoded as "bytes", the subsequent query must use --encode-type "bytes" to use it.

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

For the chain/get_kv_table_rows endpoint, when the return type's “more” field is set to true, the “next_key” field will be set to a value that can be provided with --lower (or --upper if --reverse is also being used) using the same --encode-type that was provided with the original request (NOTE: This feature is not yet implemented). Also the “next_key_bytes” field will be set to a value that can be provided with --lower (or --upper if --reverse is also being used) but “ --encode-type bytes” must be used.

Copy link
Contributor

@brianjohnson5972 brianjohnson5972 left a comment

Choose a reason for hiding this comment

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

Please update the API documentation portion, indicating that this is reference to the chain/get_kv_table_rows endpoint.

@kimjh2005 kimjh2005 merged commit f8e9a0e into develop Dec 9, 2020
@kj4ezj kj4ezj deleted the fix_next_key_bytes branch July 19, 2021 17:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants