zhongjiajie commented on issue #4887: [AIRFLOW-4055] Add AWS SQS Sensor URL: https://github.com/apache/airflow/pull/4887#issuecomment-475936736 > BTW, In this sensors, We push some data to XCom, So should we change the sensors name and add some word like `subscribe` or `push`, just like https://github.com/apache/airflow/blob/afa06aefa5e0d70836bd11c0116ef3c1d2db4fc3/tests/contrib/sensors/test_redis_pub_sub_sensor.py > > I have this idea due to most of are check if in some specific state but this is not. WDYT? @mans2singh I not familiar with with AWS, but I think sensor is check untill specific state exists. But this sensor check and push some information to Xcom, So I come out with this idea. Maybe we should left that to committer to discuss.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
