Incorporating the code review comments from Francisco.

lokeshrangineni · lokeshrangineni · commit 6084fb916561 · 2024-12-18T09:58:06.000-05:00
Signed-off-by: lrangine &lt;19699092+lokeshrangineni@users.noreply.github.com&gt;
diff --git a/docs/how-to-guides/starting-feast-servers-tls-mode.md b/docs/how-to-guides/starting-feast-servers-tls-mode.md
@@ -192,5 +192,5 @@ INFO:     Uvicorn running on https://0.0.0.0:8888 (Press CTRL+C to quit)
 
 
 ## Adding public key to CA trust store and configuring the feast to use the trust store.
-You can pass the public key for SSL verification using `cert` parameter, however, it is sometimes a hassle to maintain individual certificate and pass the public certificate individually. 
-The alternate recommendation is to add the public certificate to CA trust store and set the path as environment variable `FEAST_CA_CERT_FILE_PATH`. Feast will refer the trust store path set as environment variable as `FEAST_CA_CERT_FILE_PATH`
+You can pass the public key for SSL verification using the `cert` parameter, however, it is sometimes difficult to maintain individual certificates and pass them individually.
+The alternative recommendation is to add the public certificate to CA trust store and set the path as an environment variable (e.g., `FEAST_CA_CERT_FILE_PATH`). Feast will use the trust store path in the  `FEAST_CA_CERT_FILE_PATH` environment variable.
diff --git a/sdk/python/feast/infra/registry/remote.py b/sdk/python/feast/infra/registry/remote.py
@@ -61,8 +61,9 @@ class RemoteRegistryConfig(RegistryConfig):
     If registry_type is 'remote', then this configuration is needed to connect to remote registry server in TLS mode. If the remote registry started in non-tls mode then this configuration is not needed."""
 
     is_tls: bool = False
-    """ bool: if you are planning to connect the registry server which started in TLS(SSL) mode then this should be true.
-    If you are planning to add the public certificate as part of the trust store instead of passing it as a `cert` parameters then setting this field to `true` is a mandatory.
+    """     bool: Set to `True` if you plan to connect to a registry server running in TLS (SSL) mode.
+    If you intend to add the public certificate to the trust store instead of passing it via the `cert` parameter, this field must be set to `True`.
+    If you are planning to add the public certificate as part of the trust store instead of passing it as a `cert` parameters then setting this field to `true` is mandatory.
     """
 
 
diff --git a/sdk/python/feast/ssl_ca_trust_store_setup.py b/sdk/python/feast/ssl_ca_trust_store_setup.py
@@ -7,7 +7,7 @@
 
 def configure_ca_trust_store_env_variables():
     """
-    configures the environment variable so that other libraries or servers refer the TLS ca file path.
+    configures the environment variable so that other libraries or servers refer to the TLS ca file path.
     :param ca_file_path:
     :return:
     """
