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 2019/06/09 09:09:10 UTC

[GitHub] [airflow] BasPH commented on issue #5383: [AIRFLOW-4737] Increase and document celery queue name limit

BasPH commented on issue #5383: [AIRFLOW-4737] Increase and document celery queue name limit
URL: https://github.com/apache/airflow/pull/5383#issuecomment-500197493
 
 
   I suggest to widen the restrictions of the module name. Opened a PR (https://github.com/apache/airflow/pull/5391) to allow module names to start with 0-9 and now have max 60 chars, this is green on all our airflow/migration module names. @ashb WDYT?
   
   @villasv For the missing-docstring messages I suggest to use the Python docstring format so that Pylint will be happy:
   ```python
   """use batch_alter_table to support SQLite workaround"""
   ```

----------------------------------------------------------------
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


With regards,
Apache Git Services