Merge lp:~kalebral-deactivatedaccount/drizzle-automation/crash-me-remove-temp-limits into lp:drizzle-automation

Proposed by Lee Bieber
Status: Merged
Merged at revision: not available
Proposed branch: lp:~kalebral-deactivatedaccount/drizzle-automation/crash-me-remove-temp-limits
Merge into: lp:drizzle-automation
Diff against target: 20 lines (+1/-8)
1 file modified
drizzle/automation/crashme/run.py (+1/-8)
To merge this branch: bzr merge lp:~kalebral-deactivatedaccount/drizzle-automation/crash-me-remove-temp-limits
Reviewer Review Type Date Requested Status
Jay Pipes Approve
Review via email: mp+20663@code.launchpad.net

Description of the change

Thanks to Eric, remove temporary hack now that bug #419350 is fixed, no longer need to write temporary limits to limits configuration file

To post a comment you must log in.
Revision history for this message
Jay Pipes (jaypipes) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'drizzle/automation/crashme/run.py'
2--- drizzle/automation/crashme/run.py 2010-01-12 20:54:51 +0000
3+++ drizzle/automation/crashme/run.py 2010-03-04 17:22:13 +0000
4@@ -149,15 +149,8 @@
5 logging.info("Removing %s" % output_filename)
6 os.remove(output_filename)
7
8- # TEMPORARY HACK FOR NOW
9- # for now set some variable limits, these are filed as bug #419350 and should be removed after the bug is fixed
10- logging.info("Writing temporary limits to %s - TO BE REMOVED after bug #419350 is fixed" % output_filename)
11 output_file= open(output_filename,"w")
12- output_file.writelines("query_size=1048576\n")
13- output_file.writelines("max_text_size=1048576\n")
14- output_file.writelines("where_string_size=1048576\n")
15- output_file.writelines("select_string_size=1048576\n")
16- # do not remove this one, as we don't support '+' for concatenation
17+ # don't support '+' for concatenation
18 output_file.writelines("func_extra_concat_as_+=no\n")
19 output_file.flush()
20 output_file.close()

Subscribers

People subscribed via source and target branches

to all changes: