Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-11-03 Thread via GitHub
pankajkoti closed pull request #34891: Enable pgvector support for Postgres provider URL: https://github.com/apache/airflow/pull/34891 -- 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 co

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-11-03 Thread via GitHub
pankajkoti commented on PR #34891: URL: https://github.com/apache/airflow/pull/34891#issuecomment-179217 Closing this PR in lieu of a separate provider PR: https://github.com/apache/airflow/pull/35399 -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-19 Thread via GitHub
Taragolis commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1365275358 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -320,6 +320,32 @@ def _generate_insert_sql( return sql +def ingest_embedding(self, table: st

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-19 Thread via GitHub
mpgreg commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1365231569 ## airflow/providers/postgres/operators/postgres.py: ## @@ -80,3 +86,60 @@ def __init__( AirflowProviderDeprecationWarning, stacklevel=2,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-18 Thread via GitHub
Taragolis commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1364142683 ## airflow/providers/postgres/operators/postgres.py: ## @@ -80,3 +86,60 @@ def __init__( AirflowProviderDeprecationWarning, stacklevel=2,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-18 Thread via GitHub
jlaneve commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1363762415 ## airflow/providers/postgres/operators/postgres.py: ## @@ -80,3 +86,60 @@ def __init__( AirflowProviderDeprecationWarning, stacklevel=2,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-17 Thread via GitHub
uranusjr commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1363227895 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -320,6 +320,34 @@ def _generate_insert_sql( return sql +def ingest_embedding(self, table: str

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-17 Thread via GitHub
pankajkoti commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1363167734 ## airflow/providers/postgres/operators/postgres.py: ## @@ -80,3 +86,60 @@ def __init__( AirflowProviderDeprecationWarning, stacklevel=2,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-17 Thread via GitHub
Taragolis commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1362910782 ## airflow/providers/postgres/operators/postgres.py: ## @@ -80,3 +86,60 @@ def __init__( AirflowProviderDeprecationWarning, stacklevel=2,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-17 Thread via GitHub
pankajkoti commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1362805154 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -320,6 +320,34 @@ def _generate_insert_sql( return sql +def ingest_embedding(self, table: s

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-13 Thread via GitHub
Taragolis commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1358562950 ## airflow/providers/postgres/operators/postgres.py: ## @@ -82,3 +87,61 @@ def __init__( AirflowProviderDeprecationWarning, stacklevel=2,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-12 Thread via GitHub
uranusjr commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1357810354 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -320,6 +320,36 @@ def _generate_insert_sql( return sql +def ingest_embedding( +self,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-12 Thread via GitHub
sunank200 commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1357077212 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -320,6 +320,29 @@ def _generate_insert_sql( return sql +def ingest_embedding( +self,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-12 Thread via GitHub
Taragolis commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1356852725 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -320,6 +320,29 @@ def _generate_insert_sql( return sql +def ingest_embedding( +self,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-12 Thread via GitHub
kaxil commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1356845699 ## airflow/providers/postgres/operators/postgres.py: ## @@ -82,3 +85,42 @@ def __init__( AirflowProviderDeprecationWarning, stacklevel=2,

Re: [PR] Enable pgvector support for Postgres provider [airflow]

2023-10-12 Thread via GitHub
eladkal commented on code in PR #34891: URL: https://github.com/apache/airflow/pull/34891#discussion_r1356844234 ## airflow/providers/postgres/provider.yaml: ## @@ -53,6 +53,7 @@ dependencies: - apache-airflow>=2.4.0 - apache-airflow-providers-common-sql>=1.3.1 - psycop

[PR] Enable pgvector support for Postgres provider [airflow]

2023-10-12 Thread via GitHub
sunank200 opened a new pull request, #34891: URL: https://github.com/apache/airflow/pull/34891 This PR is part of our larger effort to add first-class integrations to support LLMOps that was presented at the Airflow Summit. This PR specifically adds the [pgvector](https://github