Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the range queries documentation with a new example #97

Closed
DawidNiezgodka opened this issue Oct 6, 2022 · 0 comments · Fixed by #99
Closed

Update the range queries documentation with a new example #97

DawidNiezgodka opened this issue Oct 6, 2022 · 0 comments · Fixed by #99
Assignees
Labels
area/documentation Relates to the documentation

Comments

@DawidNiezgodka
Copy link
Contributor

The current example for range queries is misleading because of the possible override of records. Consider the ingestion of the following two records:

  {
    "key": 1,
    "value": {
      "userId": 1,
      "purchaseId": "abc",
      "rating": 7
    }
  }

and then:

  {
    "key": 1,
    "value": {
      "userId": 1,
      "purchaseId": "def",
      "rating": 7
    }
  }

The ingestion of the second leads to the range index for 0[...]1_0[...]7 being overwritten.
We will update the docu with an example of timestamp with the assumption of events that happen continuously and it is not possible to insert the same time for the identical key.

@DawidNiezgodka DawidNiezgodka added the area/documentation Relates to the documentation label Oct 6, 2022
@DawidNiezgodka DawidNiezgodka self-assigned this Oct 6, 2022
@raminqaf raminqaf added this to the Complex GraphQL queries milestone Oct 12, 2022
DawidNiezgodka added a commit that referenced this issue Oct 12, 2022
This PR resolves #97.

Co-authored-by: Christoph Böhm <christoph.boehm@bakdata.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Relates to the documentation
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants