Use plus tunings for lookahead scan more widely#9813
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe scan tuning policy for architectures ChangesScan lookahead tuning
Assessment against linked issues
Comment |
|
I thought a bit about what to do for I think ultimately, C32 is just not an important use case, and we may just as well accept the regression. |
This comment has been minimized.
This comment has been minimized.
|
We have received a bug report for complex type (C64) perf regressions before (although on sort, not scan). So it is hard to say how important or not it is. |
🥳 CI Workflow Results🟩 Finished in 2h 36m: Pass: 100%/287 | Total: 6d 16h | Max: 2h 34m | Hits: 48%/400481See results here. |
Fixes: #9811
Performance impact: