From 41da8f660143bae0ebf106e706a2481b40d842c7 Mon Sep 17 00:00:00 2001 From: Miles Olson Date: Mon, 28 Oct 2024 16:14:38 -0700 Subject: [PATCH] Add potential SQL deprecation warning at Encoder instantiation time (#2974) Summary: Pull Request resolved: https://github.com/facebook/Ax/pull/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 --- ax/storage/sqa_store/encoder.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/ax/storage/sqa_store/encoder.py b/ax/storage/sqa_store/encoder.py index 4f7fe21d098..5230573e37a 100644 --- a/ax/storage/sqa_store/encoder.py +++ b/ax/storage/sqa_store/encoder.py @@ -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