incorporating code review comments.

Signed-off-by: Lokesh Rangineni <[email protected]>

Makefile

@@ -377,9 +377,6 @@ kill-trino-locally:
 install-protoc-dependencies:
 	pip install --ignore-installed protobuf==4.24.0 "grpcio-tools>=1.56.2,<2" mypy-protobuf==3.1.0
 
-install-feast-ci-locally:
-	pip install -e ".[ci]"
-
 # Docker
 
 build-docker: build-feature-server-python-aws-docker build-feature-transformation-server-docker build-feature-server-java-docker

sdk/python/feast/infra/online_stores/remote.py

@@ -55,7 +55,7 @@ def online_write_batch(
         ],
         progress: Optional[Callable[[int], Any]],
     ) -> None:
-        pass
+        raise NotImplementedError
 
     def online_read(
         self,
@@ -156,12 +156,12 @@ def update(
         entities_to_keep: Sequence[Entity],
         partial: bool,
     ):
-        pass
+        raise NotImplementedError
 
     def teardown(
         self,
         config: RepoConfig,
         tables: Sequence[FeatureView],
         entities: Sequence[Entity],
     ):
-        pass
+        raise NotImplementedError