Skip to content
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

Fix with parameters passing in 'get_mapping' func #71

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

Conversation

bubanoid
Copy link
Contributor

@bubanoid bubanoid commented Oct 19, 2017

Там треба поправити. Але якщо зробити це так, як я зробив, то в модулі auctions_server.py функція auctions_proxy змінить свою поведінку. В ній викликається викликається get з Мемоайзера:

    proxy_path = auctions_server.proxy_mappings.get(
        str(auction_doc_id),
        get_mapping,
        (auctions_server.config['REDIS'], str(auction_doc_id), False), max_age=60
    )

і відповідно get_mapping тепер викличеться із параметром master=False і цей False передасться в get_database(config, master=master).get(auction_id). Якщо по логіці треба, щоб get_database викликався із master=True, то треба додатково змінити у функції auctions_proxy параметр master з яким викликається Мемоайзер з False на True.


This change is Reviewable

@yshalenyk
Copy link
Contributor

@bubanoid для еско вже не актуально.

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.

None yet

2 participants