pankajkoti commented on code in PR #39365: URL: https://github.com/apache/airflow/pull/39365#discussion_r1587974655
########## tests/system/providers/pinecone/example_pinecone_openai.py: ########## @@ -75,15 +74,15 @@ start_date=datetime(2023, 1, 1), catchup=False, ) as dag: - - @setup - @task - def create_index(): - from airflow.providers.pinecone.hooks.pinecone import PineconeHook - - hook = PineconeHook() - hook.create_index(index_name=index_name, dimension=1536) Review Comment: My opinion is that we should show how to create an index using the create_index hook method. Additionally, we should make sure that the operator is able to create index by providing required set of positional arguments and that keyword arguments have default values specified so that if the user does not provide those values the index is created seamlessly from the operator. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org