Skip to content

Commit 8ff17b7

Browse files
committed
Temporary fix to resolve circular import issue
1 parent d02919a commit 8ff17b7

File tree

2 files changed

+21
-22
lines changed

2 files changed

+21
-22
lines changed

ads/feature_store/common/spark_session_singleton.py

Lines changed: 21 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,7 @@
99

1010
from ads.common.decorator.runtime_dependency import OptionalDependency
1111
import os
12-
1312
from ads.common.oci_client import OCIClientFactory
14-
from ads.feature_store.common.utils.utility import get_env_bool
1513

1614
try:
1715
from delta import configure_spark_with_delta_pip
@@ -33,6 +31,27 @@
3331
raise
3432

3533

34+
def get_env_bool(env_var: str, default: bool = False) -> bool:
35+
"""
36+
:param env_var: Environment variable name
37+
:param default: Default environment variable value
38+
:return: Value of the boolean env variable
39+
"""
40+
env_val = os.getenv(env_var)
41+
if env_val is None:
42+
env_val = default
43+
else:
44+
env_val = env_val.lower()
45+
if env_val == "true":
46+
env_val = True
47+
elif env_val == "false":
48+
env_val = False
49+
else:
50+
raise ValueError("For environment variable: {0} only string values T/true or F/false are allowed but: \
51+
{1} was provided.".format(env_var, env_val))
52+
return env_val
53+
54+
3655
def developer_enabled():
3756
return get_env_bool("DEVELOPER_MODE", False)
3857

ads/feature_store/common/utils/utility.py

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -63,26 +63,6 @@ def get_execution_engine_type(
6363
else ExecutionEngine.SPARK
6464
)
6565

66-
def get_env_bool(env_var: str, default: bool = False) -> bool:
67-
"""
68-
:param env_var: Environment variable name
69-
:param default: Default environment variable value
70-
:return: Value of the boolean env variable
71-
"""
72-
env_val = os.getenv(env_var)
73-
if env_val is None:
74-
env_val = default
75-
else:
76-
env_val = env_val.lower()
77-
if env_val == "true":
78-
env_val = True
79-
elif env_val == "false":
80-
env_val = False
81-
else:
82-
raise ValueError("For environment variable: {0} only string values T/true or F/false are allowed but: \
83-
{1} was provided.".format(env_var, env_val))
84-
return env_val
85-
8666
def get_metastore_id(feature_store_id: str):
8767
"""
8868
Retrieves the metastore ID for a given feature store ID.

0 commit comments

Comments
 (0)