Skip to content

Conversation

@OleinikovasV
Copy link
Member

@OleinikovasV OleinikovasV commented Feb 8, 2025

Previous fix was not disabling the columns because schema is set to None by default. :(
Moved this column check patch upstream to ensure it is being checked by default.

Related to bug: #94

@OleinikovasV OleinikovasV changed the title Disable binding affinity query Disable binding affinity query - again Feb 8, 2025
@github-actions
Copy link

github-actions bot commented Feb 8, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/plinder/core/scores
  index.py 46
  query.py
Project Total  

This report was generated by python-coverage-comment-action

@OleinikovasV OleinikovasV requested a review from Ninjani February 8, 2025 10:55
@OleinikovasV OleinikovasV merged commit d303084 into main Feb 8, 2025
3 checks passed
@OleinikovasV OleinikovasV deleted the disable_binding_affinity_query branch February 8, 2025 11:07
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.

3 participants