You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/02/17 22:45:05 UTC
[GitHub] [airflow] ConstantinoSchillebeeckx opened a new issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
ConstantinoSchillebeeckx opened a new issue #14286:
URL: https://github.com/apache/airflow/issues/14286
<!--
Welcome to Apache Airflow! For a smooth issue process, try to answer the following questions.
Don't worry if they're not all applicable; just try to include what you can :-)
If you need to include code snippets or logs, please put them in fenced code
blocks. If they're super-long, please use the details tag like
<details><summary>super-long log</summary> lots of stuff </details>
Please delete these comment blocks before submitting the issue.
-->
<!--
IMPORTANT!!!
PLEASE CHECK "SIMILAR TO X EXISTING ISSUES" OPTION IF VISIBLE
NEXT TO "SUBMIT NEW ISSUE" BUTTON!!!
PLEASE CHECK IF THIS ISSUE HAS BEEN REPORTED PREVIOUSLY USING SEARCH!!!
Please complete the next sections or the issue will be closed.
These questions are the first thing we need to know to understand the context.
-->
**Apache Airflow version**: 2.0.1
**Environment**:
- **Cloud provider or hardware configuration**: MacBook Pro
- **OS** (e.g. from /etc/os-release): Catalina 10.15.7
- **Kernel** (e.g. `uname -a`): Darwin CSchillebeeckx-0589.local 19.6.0 Darwin Kernel Version 19.6.0: Thu Oct 29 22:56:45 PDT 2020; root:xnu-6153.141.2.2~1/RELEASE_X86_64 x86_64
- **Install tools**: pip
requirements.txt
```
apache-airflow[crypto,celery,amazon,mysql,jdbc,password,redis,slack,snowflake,ssh,google,databricks,mongo,zendesk,papermill,salesforce]==2.0.1
pyarrow==0.17.1
iniconfig==1.1.1
sqlparse==0.4.1
google-api-python-client===1.12.8
google-auth==1.27.0
google-api-core==1.26.0
avro-python3==1.10.0
databricks-connect==7.3.8
matplotlib==3.3.4
scikit-learn==0.24.1
ipykernel==5.4.3
flower==0.9.7
```
**What happened**:
I'm using MySQL as a backend and am using the `GoogleBaseHook`; when I run the webserver I see:
```
webserver_1 | [2021-02-16 23:39:13,008] {providers_manager.py:299} WARNING - Exception when importing 'airflow.providers.google.cloud.hooks.cloud_sql.CloudSQLHook' from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
webserver_1 | [2021-02-16 23:39:13,009] {providers_manager.py:299} WARNING - Exception when importing 'airflow.providers.google.cloud.hooks.cloud_sql.CloudSQLDatabaseHook' from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
webserver_1 | [2021-02-16 23:39:13,066] {providers_manager.py:299} WARNING - Exception when importing 'airflow.providers.google.cloud.hooks.cloud_sql.CloudSQLHook' from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
webserver_1 | [2021-02-16 23:39:13,067] {providers_manager.py:299} WARNING - Exception when importing 'airflow.providers.google.cloud.hooks.cloud_sql.CloudSQLDatabaseHook' from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
```
If I add the extra package `postgres` the warnings are not present anymore.
**What you expected to happen**:
No warnings
**How to reproduce it**:
<!---
As minimally and precisely as possible. Keep in mind we do not have access to your cluster or dags.
If you are using kubernetes, please attempt to recreate the issue using minikube or kind.
## Install minikube/kind
- Minikube https://minikube.sigs.k8s.io/docs/start/
- Kind https://kind.sigs.k8s.io/docs/user/quick-start/
If this is a UI bug, please provide a screenshot of the bug or a link to a youtube video of the bug in action
You can include images using the .md style of
To record a screencast, mac users can use QuickTime and then create an unlisted youtube video with the resulting .mov file.
--->
**Anything else we need to know**:
<!--
How often does this problem occur? Once? Every time etc?
Any relevant logs to include? Put them here in side a detail tag:
<details><summary>x.log</summary> lots of stuff </details>
-->
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-802249427
I'll take a look shortly (tomorrow). This should be an easy fix.
--
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:
users@infra.apache.org
[GitHub] [airflow] ConstantinoSchillebeeckx commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
ConstantinoSchillebeeckx commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781589580
> This should be a one-time only message per webserver process only at the start of the process. Do you see it more often/repeated @ConstantinoSchillebeeckx ?
I'm seeing the warnings constantly being emitted
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] OBITORASU commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
OBITORASU commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781812879
Repeating the info can indeed be annoying for the user if the server keeps emitting ```INFO``` same as ```WARNINGS``` in the above example. Any ideas on how to handle this case?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] ashb commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
ashb commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-802094734
@potiuk We've fixed this now with the latest batch of provider releases haven't we?
--
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781226323
Sure. Feel free. I think the gist is that we should log those warnings at info level not warning :)
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] ConstantinoSchillebeeckx commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
ConstantinoSchillebeeckx commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-802808281
I can pick this up if no one minds; if someone can nudge me in the right direction that would be great.
--
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-782289802
I will check it during the weekend
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] ashb commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
ashb commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-803130448
> @ConstantinoSchillebeeckx - I think the main task is to diagnose it to see what is really the issue here. I expected that those warnings will only be printed once (which would be acceptable) but seems in the webserver they continue spamming the logs - I will have time to take a look tomorrow, but if you want, you can pick it before.
On webserver boot up, the app is loaded once before starting gunicorn, then again for each of the (by default 4) workers, and then by default one webserver worker is replaced every 30s. Each time a web process starts up it will load the providers and produce these logs.
--
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:
users@infra.apache.org
[GitHub] [airflow] OBITORASU edited a comment on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
OBITORASU edited a comment on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781812879
Repeating the info can indeed be annoying for the user if the server keeps emitting ```INFO``` same as ```WARNINGS``` in the above example. Any ideas on how to handle this particular case?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] OBITORASU commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
OBITORASU commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-802619968
Dropping in because the thread is revived. I am kind of in a bad spot with my time management (stacked in middle of exams), so I don't think I would be able to put enough time and effort on this issue. Due to the above circumstances, I would like unassign myself from this issue. I do hope some other new contributor picks this one up. Sorry for the inconvenience :(
--
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-780922211
Yep. We could remove the warnings indeed.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] OBITORASU commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
OBITORASU commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-782291177
I guess I will wait until this is investigated further then.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-803147283
@OBITORASU @ConstantinoSchillebeeckx -> #14903 to address it.
--
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781568420
This should be a one-time only message per webserver process only at the start of the process. Do you see it more often/repeated @ConstantinoSchillebeeckx ?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] OBITORASU commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
OBITORASU commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781209649
Can I work on this particular issue? I am assuming addding ```postgres``` to ```apache-airflow[crypto,celery,amazon,mysql,jdbc,password,redis,slack,snowflake,ssh,google,databricks,mongo,zendesk,papermill,salesforce]==2.0.1``` this line should cover the warnings (correct me if I am wrong).
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] boring-cyborg[bot] commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-780903984
Thanks for opening your first issue here! Be sure to follow the issue template!
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] ConstantinoSchillebeeckx commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
ConstantinoSchillebeeckx commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-782273068
> Does it ever stop @ConstantinoSchillebeeckx or does it continue forever?
@potiuk I don't know that I can comment about it continuing _forever_ 🤣 ; however I did observe these `WARNINGS` for over 20 minutes.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-782289623
:rofl: Indeed I do not want you to wait THAT long.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-802248845
Still needs investigation - it's not really a provider's issue but rather provider's manager (which is part of Airflow itself. So It still needs taking a look and likely releasing in 2.0.2
--
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-803136874
Yep. Just reproduced it. The warnings come every 30 s
--
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781572326
@OBITORASU -> Just hold on for a moment, @ConstantinoSchillebeeckx 's point made me think ^^ - this migth be a bit better approach to turn it into more informative message (see above 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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-802995704
@ConstantinoSchillebeeckx - I think the main task is to diagnose it to see what is really the issue here. I expected that those warnings will only be printed once (which would be acceptable) but seems in the webserver they continue spamming the logs - I will have time to take a look tomorrow, but if you want, you can pick it before.
--
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781226981
Those warnings here are not really warnings - they will happen if you do not have all providers installed, but they should not be treated as warnings, this is expected behaviour, so it should be INFO.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] ConstantinoSchillebeeckx commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
ConstantinoSchillebeeckx commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781396046
Even at `INFO`, isn't this going to be really chatty? I think the webserver will just keep emitting those statements; I don't know this codebase well at all, but wouldn't a better fix be not to emit that message if it's not needed?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-782243227
That's strange to see it repeated. This might happen once per process if you have many processes but I am nto sure this is what happens here. Maybe it just repeats few times and stops? I think this might happen by gunicorn spawning few processes after webserver starts. Does it ever stop @ConstantinoSchillebeeckx or does it continue forever?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781571702
What we could do indeed, we could actually handle this a bit better - we could capture the warnings and print a nice summary indicating what is really going on. In this case we could aggregate all of the warning of this form:
```
WARNING: No module named 'airflow.providers.postgres'"
```
into
```
INFO: "Some of the features of `apache-airflow-providers-google` provider are not available . Please install 'apache-airflow-providers-postgres' to use "CloudSQLHook". "CloudSQLDatabaseHook" ...
```
This could be much nicer and informative and I think they are worth seing in logs as "INFO".
WDYT @ConstantinoSchillebeeckx ?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-803133194
Yep. Makes sense. Just started looking at it :)
--
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:
users@infra.apache.org
[GitHub] [airflow] OBITORASU edited a comment on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
OBITORASU edited a comment on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781209649
Can I work on this particular issue? I am assuming addding ```postgres``` to the ```apache-airflow[crypto,celery,amazon,mysql,jdbc,password,redis,slack,snowflake,ssh,google,databricks,mongo,zendesk,papermill,salesforce]==2.0.1``` line should cover the warnings (correct me if I am wrong).
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk closed issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
potiuk closed issue #14286:
URL: https://github.com/apache/airflow/issues/14286
--
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:
users@infra.apache.org
[GitHub] [airflow] OBITORASU commented on issue #14286: from 'apache-airflow-providers-google' package: No module named 'airflow.providers.postgres'
Posted by GitBox <gi...@apache.org>.
OBITORASU commented on issue #14286:
URL: https://github.com/apache/airflow/issues/14286#issuecomment-781329463
Checking the error log and your suggestions above, should my approach be to introduce changes to ```providers_manager.py``` file and change the ```log.warning``` to ```log.info``` on line 295 to fix this issue? Hoping to hear back from you, thanks for your time and patience!
----------------------------------------------------------------
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:
users@infra.apache.org