oursql dialect is missing raise_on_warning option

Issue #2532 resolved
Marcin Lulek created an issue

adding this code to create_connect_args() corrects this issue

    opts['raise_on_warnings']('raise_on_warnings') = opts.get('raise_on_warnings', True)
    util.coerce_kw_type(opts, 'raise_on_warnings', bool)

Comments (5)

  1. Marcin Lulek reporter

    the code should look like this:

            opts['raise_on_warnings']('raise_on_warnings') = opts.get('raise_on_warnings', True)
            util.coerce_kw_type(opts, 'raise_on_warnings', bool)
    
  2. Mike Bayer repo owner

    thanks, part of the effort i wanted to make here was to look up an existing ticket as this seemed familiar.

  3. Log in to comment