ferruzzi merged PR #38850:
URL: https://github.com/apache/airflow/pull/38850
--
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:
ferruzzi commented on PR #38850:
URL: https://github.com/apache/airflow/pull/38850#issuecomment-2048444921
Gah, static check is just a merge issue, imported the sensor twice. Easy
fix.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
ferruzzi commented on PR #38850:
URL: https://github.com/apache/airflow/pull/38850#issuecomment-2048127209
Dependency merged; I'll get this rebased on top of that one and merge it
this afternoon.
--
This is an automated message from the Apache Git Service.
To respond to the message,
ferruzzi commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1558544946
##
airflow/providers/amazon/aws/operators/bedrock.py:
##
@@ -25,7 +25,10 @@
from airflow.exceptions import AirflowException
Review Comment:
I had to refactor
ferruzzi commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1558543614
##
tests/system/providers/amazon/aws/example_bedrock.py:
##
@@ -127,6 +137,7 @@ def delete_custom_model(model_name: str):
ferruzzi commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1558148127
##
tests/system/providers/amazon/aws/example_bedrock.py:
##
@@ -127,6 +137,7 @@ def delete_custom_model(model_name: str):
o-nikolas commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1558137372
##
tests/system/providers/amazon/aws/example_bedrock.py:
##
@@ -127,6 +137,7 @@ def delete_custom_model(model_name: str):
ferruzzi commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1558135437
##
airflow/providers/amazon/aws/sensors/bedrock.py:
##
@@ -17,36 +17,89 @@
# under the License.
from __future__ import annotations
+import abc
from typing
ferruzzi commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1558108644
##
airflow/providers/amazon/aws/operators/bedrock.py:
##
@@ -25,7 +25,10 @@
from airflow.exceptions import AirflowException
Review Comment:
dependency cleared.
vincbeck commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1557596522
##
airflow/providers/amazon/aws/sensors/bedrock.py:
##
@@ -17,36 +17,89 @@
# under the License.
from __future__ import annotations
+import abc
from typing
ferruzzi commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1556503642
##
airflow/providers/amazon/aws/operators/bedrock.py:
##
@@ -25,7 +25,10 @@
from airflow.exceptions import AirflowException
Review Comment:
This comment is
ferruzzi commented on code in PR #38850:
URL: https://github.com/apache/airflow/pull/38850#discussion_r1556503642
##
airflow/providers/amazon/aws/operators/bedrock.py:
##
@@ -25,7 +25,10 @@
from airflow.exceptions import AirflowException
Review Comment:
This comment is
ferruzzi opened a new pull request, #38850:
URL: https://github.com/apache/airflow/pull/38850
Adds support for provisioning model throughput on Amazon Bedrock including:
operators, sensors, waiters, and triggerer, as well as unit tests and system
tests for all of the above, and doc
13 matches
Mail list logo