Add aurora errno to rejectReadOnly check

[jeffcharles]

Description

Include errno 1290 in rejectReadOnly check. Aurora returns this because the database is running with the --read-only option.

Checklist

• Code compiles correctly
• Created tests which fail without the change (if possible)
• All tests passing
• Extended the README / documentation, if necessary
• Added myself / the copyright holder to the AUTHORS file

(Not sure how to add a test for this easily w/o running another MySQL instance with a TCP reverse proxy, and even then it wouldn't necessarily reflect how Aurora runs in practice in the future)

[methane]

1290 error can be used for other purpose.
In other words, there is danger about retrying.

err := db.Exec(`UPDATE version SET version=version+1;
LOAD DATA INFILE "/tmp/data.csv" INTO TABLE data;`)
// ERROR 1290 (HY000): The MySQL server is running with the --secure-file-priv option so it cannot execute this statement

If such query is retried, version number is incremented multiple times.
But I understand your requirement and RejectReadOnly option is for it. So I approve OK this change.

Please add note to RejectReadOnly option like:

NOTE: ERROR 1290 will be returned for `read-only` server and this option cause retry for the error.
But same error number is used for some other cases.  You should ensure your application
never cause ERROR 1290 except `read-only` mode.

[bryanpaluch]

Any movement on this?

[julienschmidt]

Seems like our CI somehow hangs.
The easiest workaround would probably if you rebase on the current master or merge the current master to trigger another build.

[jeffcharles]

Squashed and rebased on top of master

[julienschmidt]

thanks a lot!