weaviate icon indicating copy to clipboard operation
weaviate copied to clipboard

Graph connection is not made when using the group function

Open bobvanluijt opened this issue 4 years ago • 4 comments

The following query:

{
  Get {
    Publication(
      group: {
        type: merge
        force: 0.1
      }
    ) {
      name
      hasArticles {
        ... on Article {
          title
        }
      }
    }
  }
}

results in:

{
  "data": {
    "Get": {
      "Publication": null
    }
  },
  "errors": [
    {
      "locations": [
        {
          "column": 5,
          "line": 3
        }
      ],
      "message": "interface conversion: interface {} is strfmt.UUID, not string",
      "path": [
        "Get",
        "Publication"
      ]
    }
  ]
}

bobvanluijt avatar Apr 26 '21 13:04 bobvanluijt

Thanks for discovering this bug. If I remember correctly, group/merge was only implemented for primitive props, not for ref-props. So even when this is fixed - and the error gone - I'm not sure what the behavior is going to be.

etiennedi avatar Apr 26 '21 13:04 etiennedi

Aha - gotcha, in the past I did demo this and it seemed to work well

bobvanluijt avatar Apr 26 '21 13:04 bobvanluijt

Interesting, do you remember when it last worked? I think this feature hasn't been touched in ages, but not sure.

etiennedi avatar Apr 26 '21 14:04 etiennedi

Just validated against v1.0.0 and get the same error, so this must have been broken before this already.

etiennedi avatar Apr 26 '21 14:04 etiennedi