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: fallback on btree indexes when hash is unavailable #1856

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

olirice
Copy link
Contributor

@olirice olirice commented Dec 4, 2024

What kind of change does this PR introduce?

Edits an existing migration to fallback onto btree indexes when hash indexes are unavailable.

This change allows auth to be compatible with the orioledb storage engine. At time of writing orioledb only support btree indexes. That issue is expected to be addressed by the end of Q1 - 2025. Once that support is added, both heap and orioledb projects will both use the code path for hash indexes. hash indexes are preferred because they are safer from timing attacks

A reminder has been set for May 1st for us to

  • remove the fork in this migration
  • identify and update the index type (from btree to hash) on any orioledb-alpha projects that are still in existence

@olirice olirice requested a review from a team as a code owner December 4, 2024 15:18
@olirice olirice force-pushed the or/fallback-on-btree-when-hash-unavailable branch from f047ba4 to e98b9d3 Compare December 4, 2024 15:22
@coveralls
Copy link

coveralls commented Dec 4, 2024

Pull Request Test Coverage Report for Build 12162834089

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 56.998%

Totals Coverage Status
Change from base Build 12132961712: 0.0%
Covered Lines: 9546
Relevant Lines: 16748

💛 - Coveralls

@J0 J0 changed the title Fallback on btree indexes when hash is unavailable fix: fallback on btree indexes when hash is unavailable Dec 4, 2024
@J0 J0 merged commit b33bc31 into master Dec 4, 2024
3 of 6 checks passed
@J0 J0 deleted the or/fallback-on-btree-when-hash-unavailable branch December 4, 2024 16:24
J0 added a commit that referenced this pull request Dec 4, 2024
J0 added a commit that referenced this pull request Dec 4, 2024
Reverts #1856. We plan to roll this out in the next version
(v2.166.0) as per internal discussion
cstockton pushed a commit that referenced this pull request Dec 5, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.165.0](v2.164.0...v2.165.0)
(2024-12-05)


### Features

* add email validation function to lower bounce rates
([#1845](#1845))
([2c291f0](2c291f0))
* use embedded migrations for `migrate` command
([#1843](#1843))
([e358da5](e358da5))


### Bug Fixes

* fallback on btree indexes when hash is unavailable
([#1856](#1856))
([b33bc31](b33bc31))
* return the error code instead of status code
([#1855](#1855))
([834a380](834a380))
* revert fallback on btree indexes when hash is unavailable
([#1858](#1858))
([1c7202f](1c7202f))
* update ip mismatch error message
([#1849](#1849))
([49fbbf0](49fbbf0))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

4 participants