118: queryRecords should return [] when no matching records exist, not an error #121

Merged

telackey merged 1 commits from telackey/118 into main 2023-11-28 20:41:28 +00:00

Conversation 0 Commits 1 Files Changed 1 +8 -1

telackey commented 2023-11-28 19:30:24 +00:00

Member

Copy Link

Whenever a key is used which doesn't exist in the DB at all, an error is returned like this:

[
  {
    message: `rpc error: code = Unknown desc = store doesn't have key: "\\atypeApplicationRecord"`,
    path: [ 'queryRecords' ]
  }
]

That doesn't make any sense, as it is not an error to find out that the thing you asked for does not exist. This changes it to return an empty list.

Whenever a key is used which doesn't exist in the DB at all, an error is returned like this: ``` [ { message: `rpc error: code = Unknown desc = store doesn't have key: "\\atypeApplicationRecord"`, path: [ 'queryRecords' ] } ] ``` That doesn't make any sense, as it is not an error to find out that the thing you asked for does not exist. This changes it to return an empty list.

telackey added 1 commit 2023-11-28 19:30:24 +00:00

118: queryRecords should return [] when no matching records exist, not an error cd144330aa

telackey requested review from dboreham 2023-11-28 19:30:37 +00:00

dboreham approved these changes 2023-11-28 20:31:46 +00:00

telackey merged commit 0adc932400 into main 2023-11-28 20:41:28 +00:00

telackey deleted branch telackey/118 2023-11-28 20:41:29 +00:00

telackey referenced this issue from a commit 2023-11-28 20:41:30 +00:00

118: queryRecords should return [] when no matching records exist, not an error (#121)

telackey referenced this issue from a commit 2023-11-28 23:22:08 +00:00

Remove extra error handling (not needed with https://git.vdb.to/cerc-io/laconicd/pulls/121)

telackey referenced this pull request 2023-11-29 17:34:56 +00:00

queryRecords should return [] when no records of the type exist, not an error #118

Sign in to join this conversation.

Reviewers

No reviewers

dboreham

Labels

Clear labels   

bug

Something isn't working

  

C:CLI

  

C:Crypto

  

C:Encoding

  

C:Proto

  

C:Types

  

dependencies

Pull requests that update a dependency file

  

docker

Pull requests that update Docker code

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

go

Pull requests that update Go code

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

high priority

  

in progress

currently working on

  

invalid

This doesn't seem right

  

javascript

Pull requests that update Javascript code

  

low priority

  

medium priority

  

question

Further information is requested

  

Status: Stale

  

Type: ADR

  

Type: Build

  

Type: CI

  

Type: Docs

  

Type: Tests

  

urgent

Top priority issue

  

wontfix

This will not be worked on

  

Copied from Github

An issue or PR manually copied from GitHub.

  

Kind/Breaking

Breaking change that won't be backward compatible

  

Kind/Bug

Something is not working

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Feature

New functionality

  

Kind/Security

This is security issue

  

Kind/Testing

Issue or pull request related to testing

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

No Label

bug

C:CLI

C:Crypto

C:Encoding

C:Proto

C:Types

dependencies

docker

documentation

duplicate

enhancement

go

good first issue

help wanted

high priority

in progress

invalid

javascript

low priority

medium priority

question

Status: Stale

Type: ADR

Type: Build

Type: CI

Type: Docs

Type: Tests

urgent

wontfix

Copied from Github

Kind/Breaking

Kind/Bug

Kind/Documentation

Kind/Enhancement

Kind/Feature

Kind/Security

Kind/Testing

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Status

Abandoned

Status

Blocked

Status

Need More Info

Milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cerc-io/laconicd-deprecated#121

No description provided.