Bisk1 commented on PR #33746:
URL: https://github.com/apache/airflow/pull/33746#issuecomment-1704641375
Sorry, I missed that message somehow - thanks for fixing it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
uranusjr opened a new pull request, #34063:
URL: https://github.com/apache/airflow/pull/34063
This should restore code compatibility to 2.6 and prior. Note that the
constants would not appear in documentation since they are deprecated and not
use in Airflow core anyway anymore.
See
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-main by this push:
new 8bcbee54a5 Updating
boring-cyborg[bot] commented on issue #34062:
URL: https://github.com/apache/airflow/issues/34062#issuecomment-1704555684
Thanks for opening your first issue here! Be sure to follow the issue
template! If you are willing to raise PR to address this issue please do so, no
need to wait for
immiao opened a new issue, #34062:
URL: https://github.com/apache/airflow/issues/34062
### What do you see as an issue?
In latest ERD schema -
https://airflow.apache.org/docs/apache-airflow/stable/database-erd-ref.html
`dag_run` to `task_instance` is showing 1 to 1
This is an automated email from the ASF dual-hosted git repository.
uranusjr pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new c44e5b5ccc Add state utils to Public Airflow
uranusjr merged PR #34059:
URL: https://github.com/apache/airflow/pull/34059
--
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:
adam133 commented on code in PR #32903:
URL: https://github.com/apache/airflow/pull/32903#discussion_r1314359427
##
airflow/providers/databricks/hooks/databricks.py:
##
@@ -215,6 +216,67 @@ def find_job_id_by_name(self, job_name: str) -> int | None:
else:
zeotuan commented on issue #31745:
URL: https://github.com/apache/airflow/issues/31745#issuecomment-1704459295
I would like to pick up this feature.
--
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
github-actions[bot] closed issue #32824: Configuration of S3 based dags_folder
URL: https://github.com/apache/airflow/issues/32824
--
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
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 05a0542b5a Refactor unneeded 'continue' jumps in
potiuk merged PR #33845:
URL: https://github.com/apache/airflow/pull/33845
--
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:
kaxil commented on code in PR #34042:
URL: https://github.com/apache/airflow/pull/34042#discussion_r1314334593
##
airflow/serialization/serialized_objects.py:
##
@@ -438,7 +438,7 @@ def serialize(
json_pod = PodGenerator.serialize_pod(var)
return
kaxil commented on code in PR #34042:
URL: https://github.com/apache/airflow/pull/34042#discussion_r1314334386
##
airflow/serialization/serialized_objects.py:
##
@@ -438,7 +438,7 @@ def serialize(
json_pod = PodGenerator.serialize_pod(var)
return
kaxil commented on code in PR #34042:
URL: https://github.com/apache/airflow/pull/34042#discussion_r1314333805
##
airflow/serialization/serialized_objects.py:
##
@@ -438,7 +438,7 @@ def serialize(
json_pod = PodGenerator.serialize_pod(var)
return
SbtrktCtrl commented on issue #34058:
URL: https://github.com/apache/airflow/issues/34058#issuecomment-1704434167
Legends! thank you.
--
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
potiuk commented on PR #34014:
URL: https://github.com/apache/airflow/pull/34014#issuecomment-1704433673
I think we should also adda `python_requires='>3.8.0'` in `setup(` in
setup.py in order to replace the `requires-python`
--
This is an automated message from the Apache Git Service.
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new bf2d411a2a Add decorator for suppress optional
potiuk merged PR #34034:
URL: https://github.com/apache/airflow/pull/34034
--
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:
potiuk merged PR #33976:
URL: https://github.com/apache/airflow/pull/33976
--
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:
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 4f20b0f442 Move the try outside the loop when this
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 8918b435be Move the try outside the loop when this
potiuk merged PR #33975:
URL: https://github.com/apache/airflow/pull/33975
--
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:
potiuk commented on issue #34013:
URL: https://github.com/apache/airflow/issues/34013#issuecomment-1704411889
For me it looks like something indeed takes a lot of memory for the workers
but what it would be - hard to say.
Is it possible to run your workers with memray
potiuk commented on PR #34059:
URL: https://github.com/apache/airflow/pull/34059#issuecomment-1704410218
Right :)
--
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
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new d757f6a3af Fix BigQueryValueCheckOperator
potiuk merged PR #34018:
URL: https://github.com/apache/airflow/pull/34018
--
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:
potiuk closed issue #34010: `BigQueryValueCheckOperator` doesn't respect
`pass_value` in deferrable mode
URL: https://github.com/apache/airflow/issues/34010
--
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
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-main by this push:
new 2200e749b8 Updating
Taragolis commented on code in PR #34002:
URL: https://github.com/apache/airflow/pull/34002#discussion_r1314315301
##
tests/system/providers/microsoft/azure/example_azure_container_instances.py:
##
@@ -52,7 +55,39 @@
task_id="start_container",
)
+t2 =
potiuk closed issue #29242: Auto-refresh task logs in UI
URL: https://github.com/apache/airflow/issues/29242
--
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,
potiuk commented on PR #34002:
URL: https://github.com/apache/airflow/pull/34002#issuecomment-1704398249
> During my work on #33825 and #33822, I discovered that there are no system
tests for AzureContainerVolumeHook and AzureContainerRegistryHook. Is there any
setup I need to do for these
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 6ef80e8be1 Combine similar if logics in tests
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new fa5e54c4c5 Combine similar if logics in providers
potiuk merged PR #33986:
URL: https://github.com/apache/airflow/pull/33986
--
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:
potiuk merged PR #33987:
URL: https://github.com/apache/airflow/pull/33987
--
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:
potiuk commented on PR #33746:
URL: https://github.com/apache/airflow/pull/33746#issuecomment-1704386688
Added PR https://github.com/apache/airflow/pull/34059
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
potiuk opened a new pull request, #34059:
URL: https://github.com/apache/airflow/pull/34059
The state enums are likely to be used externally and they should not be
changed in backwards-incompatible way.
Discussed in #33746
---
**^ Add meaningful
shahar1 commented on issue #32650:
URL: https://github.com/apache/airflow/issues/32650#issuecomment-1704386375
@A-Costa Hey, if you're still working on it - I'd be happy to assist
(disclosure: I introduced the `match_glob` to the GCS hook).
As Elad suggested - I think that it makes sense
pierrejeambrun opened a new issue, #34058:
URL: https://github.com/apache/airflow/issues/34058
**Reproduce on main:**
- Run a DAG
- Rename the DAG
- Wait for dag file processor to remove the deleted DAG and add the new one.
(Old Dag should not appear on the home page anymore)
-
eumiro opened a new pull request, #34057:
URL: https://github.com/apache/airflow/pull/34057
(no comment)
--
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,
pierrejeambrun opened a new pull request, #34056:
URL: https://github.com/apache/airflow/pull/34056
We had questions about this in the slack, instance here:
https://apache-airflow.slack.com/archives/CSS36QQS1/p1693501503281489
This is just a clarification of the spec as to why only
eumiro opened a new pull request, #34055:
URL: https://github.com/apache/airflow/pull/34055
(no comment)
--
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,
eumiro opened a new pull request, #34054:
URL: https://github.com/apache/airflow/pull/34054
(no comment)
--
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,
eumiro opened a new pull request, #34053:
URL: https://github.com/apache/airflow/pull/34053
(no comment)
--
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,
potiuk commented on PR #33934:
URL: https://github.com/apache/airflow/pull/33934#issuecomment-1704376350
LGTM. I think using itertools for those is indeed over-the-top. I am fine
witth the current version :)
--
This is an automated message from the Apache Git Service.
To respond to the
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new c82806641f Replace single quotes by double quotes
potiuk merged PR #33864:
URL: https://github.com/apache/airflow/pull/33864
--
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:
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-main by this push:
new 92d81199c1 Updating
potiuk commented on PR #33898:
URL: https://github.com/apache/airflow/pull/33898#issuecomment-1704364401
Mypy does not like optional groups()
--
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
eumiro opened a new pull request, #34052:
URL: https://github.com/apache/airflow/pull/34052
(no comment)
--
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,
potiuk commented on PR #33902:
URL: https://github.com/apache/airflow/pull/33902#issuecomment-1704358675
Closing unless the solution that alrready exists in Airflow for that very
purpose cannot be used, then we can reconsider (but it's not generic enough
anyway).
--
This is an automated
potiuk commented on PR #33902:
URL: https://github.com/apache/airflow/pull/33902#issuecomment-1704358422
This is quite certainly wrong approach.
You do not need to do it in such complex way. You should use `_CMD` version
of the variables for that.
You can have
potiuk closed pull request #33902: allow optional password replacement
URL: https://github.com/apache/airflow/pull/33902
--
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
eumiro commented on code in PR #33846:
URL: https://github.com/apache/airflow/pull/33846#discussion_r1314290974
##
airflow/jobs/backfill_job_runner.py:
##
@@ -656,8 +656,6 @@ def _per_task_process(key, ti: TaskInstance, session):
potiuk commented on code in PR #33837:
URL: https://github.com/apache/airflow/pull/33837#discussion_r1314289102
##
airflow/models/taskinstance.py:
##
@@ -2966,12 +2966,9 @@ def as_dict(self):
stacklevel=2,
)
new_dict = dict(self.__dict__)
-
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-2-7
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-2-7 by this push:
new 6dc563d776 Updating
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 9ed992ac0f Refactor unneeded 'continue' jumps in
potiuk merged PR #33834:
URL: https://github.com/apache/airflow/pull/33834
--
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:
potiuk merged PR #33848:
URL: https://github.com/apache/airflow/pull/33848
--
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:
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new ff1a5f2a99 Refactor unneeded 'continue' jumps in
potiuk commented on PR #33848:
URL: https://github.com/apache/airflow/pull/33848#issuecomment-1704351067
Static checks.
--
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
ephraimbuddy merged PR #34029:
URL: https://github.com/apache/airflow/pull/34029
--
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:
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new dd39933f62 Fix link for running Airflow
ephraimbuddy commented on PR #33903:
URL: https://github.com/apache/airflow/pull/33903#issuecomment-1704343282
Cherrypicked!
--
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.
potiuk commented on PR #33848:
URL: https://github.com/apache/airflow/pull/33848#issuecomment-1704343100
Static checks.
--
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
potiuk commented on PR #33867:
URL: https://github.com/apache/airflow/pull/33867#issuecomment-1704342784
> Maybe we could add a note somewhere (or do we already?) explaining that
from __future__ import annotations is needed for some of the examples.
I don't think we have a good place
potiuk closed pull request #33867: Update taskflow.rst, fix Multiple outputs
inference example
URL: https://github.com/apache/airflow/pull/33867
--
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
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch v2-7-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 590412ba589fec20edde698057190f725770a29f
Author: Miroslav Šedivý <6774676+eum...@users.noreply.github.com>
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a change to branch v2-7-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
omit e571741629 Limit celery by excluding 5.3.2 and 5.3.3 (#34031)
omit 6cb5ef1bff Update
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch v2-7-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit b5dbf97317c4ea0dde6a2b006d34b694c95d2f9a
Author: Daniel Standish
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch v2-7-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 891fae5025e44833afc0fc6f7a5ff34515753e26
Author: Ephraim Anierobi
AuthorDate: Mon Aug 28 16:07:12 2023
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch v2-7-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit d404d3f84a5f6cdc204c758f76514e2829525dd0
Author: Jarek Potiuk
AuthorDate: Sat Sep 2 15:19:21 2023 +0200
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 562357528d Refactor unneeded 'continue' jumps in
potiuk commented on PR #33865:
URL: https://github.com/apache/airflow/pull/33865#issuecomment-1704341787
> @potiuk do we cherry-pick dev and tests? is it ok to keep all these
changes in the same PR or should I split them in smaller ones?
We do cherry-pick dev and tests except
potiuk merged PR #33839:
URL: https://github.com/apache/airflow/pull/33839
--
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:
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 5833320291 Fix issue with unit tests in
potiuk merged PR #34037:
URL: https://github.com/apache/airflow/pull/34037
--
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:
potiuk commented on PR #34037:
URL: https://github.com/apache/airflow/pull/34037#issuecomment-1704339899
fantastic :).
--
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
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new e75cecab72 Ensure that tasks wait for
ephraimbuddy merged PR #33903:
URL: https://github.com/apache/airflow/pull/33903
--
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:
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-main by this push:
new 04ae30fb86 Updating
Bisk1 commented on code in PR #34037:
URL: https://github.com/apache/airflow/pull/34037#discussion_r1314275322
##
airflow/config_templates/unit_tests.cfg:
##
@@ -59,6 +59,8 @@ unit_test_mode = True
killed_task_cleanup_time = 5
# We only allow our own classes to be
hussein-awala commented on PR #33903:
URL: https://github.com/apache/airflow/pull/33903#issuecomment-1704326323
I understand that `+602 −236 ` is scary for a patch version, but most of the
changed code is documentation and tests, and for the changes in
`trigger_rule_dep`, @dstandish has
patgarz opened a new pull request, #34051:
URL: https://github.com/apache/airflow/pull/34051
CC: @vincbeck as owner of AIP-56
apache/airflow#32217 implemented the base and FAB user management
interfaces. As a part of that PR, `get_user_name` was set to be `FirstName
rajrao commented on code in PR #34016:
URL: https://github.com/apache/airflow/pull/34016#discussion_r1314272113
##
docs/apache-airflow/core-concepts/xcoms.rst:
##
@@ -25,9 +25,23 @@ XComs (short for "cross-communications") are a mechanism
that let :doc:`tasks` t
An XCom is
potiuk closed issue #34044: Airflow should support different types of dataset
event consumer DAGs
URL: https://github.com/apache/airflow/issues/34044
--
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
potiuk commented on issue #34044:
URL: https://github.com/apache/airflow/issues/34044#issuecomment-1704324263
I think such a discussion should happen in the devlist not in GitHub issue.
I believe very soon we are going to have some proposal (or few proposals) on
the devlist that will be
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 6f1f624f04 Remove azure-storage-common from
potiuk merged PR #34038:
URL: https://github.com/apache/airflow/pull/34038
--
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:
potiuk commented on PR #34038:
URL: https://github.com/apache/airflow/pull/34038#issuecomment-1704320682
Less dependencies is better :). And seems that all tests pass :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 613ae4b78e Make AWS SecretsManager tests
potiuk merged PR #34046:
URL: https://github.com/apache/airflow/pull/34046
--
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:
pankajastro commented on PR #34038:
URL: https://github.com/apache/airflow/pull/34038#issuecomment-1704319936
> Why do we have it if we don't use it?
> Did we remove some functionality and forgot to remove the library?
yes I believe should have been removed in my previous PR. Look
jaegwonseo commented on code in PR #33796:
URL: https://github.com/apache/airflow/pull/33796#discussion_r1314256500
##
airflow/providers/apache/druid/hooks/druid.py:
##
@@ -149,6 +154,20 @@ def submit_indexing_job(
self.log.info("Successful index")
+def
jaegwonseo commented on code in PR #33796:
URL: https://github.com/apache/airflow/pull/33796#discussion_r1314253423
##
airflow/providers/apache/druid/CHANGELOG.rst:
##
@@ -27,6 +27,19 @@
Changelog
-
+4.0.0
+.
+
+Features
+
+
+* ``Add druid ingestion
Bisk1 commented on PR #34027:
URL: https://github.com/apache/airflow/pull/34027#issuecomment-1704264908
> Looks good but the renaming should be in a separate PR if it's actually
needed
@ephraimbuddy
Ok I removed it and moved to separate PR
Bisk1 opened a new pull request, #34049:
URL: https://github.com/apache/airflow/pull/34049
Follow-up to discussion in
https://github.com/apache/airflow/pull/34027#pullrequestreview-1608444868
Some variables used when invoking calculate_dagrun_date_fields were renamed
to make it
NJ-Tokki opened a new issue, #34048:
URL: https://github.com/apache/airflow/issues/34048
### Description
Feature
Kubernetes offers a [list of recommended
labels](https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/#labels)
for grouping resource
potiuk commented on code in PR #34037:
URL: https://github.com/apache/airflow/pull/34037#discussion_r1314236718
##
airflow/config_templates/unit_tests.cfg:
##
@@ -59,6 +59,8 @@ unit_test_mode = True
killed_task_cleanup_time = 5
# We only allow our own classes to be
1 - 100 of 124 matches
Mail list logo