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

fix: do not panic when empty tags are queried #24784

Merged
merged 1 commit into from
Mar 18, 2024

Conversation

davidby-influx
Copy link
Contributor

Do not panic if a cursor array is nil and the number of timestamps is retrieved.

closes #24536

Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes #24536
Copy link
Member

@gwossum gwossum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@davidby-influx davidby-influx merged commit bc80e88 into master-1.x Mar 18, 2024
9 checks passed
@davidby-influx davidby-influx deleted the DSB_arraycursor_fix branch March 18, 2024 22:28
davidby-influx added a commit that referenced this pull request Mar 18, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes #24536

(cherry picked from commit bc80e88)
davidby-influx added a commit that referenced this pull request Mar 18, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes #24536

(cherry picked from commit bc80e88)
davidby-influx added a commit that referenced this pull request Mar 18, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes #24536

(cherry picked from commit bc80e88)
davidby-influx added a commit that referenced this pull request Mar 18, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes #24536

(cherry picked from commit bc80e88)
davidby-influx added a commit that referenced this pull request Mar 19, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes #24536

(cherry picked from commit bc80e88)
davidby-influx added a commit that referenced this pull request Mar 19, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes #24536

(cherry picked from commit bc80e88)
chengshiwen pushed a commit to chengshiwen/influxdb that referenced this pull request Aug 11, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes influxdata#24536
chengshiwen pushed a commit to chengshiwen/influxdb that referenced this pull request Aug 27, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes influxdata#24536
chengshiwen pushed a commit to chengshiwen/influxdb that referenced this pull request Aug 28, 2024
Do not panic if a cursor array is nil and the number
of timestamps is retrieved.

closes influxdata#24536
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants