[GitHub] [airflow] codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default connection entry for PinotDbApiHook

2020-03-17 Thread GitBox
codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default 
connection entry for PinotDbApiHook
URL: https://github.com/apache/airflow/pull/6457#issuecomment-552073991
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=h1) 
Report
   > Merging 
[#6457](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=desc) into 
[master](https://codecov.io/gh/apache/airflow/commit/1fa7ac1b3978469ba3ec9b902b4f2b4890f8199d=desc)
 will **increase** coverage by `1.35%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/airflow/pull/6457/graphs/tree.svg?width=650=150=pr=WdLKlKHOAU)](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree)
   
   ```diff
   @@Coverage Diff @@
   ##   master#6457  +/-   ##
   ==
   + Coverage   85.34%   86.69%   +1.35% 
   ==
 Files 791  915 +124 
 Lines   4012444189+4065 
   ==
   + Hits3424338311+4068 
   + Misses   5881 5878   -3 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree) | 
Coverage Δ | |
   |---|---|---|
   | 
[airflow/utils/db.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy91dGlscy9kYi5weQ==)
 | `97.50% <100.00%> (-0.44%)` | :arrow_down: |
   | 
[airflow/operators/mysql\_to\_hive.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfdG9faGl2ZS5weQ==)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...irflow/providers/apache/cassandra/sensors/table.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvYXBhY2hlL2Nhc3NhbmRyYS9zZW5zb3JzL3RhYmxlLnB5)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...rflow/providers/apache/cassandra/sensors/record.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvYXBhY2hlL2Nhc3NhbmRyYS9zZW5zb3JzL3JlY29yZC5weQ==)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[airflow/operators/s3\_to\_hive\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvczNfdG9faGl2ZV9vcGVyYXRvci5weQ==)
 | `0.00% <0.00%> (-93.97%)` | :arrow_down: |
   | 
[airflow/contrib/hooks/grpc\_hook.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2dycGNfaG9vay5weQ==)
 | `0.00% <0.00%> (-91.94%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==)
 | `44.44% <0.00%> (-55.56%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==)
 | `52.94% <0.00%> (-47.06%)` | :arrow_down: |
   | 
[airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==)
 | `47.18% <0.00%> (-44.79%)` | :arrow_down: |
   | 
[...viders/cncf/kubernetes/operators/kubernetes\_pod.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvY25jZi9rdWJlcm5ldGVzL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZC5weQ==)
 | `69.69% <0.00%> (-27.96%)` | :arrow_down: |
   | ... and [1017 
more](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree-more) 
| |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=footer). 
Last update 
[1fa7ac1...2991e61](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [airflow] codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default connection entry for PinotDbApiHook

2020-03-17 Thread GitBox
codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default 
connection entry for PinotDbApiHook
URL: https://github.com/apache/airflow/pull/6457#issuecomment-552073991
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=h1) 
Report
   > Merging 
[#6457](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=desc) into 
[master](https://codecov.io/gh/apache/airflow/commit/1fa7ac1b3978469ba3ec9b902b4f2b4890f8199d=desc)
 will **increase** coverage by `1.35%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/airflow/pull/6457/graphs/tree.svg?width=650=150=pr=WdLKlKHOAU)](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree)
   
   ```diff
   @@Coverage Diff @@
   ##   master#6457  +/-   ##
   ==
   + Coverage   85.34%   86.69%   +1.35% 
   ==
 Files 791  915 +124 
 Lines   4012444189+4065 
   ==
   + Hits3424338311+4068 
   + Misses   5881 5878   -3 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree) | 
Coverage Δ | |
   |---|---|---|
   | 
[airflow/utils/db.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy91dGlscy9kYi5weQ==)
 | `97.50% <100.00%> (-0.44%)` | :arrow_down: |
   | 
[airflow/operators/mysql\_to\_hive.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfdG9faGl2ZS5weQ==)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...irflow/providers/apache/cassandra/sensors/table.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvYXBhY2hlL2Nhc3NhbmRyYS9zZW5zb3JzL3RhYmxlLnB5)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...rflow/providers/apache/cassandra/sensors/record.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvYXBhY2hlL2Nhc3NhbmRyYS9zZW5zb3JzL3JlY29yZC5weQ==)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[airflow/operators/s3\_to\_hive\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvczNfdG9faGl2ZV9vcGVyYXRvci5weQ==)
 | `0.00% <0.00%> (-93.97%)` | :arrow_down: |
   | 
[airflow/contrib/hooks/grpc\_hook.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2dycGNfaG9vay5weQ==)
 | `0.00% <0.00%> (-91.94%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==)
 | `44.44% <0.00%> (-55.56%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==)
 | `52.94% <0.00%> (-47.06%)` | :arrow_down: |
   | 
[airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==)
 | `47.18% <0.00%> (-44.79%)` | :arrow_down: |
   | 
[...viders/cncf/kubernetes/operators/kubernetes\_pod.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvY25jZi9rdWJlcm5ldGVzL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZC5weQ==)
 | `69.69% <0.00%> (-27.96%)` | :arrow_down: |
   | ... and [1017 
more](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree-more) 
| |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=footer). 
Last update 
[1fa7ac1...2991e61](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [airflow] codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default connection entry for PinotDbApiHook

2020-03-17 Thread GitBox
codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default 
connection entry for PinotDbApiHook
URL: https://github.com/apache/airflow/pull/6457#issuecomment-552073991
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=h1) 
Report
   > Merging 
[#6457](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=desc) into 
[master](https://codecov.io/gh/apache/airflow/commit/1fa7ac1b3978469ba3ec9b902b4f2b4890f8199d=desc)
 will **increase** coverage by `1.35%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/airflow/pull/6457/graphs/tree.svg?width=650=150=pr=WdLKlKHOAU)](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree)
   
   ```diff
   @@Coverage Diff @@
   ##   master#6457  +/-   ##
   ==
   + Coverage   85.34%   86.69%   +1.35% 
   ==
 Files 791  915 +124 
 Lines   4012444189+4065 
   ==
   + Hits3424338311+4068 
   + Misses   5881 5878   -3 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree) | 
Coverage Δ | |
   |---|---|---|
   | 
[airflow/utils/db.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy91dGlscy9kYi5weQ==)
 | `97.50% <100.00%> (-0.44%)` | :arrow_down: |
   | 
[airflow/operators/mysql\_to\_hive.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfdG9faGl2ZS5weQ==)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...irflow/providers/apache/cassandra/sensors/table.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvYXBhY2hlL2Nhc3NhbmRyYS9zZW5zb3JzL3RhYmxlLnB5)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[...rflow/providers/apache/cassandra/sensors/record.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvYXBhY2hlL2Nhc3NhbmRyYS9zZW5zb3JzL3JlY29yZC5weQ==)
 | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | 
[airflow/operators/s3\_to\_hive\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvczNfdG9faGl2ZV9vcGVyYXRvci5weQ==)
 | `0.00% <0.00%> (-93.97%)` | :arrow_down: |
   | 
[airflow/contrib/hooks/grpc\_hook.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL2dycGNfaG9vay5weQ==)
 | `0.00% <0.00%> (-91.94%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==)
 | `44.44% <0.00%> (-55.56%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==)
 | `52.94% <0.00%> (-47.06%)` | :arrow_down: |
   | 
[airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==)
 | `47.18% <0.00%> (-44.79%)` | :arrow_down: |
   | 
[...viders/cncf/kubernetes/operators/kubernetes\_pod.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvY25jZi9rdWJlcm5ldGVzL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZC5weQ==)
 | `69.69% <0.00%> (-27.96%)` | :arrow_down: |
   | ... and [1017 
more](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree-more) 
| |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=footer). 
Last update 
[1fa7ac1...2991e61](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [airflow] codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default connection entry for PinotDbApiHook

2020-01-22 Thread GitBox
codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default 
connection entry for PinotDbApiHook
URL: https://github.com/apache/airflow/pull/6457#issuecomment-552073991
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=h1) 
Report
   > Merging 
[#6457](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=desc) into 
[master](https://codecov.io/gh/apache/airflow/commit/0cb9d2d124939ba19c539a8056ad5396c6f2dd39?src=pr=desc)
 will **increase** coverage by `1.29%`.
   > The diff coverage is `76.62%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/airflow/pull/6457/graphs/tree.svg?width=650=WdLKlKHOAU=150=pr)](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree)
   
   ```diff
   @@Coverage Diff @@
   ##   master#6457  +/-   ##
   ==
   + Coverage   83.76%   85.05%   +1.29% 
   ==
 Files 635  791 +156 
 Lines   3685140125+3274 
   ==
   + Hits3086734128+3261 
   - Misses   5984 5997  +13
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree) | 
Coverage Δ | |
   |---|---|---|
   | 
[airflow/contrib/operators/pubsub\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9wdWJzdWJfb3BlcmF0b3IucHk=)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[...ample\_dags/example\_branch\_python\_dop\_operator\_3.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9leGFtcGxlX2RhZ3MvZXhhbXBsZV9icmFuY2hfcHl0aG9uX2RvcF9vcGVyYXRvcl8zLnB5)
 | `75% <ø> (ø)` | :arrow_up: |
   | 
[...flow/contrib/example\_dags/example\_qubole\_sensor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2V4YW1wbGVfZGFncy9leGFtcGxlX3F1Ym9sZV9zZW5zb3IucHk=)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[...irflow/contrib/operators/redis\_publish\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9yZWRpc19wdWJsaXNoX29wZXJhdG9yLnB5)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[airflow/contrib/hooks/qubole\_hook.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL3F1Ym9sZV9ob29rLnB5)
 | `52.67% <ø> (-2.97%)` | :arrow_down: |
   | 
[...w/example\_dags/example\_external\_task\_marker\_dag.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9leGFtcGxlX2RhZ3MvZXhhbXBsZV9leHRlcm5hbF90YXNrX21hcmtlcl9kYWcucHk=)
 | `100% <ø> (ø)` | |
   | 
[airflow/executors/local\_executor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9leGVjdXRvcnMvbG9jYWxfZXhlY3V0b3IucHk=)
 | `92.48% <ø> (+10.3%)` | :arrow_up: |
   | 
[...low/contrib/operators/wasb\_delete\_blob\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy93YXNiX2RlbGV0ZV9ibG9iX29wZXJhdG9yLnB5)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[airflow/contrib/sensors/sagemaker\_tuning\_sensor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvc2FnZW1ha2VyX3R1bmluZ19zZW5zb3IucHk=)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[...flow/contrib/sensors/sagemaker\_transform\_sensor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvc2FnZW1ha2VyX3RyYW5zZm9ybV9zZW5zb3IucHk=)
 | `100% <ø> (ø)` | :arrow_up: |
   | ... and [744 
more](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree-more) 
| |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=footer). 
Last update 
[0cb9d2d...8951bd1](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [airflow] codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default connection entry for PinotDbApiHook

2020-01-22 Thread GitBox
codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default 
connection entry for PinotDbApiHook
URL: https://github.com/apache/airflow/pull/6457#issuecomment-552073991
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=h1) 
Report
   > Merging 
[#6457](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=desc) into 
[master](https://codecov.io/gh/apache/airflow/commit/0cb9d2d124939ba19c539a8056ad5396c6f2dd39?src=pr=desc)
 will **increase** coverage by `1.29%`.
   > The diff coverage is `76.62%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/airflow/pull/6457/graphs/tree.svg?width=650=WdLKlKHOAU=150=pr)](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree)
   
   ```diff
   @@Coverage Diff @@
   ##   master#6457  +/-   ##
   ==
   + Coverage   83.76%   85.05%   +1.29% 
   ==
 Files 635  791 +156 
 Lines   3685140125+3274 
   ==
   + Hits3086734128+3261 
   - Misses   5984 5997  +13
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree) | 
Coverage Δ | |
   |---|---|---|
   | 
[airflow/contrib/operators/pubsub\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9wdWJzdWJfb3BlcmF0b3IucHk=)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[...ample\_dags/example\_branch\_python\_dop\_operator\_3.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9leGFtcGxlX2RhZ3MvZXhhbXBsZV9icmFuY2hfcHl0aG9uX2RvcF9vcGVyYXRvcl8zLnB5)
 | `75% <ø> (ø)` | :arrow_up: |
   | 
[...flow/contrib/example\_dags/example\_qubole\_sensor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2V4YW1wbGVfZGFncy9leGFtcGxlX3F1Ym9sZV9zZW5zb3IucHk=)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[...irflow/contrib/operators/redis\_publish\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9yZWRpc19wdWJsaXNoX29wZXJhdG9yLnB5)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[airflow/contrib/hooks/qubole\_hook.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2hvb2tzL3F1Ym9sZV9ob29rLnB5)
 | `52.67% <ø> (-2.97%)` | :arrow_down: |
   | 
[...w/example\_dags/example\_external\_task\_marker\_dag.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9leGFtcGxlX2RhZ3MvZXhhbXBsZV9leHRlcm5hbF90YXNrX21hcmtlcl9kYWcucHk=)
 | `100% <ø> (ø)` | |
   | 
[airflow/executors/local\_executor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9leGVjdXRvcnMvbG9jYWxfZXhlY3V0b3IucHk=)
 | `92.48% <ø> (+10.3%)` | :arrow_up: |
   | 
[...low/contrib/operators/wasb\_delete\_blob\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy93YXNiX2RlbGV0ZV9ibG9iX29wZXJhdG9yLnB5)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[airflow/contrib/sensors/sagemaker\_tuning\_sensor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvc2FnZW1ha2VyX3R1bmluZ19zZW5zb3IucHk=)
 | `100% <ø> (ø)` | :arrow_up: |
   | 
[...flow/contrib/sensors/sagemaker\_transform\_sensor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL3NlbnNvcnMvc2FnZW1ha2VyX3RyYW5zZm9ybV9zZW5zb3IucHk=)
 | `100% <ø> (ø)` | :arrow_up: |
   | ... and [744 
more](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree-more) 
| |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=footer). 
Last update 
[0cb9d2d...8951bd1](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [airflow] codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default connection entry for PinotDbApiHook

2019-11-08 Thread GitBox
codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default 
connection entry for PinotDbApiHook
URL: https://github.com/apache/airflow/pull/6457#issuecomment-552073991
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=h1) 
Report
   > Merging 
