Need to set mysql_options to enable LOAD DATA LOCAL INFILE #233

Closed
sodabrew opened this Issue Dec 28, 2011 · 5 comments

Projects

None yet

3 participants

@sodabrew
Collaborator

Per MySQL Manual:
http://dev.mysql.com/doc/refman/5.1/en/load-data-local.html

"""
If you build MySQL from source but do not invoke configure with the --enable-local-infile option, LOAD DATA LOCAL cannot be used by any client unless it is written explicitly to invoke mysql_options(... MYSQL_OPT_LOCAL_INFILE, 0). See Section 20.9.3.49, “mysql_options()”.
"""

I'd like to expose this as a connection flag.

[I'm working through my use-case right now, will submit a patch when I have a reasonable idea on how/where to expose this flag.]

@sodabrew sodabrew added a commit that referenced this issue Jan 28, 2012
@sodabrew sodabrew Issue #233 version bump. b459d81
@sodabrew sodabrew added a commit to sodabrew/mysql2 that referenced this issue Jan 28, 2012
@sodabrew sodabrew Issue #233 refactor mysql_options calls and add an options method to …
…the class.
eaf80c8
@sodabrew sodabrew added a commit to sodabrew/mysql2 that referenced this issue Jan 28, 2012
@sodabrew sodabrew Issue #233 add option :local_infile to the constructor. 54d452b
@sodabrew
Collaborator

See pull request #242 for correct changesets to pull.

@thbar

Woot - works for me, thanks!

@sodabrew sodabrew added a commit to sodabrew/mysql2 that referenced this issue Apr 4, 2012
@sodabrew sodabrew Issue #233 fixes to the options refactor plus tests now pass. ab19d05
@sodabrew sodabrew added a commit to sodabrew/mysql2 that referenced this issue Apr 4, 2012
@sodabrew sodabrew Issue #233 refactor mysql_options calls and add an options method to …
…the class.
324afe8
@sodabrew sodabrew added a commit to sodabrew/mysql2 that referenced this issue Apr 4, 2012
@sodabrew sodabrew Issue #233 add option :local_infile to the constructor. 9a4c379
@sodabrew sodabrew added a commit to sodabrew/mysql2 that referenced this issue Apr 4, 2012
@sodabrew sodabrew Issue #233 fixes to the options refactor plus tests now pass. eebce9b
@sodabrew
Collaborator

Pull request #252 is take three: updated to a current branch, tests passing, looking good.

@sodabrew
Collaborator

Thanks for integrating my changes!

@sodabrew sodabrew closed this Aug 3, 2012
@brianmario
Owner

No need to thank me, thank you for contributing them! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment