[GitHub] [airflow] feluelle commented on issue #5994: [AIRFLOW-5395] Fix unloading with column names containing a dot in RedshiftToS3Operator

2019-10-11 Thread GitBox
feluelle commented on issue #5994: [AIRFLOW-5395] Fix unloading with column names containing a dot in RedshiftToS3Operator URL: https://github.com/apache/airflow/pull/5994#issuecomment-541087863 Closed in favor of #6309 This

[GitHub] [airflow] feluelle commented on issue #5994: [AIRFLOW-5395] Fix unloading with column names containing a dot in RedshiftToS3Operator

2019-10-04 Thread GitBox
feluelle commented on issue #5994: [AIRFLOW-5395] Fix unloading with column names containing a dot in RedshiftToS3Operator URL: https://github.com/apache/airflow/pull/5994#issuecomment-538339594 > @feluelle Thought if you want to "take-over" the other PR that might be better - open a new

[GitHub] [airflow] feluelle commented on issue #5994: [AIRFLOW-5395] Fix unloading with column names containing a dot in RedshiftToS3Operator

2019-09-29 Thread GitBox
feluelle commented on issue #5994: [AIRFLOW-5395] Fix unloading with column names containing a dot in RedshiftToS3Operator URL: https://github.com/apache/airflow/pull/5994#issuecomment-536299589 @OmerJog that's true, but if the other one gets merged this change is superfluous. I

[GitHub] [airflow] feluelle commented on issue #5994: [AIRFLOW-5395] Fix unloading with column names containing a dot in RedshiftToS3Operator

2019-09-03 Thread GitBox
feluelle commented on issue #5994: [AIRFLOW-5395] Fix unloading with column names containing a dot in RedshiftToS3Operator URL: https://github.com/apache/airflow/pull/5994#issuecomment-527566459 I just saw there is a PR #4610 which adds the new `HEADER` argument and apparently the author