You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/12/13 18:42:05 UTC

[GitHub] [superset] bkyryliuk opened a new pull request #17731: fix: call op.get_bind() inside the function, not on import

bkyryliuk opened a new pull request #17731:
URL: https://github.com/apache/superset/pull/17731


   ### SUMMARY
   fix: call op.get_bind() inside the function, not on import
   Error log:
   ```
   s/abe27eaf93db_add_extra_config_column_to_alerts.py", line 34, in <module>
       connection = op.get_bind()
     File "<string>", line 7, in get_bind
     File "/srv/yaps/mounts/superset/superset_build/bcf71bd610347f780fa22f8365c6a015a70248e5-a38d46655a6ec6c4745dd039486ca598/dropbox/apx/superset/superset_cli.runfiles/__main__/thirdparty/superset_python3/pip/alembic/alembic-cpython-38/lib/alembic/util/langhelpers.py", line 78, in _name_error
       raise NameError(
   NameError: Can't invoke function 'get_bind', as the proxy object has not yet been established for the Alembic 'Operations' class.  Try placing this code inside a callable.
   ```


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

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] betodealmeida commented on pull request #17731: fix: call op.get_bind() inside the function, not on import

Posted by GitBox <gi...@apache.org>.
betodealmeida commented on pull request #17731:
URL: https://github.com/apache/superset/pull/17731#issuecomment-992827865


   https://github.com/apache/superset/pull/17728
   
   That MySQL test is broken for some reason. :-/


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

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] betodealmeida commented on pull request #17731: fix: call op.get_bind() inside the function, not on import

Posted by GitBox <gi...@apache.org>.
betodealmeida commented on pull request #17731:
URL: https://github.com/apache/superset/pull/17731#issuecomment-992908156


   Closing since I merged https://github.com/apache/superset/pull/17728.


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

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] betodealmeida closed pull request #17731: fix: call op.get_bind() inside the function, not on import

Posted by GitBox <gi...@apache.org>.
betodealmeida closed pull request #17731:
URL: https://github.com/apache/superset/pull/17731


   


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

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org