129: Index multivalued attributes. #128

Merged

telackey merged 4 commits from telackey/tags into main 2023-12-19 06:55:12 +00:00

Conversation 0 Commits 4 Files Changed 3 +80 -9

telackey commented 2023-12-18 22:24:33 +00:00

Member

Copy Link

This fixes #129, by indexing each value of a multivalued attribute.

This handles at least the most common use case, so that we can search on a single value of the attribute.

❯ laconic -c ~/.laconic/local.yml cns record list --all --type ApplicationDeploymentRequest --tags b
[
  {
    "id": "bafyreidrp4pylixp44rkxu5il72qhwwc4ir5ctdnssps5rnelstloxivwm",
    "names": null,
    "owners": [
      "FCCE01FCC2472AEDBCF33902907F33262445AC2C"
    ],
    "bondId": "4ef470a9207f00fc07663623d092a14c310794b616eb53b085cfe6976e82f56d",
    "createTime": "2023-12-18T22:13:23Z",
    "expiryTime": "2024-12-17T22:13:23Z",
    "attributes": {
      "type": "ApplicationDeploymentRequest",
      "version": "1.0.6",
      "application": "crn://cerc-io/applications/test-progressive-web-app@0.1.1",
      "config": {
        "env": {
          "CERC_WEBAPP_DEBUG": "57588a9d"
        }
      },
      "tags": [
        "a",
        "b",
        "c"
      ]
    }
  }
]

This fixes #129, by indexing each value of a multivalued attribute. This handles at least the most common use case, so that we can search on a single value of the attribute. ``` ❯ laconic -c ~/.laconic/local.yml cns record list --all --type ApplicationDeploymentRequest --tags b [ { "id": "bafyreidrp4pylixp44rkxu5il72qhwwc4ir5ctdnssps5rnelstloxivwm", "names": null, "owners": [ "FCCE01FCC2472AEDBCF33902907F33262445AC2C" ], "bondId": "4ef470a9207f00fc07663623d092a14c310794b616eb53b085cfe6976e82f56d", "createTime": "2023-12-18T22:13:23Z", "expiryTime": "2024-12-17T22:13:23Z", "attributes": { "type": "ApplicationDeploymentRequest", "version": "1.0.6", "application": "crn://cerc-io/applications/test-progressive-web-app@0.1.1", "config": { "env": { "CERC_WEBAPP_DEBUG": "57588a9d" } }, "tags": [ "a", "b", "c" ] } } ] ```

telackey added 1 commit 2023-12-18 22:24:34 +00:00

Index multivalued attributes. 58789fbf62

telackey self-assigned this 2023-12-18 22:24:44 +00:00

telackey changed title from 127: Index multivalued attributes. to 129: Index multivalued attributes. 2023-12-18 22:29:01 +00:00

telackey referenced this pull request 2023-12-18 22:31:47 +00:00

GQL only allows a single value of a multivalue attribute to be specified at a time #127

telackey requested review from dboreham 2023-12-18 22:34:11 +00:00

telackey added 1 commit 2023-12-18 22:35:01 +00:00

Remove debugging lines 61169a43c0

telackey added 1 commit 2023-12-18 22:37:51 +00:00

if instead of switch 911f4ec252

telackey added 1 commit 2023-12-18 23:53:40 +00:00

Add test case a310c375ab

telackey merged commit ae07fc0147 into main 2023-12-19 06:55:12 +00:00

telackey deleted branch telackey/tags 2023-12-19 06:55:12 +00:00

telackey referenced this issue from a commit 2023-12-19 06:55:14 +00:00

129: Index multivalued attributes. (#128)

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

telackey

1 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#128

No description provided.