Skip to content

Conversation

@alecgrieser
Copy link
Collaborator

This adds a test case to showcase the problem with #3734. It modifies enum.yamsql with a new test case analogous to the one in the issue. There are two new queries that are planned with an index scan over an index on an enum valued field, and both of them fail with the same NullPointerException that is alluded to in the bug report.

This adds a test case to showcase the problem with FoundationDB#3734. It modifies `enum.yamsql` with a new test case analogous to the one in the issue. There are two new queries that are planned with an index scan over an index on an enum valued field, and both of them fail with the same `NullPointerException` that is alluded to in the bug report.
@alecgrieser alecgrieser added the testing improvement Change that improves our testing label Nov 10, 2025
@github-actions
Copy link

📊 Metrics Diff Analysis Report

Summary

  • New queries: 7
  • Dropped queries: 0
  • Plan changed + metrics changed: 0
  • Plan unchanged + metrics changed: 0
ℹ️ About this analysis

This automated analysis compares query planner metrics between the base branch and this PR. It categorizes changes into:

  • New queries: Queries added in this PR
  • Dropped queries: Queries removed in this PR. These should be reviewed to ensure we are not losing coverage.
  • Plan changed + metrics changed: The query plan has changed along with planner metrics.
  • Metrics only changed: Same plan but different metrics

The last category in particular may indicate planner regressions that should be investigated.

New Queries

Count of new queries by file:

  • yaml-tests/src/test/resources/enum.metrics.yaml: 7

@alecgrieser alecgrieser requested a review from hatyo November 12, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing improvement Change that improves our testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant