Repository: feast-dev/feast

Issue:
Missing documentation for abstract `pull_all_from_table_or_query` offline store method
## Expected Behavior 

Since https://github.com/feast-dev/feast/pull/2197, offline store method `pull_all_from_table_or_query` must be overridden by custom offline stores. This is currently not documented.

Expectations: 
- [Docstring](https://github.com/feast-dev/feast/blob/b35e1e84720523cef70cba6d6306af8f193b469f/sdk/python/feast/infra/offline_stores/offline_store.py#L203) of `pull_all_from_table_or_query` contains a meaningful description of the method

- [Web doc](https://docs.feast.dev/how-to-guides/adding-a-new-offline-store) mentions that `pull_all_from_table_or_query` must be overriden.

## Current Behavior
No documentation for `pull_all_from_table_or_query`.




Discussion:

Resolving patch:
diff --git a/sdk/python/feast/infra/offline_stores/offline_store.py b/sdk/python/feast/infra/offline_stores/offline_store.py
--- a/sdk/python/feast/infra/offline_stores/offline_store.py
+++ b/sdk/python/feast/infra/offline_stores/offline_store.py
@@ -179,9 +179,24 @@ def pull_latest_from_table_or_query(
         end_date: datetime,
     ) -> RetrievalJob:
         """
+        This method pulls data from the offline store, and the FeatureStore class is used to write
+        this data into the online store. This method is invoked when running materialization (using
+        the `feast materialize` or `feast materialize-incremental` commands, or the corresponding
+        FeatureStore.materialize() method. This method pulls data from the offline store, and the FeatureStore
+        class is used to write this data into the online store.
+
         Note that join_key_columns, feature_name_columns, event_timestamp_column, and created_timestamp_column
         have all already been mapped to column names of the source table and those column names are the values passed
         into this function.
+
+        Args:
+            config: Repo configuration object
+            data_source: Data source to pull all of the columns from
+            join_key_columns: Columns of the join keys
+            feature_name_columns: Columns of the feature names needed
+            event_timestamp_column: Timestamp column
+            start_date: Starting date of query
+            end_date: Ending date of query
         """
         pass
 
@@ -210,8 +225,19 @@ def pull_all_from_table_or_query(
         end_date: datetime,
     ) -> RetrievalJob:
         """
+        Returns a Retrieval Job for all join key columns, feature name columns, and the event timestamp columns that occur between the start_date and end_date.
+
         Note that join_key_columns, feature_name_columns, event_timestamp_column, and created_timestamp_column
         have all already been mapped to column names of the source table and those column names are the values passed
         into this function.
+
+        Args:
+            config: Repo configuration object
+            data_source: Data source to pull all of the columns from
+            join_key_columns: Columns of the join keys
+            feature_name_columns: Columns of the feature names needed
+            event_timestamp_column: Timestamp column
+            start_date: Starting date of query
+            end_date: Ending date of query
         """
         pass


Test patch: