Skip to content

Commit

Permalink
Add potential SQL deprecation warning at Encoder instantiation time (#…
Browse files Browse the repository at this point in the history
…2974)

Summary:
Pull Request resolved: #2974

We are considering deprecating SQLAlchechemy support in Ax due in large part to a perceived lack of usage by our OSS community. If you are using this feature please let us know as soon as possible via Github Issues.

Reviewed By: lena-kashtelyan

Differential Revision: D64781681

fbshipit-source-id: 74f985797818bd02fbb417e6f8496bf7c109f906
  • Loading branch information
mpolson64 authored and facebook-github-bot committed Oct 28, 2024
1 parent 418fa6c commit 41da8f6
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions ax/storage/sqa_store/encoder.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,11 @@ class Encoder:
"""

def __init__(self, config: SQAConfig) -> None:
logger.error(
"ATTENTION: The Ax team is considering deprecating SQLAlchemy storage. "
"If you are currently using SQLAlchemy storage, please reach out to us "
"via GitHub Issues here: https://github.com/facebook/Ax/issues/2975"
)
self.config = config

@classmethod
Expand Down

0 comments on commit 41da8f6

Please sign in to comment.