fix: Remove ci extra from the feature transformation server dockerfile (

#2618) * fix: Remove ci extra from the feature transformation server dockerfile and pin numpy for py3.7 support Signed-off-by: Achal Shah <achals@gmail.com> * remove debug Signed-off-by: Achal Shah <achals@gmail.com> * pin requirements Signed-off-by: Achal Shah <achals@gmail.com>
feast-dev · Apr 27, 2022 · 25613b4 · 25613b4
1 parent 5200fde
commit 25613b4
Show file tree

Hide file tree

Showing 10 changed files with 47 additions and 10 deletions.
diff --git a/sdk/python/feast/infra/transformation_servers/Dockerfile b/sdk/python/feast/infra/transformation_servers/Dockerfile
@@ -9,7 +9,7 @@ COPY protos protos
 COPY README.md README.md
 
 # Install dependencies
-RUN pip3 install -e 'sdk/python[ci]'
+RUN pip3 install -e 'sdk/python'
 
 # Start feature transformation server
 CMD [ "python", "app.py" ]
diff --git a/sdk/python/requirements/py3.10-ci-requirements.txt b/sdk/python/requirements/py3.10-ci-requirements.txt
@@ -255,6 +255,8 @@ grpcio-tools==1.44.0
     # via feast (setup.py)
 h11==0.13.0
     # via uvicorn
+happybase==1.2.0
+    # via feast (setup.py)
 hiredis==2.0.0
     # via feast (setup.py)
 httplib2==0.20.4
@@ -356,9 +358,10 @@ nbformat==5.3.0
     # via great-expectations
 nodeenv==1.6.0
     # via pre-commit
-numpy==1.22.3
+numpy==1.21.6
     # via
     #   altair
+    #   feast (setup.py)
     #   great-expectations
     #   pandas
     #   pandavro
@@ -407,6 +410,8 @@ platformdirs==2.5.2
     # via virtualenv
 pluggy==1.0.0
     # via pytest
+ply==3.11
+    # via thriftpy2
 portalocker==2.4.0
     # via msal-extensions
 pre-commit==2.18.1
@@ -596,6 +601,7 @@ six==1.16.0
     #   google-cloud-core
     #   google-resumable-media
     #   grpcio
+    #   happybase
     #   mock
     #   msrestazure
     #   pandavro
@@ -640,6 +646,8 @@ termcolor==1.1.0
     # via great-expectations
 testcontainers==3.5.3
     # via feast (setup.py)
+thriftpy2==0.4.14
+    # via happybase
 toml==0.10.2
     # via
     #   black

diff --git a/sdk/python/requirements/py3.10-requirements.txt b/sdk/python/requirements/py3.10-requirements.txt
@@ -73,8 +73,9 @@ markupsafe==2.1.1
     # via jinja2
 mmh3==3.0.0
     # via feast (setup.py)
-numpy==1.22.3
+numpy==1.21.6
     # via
+    #   feast (setup.py)
     #   pandas
     #   pandavro
     #   pyarrow

diff --git a/sdk/python/requirements/py3.7-ci-requirements.txt b/sdk/python/requirements/py3.7-ci-requirements.txt
@@ -257,6 +257,8 @@ grpcio-tools==1.44.0
     # via feast (setup.py)
 h11==0.13.0
     # via uvicorn
+happybase==1.2.0
+    # via feast (setup.py)
 hiredis==2.0.0
     # via feast (setup.py)
 httplib2==0.20.4
@@ -374,6 +376,7 @@ nodeenv==1.6.0
 numpy==1.21.6
     # via
     #   altair
+    #   feast (setup.py)
     #   great-expectations
     #   pandas
     #   pandavro
@@ -422,6 +425,8 @@ platformdirs==2.5.2
     # via virtualenv
 pluggy==1.0.0
     # via pytest
+ply==3.11
+    # via thriftpy2
 portalocker==2.4.0
     # via msal-extensions
 pre-commit==2.18.1
@@ -611,6 +616,7 @@ six==1.16.0
     #   google-cloud-core
     #   google-resumable-media
     #   grpcio
+    #   happybase
     #   mock
     #   msrestazure
     #   pandavro
@@ -653,6 +659,8 @@ termcolor==1.1.0
     # via great-expectations
 testcontainers==3.5.3
     # via feast (setup.py)
+thriftpy2==0.4.14
+    # via happybase
 toml==0.10.2
     # via
     #   black

diff --git a/sdk/python/requirements/py3.7-requirements.txt b/sdk/python/requirements/py3.7-requirements.txt
@@ -81,6 +81,7 @@ mmh3==3.0.0
     # via feast (setup.py)
 numpy==1.21.6
     # via
+    #   feast (setup.py)
     #   pandas
     #   pandavro
     #   pyarrow

diff --git a/sdk/python/requirements/py3.8-ci-requirements.txt b/sdk/python/requirements/py3.8-ci-requirements.txt
@@ -259,6 +259,8 @@ grpcio-tools==1.44.0
     # via feast (setup.py)
 h11==0.13.0
     # via uvicorn
+happybase==1.2.0
+    # via feast (setup.py)
 hiredis==2.0.0
     # via feast (setup.py)
 httplib2==0.20.4
@@ -362,9 +364,10 @@ nbformat==5.3.0
     # via great-expectations
 nodeenv==1.6.0
     # via pre-commit
-numpy==1.22.3
+numpy==1.21.6
     # via
     #   altair
+    #   feast (setup.py)
     #   great-expectations
     #   pandas
     #   pandavro
@@ -413,6 +416,8 @@ platformdirs==2.5.2
     # via virtualenv
 pluggy==1.0.0
     # via pytest
+ply==3.11
+    # via thriftpy2
 portalocker==2.4.0
     # via msal-extensions
 pre-commit==2.18.1
@@ -604,6 +609,7 @@ six==1.16.0
     #   google-cloud-core
     #   google-resumable-media
     #   grpcio
+    #   happybase
     #   mock
     #   msrestazure
     #   pandavro
@@ -648,6 +654,8 @@ termcolor==1.1.0
     # via great-expectations
 testcontainers==3.5.3
     # via feast (setup.py)
+thriftpy2==0.4.14
+    # via happybase
 toml==0.10.2
     # via
     #   black

diff --git a/sdk/python/requirements/py3.8-requirements.txt b/sdk/python/requirements/py3.8-requirements.txt
@@ -75,8 +75,9 @@ markupsafe==2.1.1
     # via jinja2
 mmh3==3.0.0
     # via feast (setup.py)
-numpy==1.22.3
+numpy==1.21.6
     # via
+    #   feast (setup.py)
     #   pandas
     #   pandavro
     #   pyarrow

diff --git a/sdk/python/requirements/py3.9-ci-requirements.txt b/sdk/python/requirements/py3.9-ci-requirements.txt
@@ -255,6 +255,8 @@ grpcio-tools==1.44.0
     # via feast (setup.py)
 h11==0.13.0
     # via uvicorn
+happybase==1.2.0
+    # via feast (setup.py)
 hiredis==2.0.0
     # via feast (setup.py)
 httplib2==0.20.4
@@ -356,9 +358,10 @@ nbformat==5.3.0
     # via great-expectations
 nodeenv==1.6.0
     # via pre-commit
-numpy==1.22.3
+numpy==1.21.6
     # via
     #   altair
+    #   feast (setup.py)
     #   great-expectations
     #   pandas
     #   pandavro
@@ -407,6 +410,8 @@ platformdirs==2.5.2
     # via virtualenv
 pluggy==1.0.0
     # via pytest
+ply==3.11
+    # via thriftpy2
 portalocker==2.4.0
     # via msal-extensions
 pre-commit==2.18.1
@@ -579,10 +584,10 @@ responses==0.20.0
     # via moto
 rsa==4.8
     # via google-auth
-ruamel.yaml==0.17.17
+ruamel-yaml==0.17.17
     # via great-expectations
-ruamel.yaml.clib==0.2.6
-    # via ruamel.yaml
+ruamel-yaml-clib==0.2.6
+    # via ruamel-yaml
 s3transfer==0.5.2
     # via boto3
 scipy==1.8.0
@@ -598,6 +603,7 @@ six==1.16.0
     #   google-cloud-core
     #   google-resumable-media
     #   grpcio
+    #   happybase
     #   mock
     #   msrestazure
     #   pandavro
@@ -642,6 +648,8 @@ termcolor==1.1.0
     # via great-expectations
 testcontainers==3.5.3
     # via feast (setup.py)
+thriftpy2==0.4.14
+    # via happybase
 toml==0.10.2
     # via
     #   black

diff --git a/sdk/python/requirements/py3.9-requirements.txt b/sdk/python/requirements/py3.9-requirements.txt
@@ -73,8 +73,9 @@ markupsafe==2.1.1
     # via jinja2
 mmh3==3.0.0
     # via feast (setup.py)
-numpy==1.22.3
+numpy==1.21.6
     # via
+    #   feast (setup.py)
     #   pandas
     #   pandavro
     #   pyarrow

diff --git a/sdk/python/setup.py b/sdk/python/setup.py
@@ -57,6 +57,7 @@
     "Jinja2>=2.0.0",
     "jsonschema",
     "mmh3",
+    "numpy<1.22",  # 1.22 drops support for python 3.7.
     "pandas>=1.0.0",
     "pandavro==1.5.*",
     "protobuf>=3.10,<3.20",