[#6457](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=desc) into 
[master](https://codecov.io/gh/apache/airflow/commit/813cd431204b71f4e7c27f2bee9fafcc910da4d0?src=pr=desc)
 will **decrease** coverage by `0.76%`.
   > The diff coverage is `100%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/airflow/pull/6457/graphs/tree.svg?width=650=WdLKlKHOAU=150=pr)](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree)
   
   ```diff
   @@Coverage Diff @@
   ##   master#6457  +/-   ##
   ==
   - Coverage   84.08%   83.31%   -0.77% 
   ==
 Files 635  631   -4 
 Lines   3684936639 -210 
   ==
   - Hits3098330527 -456 
   - Misses   5866 6112 +246
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree) | 
Coverage Δ | |
   |---|---|---|
   | 
[airflow/utils/db.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy91dGlscy9kYi5weQ==)
 | `90.17% <100%> (ø)` | :arrow_up: |
   | 
[airflow/operators/mysql\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfb3BlcmF0b3IucHk=)
 | `0% <0%> (-100%)` | :arrow_down: |
   | 
[airflow/operators/mysql\_to\_hive.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfdG9faGl2ZS5weQ==)
 | `0% <0%> (-100%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==)
 | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | 
[airflow/executors/sequential\_executor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9leGVjdXRvcnMvc2VxdWVudGlhbF9leGVjdXRvci5weQ==)
 | `47.61% <0%> (-52.39%)` | :arrow_down: |
   | 
[...ow/contrib/example\_dags/example\_qubole\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2V4YW1wbGVfZGFncy9leGFtcGxlX3F1Ym9sZV9vcGVyYXRvci5weQ==)
 | `32.5% <0%> (-47.5%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==)
 | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | 
[airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==)
 | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | 
[airflow/kubernetes/kube\_client.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL2t1YmVfY2xpZW50LnB5)
 | `33.33% <0%> (-41.67%)` | :arrow_down: |
   | 
[airflow/hooks/presto\_hook.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9ob29rcy9wcmVzdG9faG9vay5weQ==)
 | `36.73% <0%> (-40.82%)` | :arrow_down: |
   | ... and [63 
more](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree-more) 
| |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=footer). 
Last update 
[813cd43...0cb9d2d](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [airflow] codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default connection entry for PinotDbApiHook

2019-11-08 Thread GitBox
codecov-io edited a comment on issue #6457: [AIRFLOW-5800] Add a default 
connection entry for PinotDbApiHook
URL: https://github.com/apache/airflow/pull/6457#issuecomment-552073991
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=h1) 
Report
   > Merging 
[#6457](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=desc) into 
[master](https://codecov.io/gh/apache/airflow/commit/813cd431204b71f4e7c27f2bee9fafcc910da4d0?src=pr=desc)
 will **decrease** coverage by `0.76%`.
   > The diff coverage is `100%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/airflow/pull/6457/graphs/tree.svg?width=650=WdLKlKHOAU=150=pr)](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree)
   
   ```diff
   @@Coverage Diff @@
   ##   master#6457  +/-   ##
   ==
   - Coverage   84.08%   83.31%   -0.77% 
   ==
 Files 635  631   -4 
 Lines   3684936639 -210 
   ==
   - Hits3098330527 -456 
   - Misses   5866 6112 +246
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=tree) | 
Coverage Δ | |
   |---|---|---|
   | 
[airflow/utils/db.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy91dGlscy9kYi5weQ==)
 | `90.17% <100%> (ø)` | :arrow_up: |
   | 
[airflow/operators/mysql\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfb3BlcmF0b3IucHk=)
 | `0% <0%> (-100%)` | :arrow_down: |
   | 
[airflow/operators/mysql\_to\_hive.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9vcGVyYXRvcnMvbXlzcWxfdG9faGl2ZS5weQ==)
 | `0% <0%> (-100%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==)
 | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | 
[airflow/executors/sequential\_executor.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9leGVjdXRvcnMvc2VxdWVudGlhbF9leGVjdXRvci5weQ==)
 | `47.61% <0%> (-52.39%)` | :arrow_down: |
   | 
[...ow/contrib/example\_dags/example\_qubole\_operator.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9jb250cmliL2V4YW1wbGVfZGFncy9leGFtcGxlX3F1Ym9sZV9vcGVyYXRvci5weQ==)
 | `32.5% <0%> (-47.5%)` | :arrow_down: |
   | 
[airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==)
 | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | 
[airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==)
 | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | 
[airflow/kubernetes/kube\_client.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL2t1YmVfY2xpZW50LnB5)
 | `33.33% <0%> (-41.67%)` | :arrow_down: |
   | 
[airflow/hooks/presto\_hook.py](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree#diff-YWlyZmxvdy9ob29rcy9wcmVzdG9faG9vay5weQ==)
 | `36.73% <0%> (-40.82%)` | :arrow_down: |
   | ... and [63 
more](https://codecov.io/gh/apache/airflow/pull/6457/diff?src=pr=tree-more) 
| |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=footer). 
Last update 
[813cd43...0cb9d2d](https://codecov.io/gh/apache/airflow/pull/6457?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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:
us...@infra.apache.org


With regards,
Apache Git Services