Skip to content

Get rid of SQLAlchemy warning messages #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TechStudent10
Copy link

@TechStudent10 TechStudent10 commented Jul 19, 2021

Fixes #2

@divyanshu-parihar
Copy link

can you share what warnings are given to you .

@TechStudent10
Copy link
Author

TechStudent10 commented Jul 20, 2021

@divyanshuParihar265

C:\Users\username_that_i_am_not_disclosing\Cloned GIT Stuff\Flask-Blog-Tutorial\venv\lib\site-
packages\flask_sqlalchemy\__init__.py:873: FSADeprecationWarning: SQLALCHEMY_TRACK_MODIFICATIONS adds significant 
overhead and will be disabled by default in the future.  Set it to True or False to suppress this warning.
  'SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and '

@divyanshu-parihar
Copy link

I think in this project , sqlalchemy event system is not used . so suppress ing warnings should be fine and other than that it also has its own resource consumption. It can be a good optimization at backend too.
https://stackoverflow.com/questions/33738467/how-do-i-know-if-i-can-disable-sqlalchemy-track-modifications

@TechStudent10
Copy link
Author

@divyanshuParihar265 the link you put redirects back here.

@divyanshu-parihar
Copy link

i don't know why this happening , but you can try copy pasting the url into your browser
.

@TechStudent10
Copy link
Author

@divyanshuParihar265 Ok!

@TechStudent10
Copy link
Author

@divyanshu-parihar you apparently changed your username, so now my pings don't work haha.

@ConnorDiddy ConnorDiddy mentioned this pull request May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warning when running.
2 participants