a9eb629bcfc8239e86916661f80f0bee9ca26b79,fonduer/snorkel/models/meta.py,,new_sessionmaker,#,37

Before Change

    // Otherwise any e.g. query starts a transaction, locking tables... very bad for e.g. multiple notebooks
    // open, multiple processes, etc.
    if snorkel_postgres:
        snorkel_engine = create_engine(snorkel_conn_string, isolation_level="AUTOCOMMIT")
    else:
        snorkel_engine = create_engine(snorkel_conn_string)

    // New sessionmaker

After Change

    // http://oddbird.net/2014/06/14/sqlalchemy-postgres-autocommit/
    // Otherwise any e.g. query starts a transaction, locking tables... very
    // bad for e.g. multiple notebooks open, multiple processes, etc.
    if Meta.snorkel_postgres and Meta.ready:
        snorkel_engine = create_engine(
            Meta.conn_string, isolation_level="AUTOCOMMIT")
    else:
        raise ValueError(
            "Meta variables have not been initialized with a postgres connection string."
        )

    // New sessionmaker
    SnorkelSession = sessionmaker(bind=snorkel_engine)
    return SnorkelSession

Instances