Skip to content

core: failed to start if box.info.replication with gaps #240

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

Merged
merged 1 commit into from
Feb 27, 2025

Conversation

oleg-jukovec
Copy link
Collaborator

@oleg-jukovec oleg-jukovec commented Feb 26, 2025

The iterator does not handle the case where there are no records in the box.info.replication table in case of deleting old records, for examle:

- 1:
    id: 1
    uuid: 77f30d9a-d344-4f7a-a18c-578fb5ec2540
    lsn: 4
    name: null
  3:
    id: 3
    uuid: 0ce528ca-ea35-42be-96b5-35ba610f9a3e
    lsn: 0
    downstream:
      status: follow
      idle: 0.14953243732452
      vclock: {1: 4}
      lag: 0.0015106201171875
    name: null

It leads to the error and unable to start queue:

queue_state.lua:50: attempt to index a nil value

Verified

This commit was signed with the committer’s verified signature.
oleg-jukovec Oleg Jukovec
The iterator does not handle the case where there are no records in
the box.info.replication table in case of deleting old records,
for examle:

```
- 1:
    id: 1
    uuid: 77f30d9a-d344-4f7a-a18c-578fb5ec2540
    lsn: 4
    name: null
  3:
    id: 3
    uuid: 0ce528ca-ea35-42be-96b5-35ba610f9a3e
    lsn: 0
    downstream:
      status: follow
      idle: 0.14953243732452
      vclock: {1: 4}
      lag: 0.0015106201171875
    name: null
```

It leads to the error and unable to start queue:

```
queue_state.lua:50: attempt to index a nil value
```
@oleg-jukovec oleg-jukovec force-pushed the oleg-jukovec/gh-no-fix-replication-nil branch from 32c4247 to 6cab6ff Compare February 26, 2025 12:05
@oleg-jukovec oleg-jukovec merged commit ccf6742 into master Feb 27, 2025
23 checks passed
@oleg-jukovec oleg-jukovec deleted the oleg-jukovec/gh-no-fix-replication-nil branch February 27, 2025 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants