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
=== modified file 'drizzle/automation/crashme/run.py'
--- drizzle/automation/crashme/run.py 2010-01-12 20:54:51 +0000
+++ drizzle/automation/crashme/run.py 2010-03-04 17:22:13 +0000
@@ -149,15 +149,8 @@
149 logging.info("Removing %s" % output_filename)149 logging.info("Removing %s" % output_filename)
150 os.remove(output_filename)150 os.remove(output_filename)
151 151
152 # TEMPORARY HACK FOR NOW
153 # for now set some variable limits, these are filed as bug #419350 and should be removed after the bug is fixed
154 logging.info("Writing temporary limits to %s - TO BE REMOVED after bug #419350 is fixed" % output_filename)
155 output_file= open(output_filename,"w")152 output_file= open(output_filename,"w")
156 output_file.writelines("query_size=1048576\n")153 # don't support '+' for concatenation
157 output_file.writelines("max_text_size=1048576\n")
158 output_file.writelines("where_string_size=1048576\n")
159 output_file.writelines("select_string_size=1048576\n")
160 # do not remove this one, as we don't support '+' for concatenation
161 output_file.writelines("func_extra_concat_as_+=no\n")154 output_file.writelines("func_extra_concat_as_+=no\n")
162 output_file.flush()155 output_file.flush()
163 output_file.close()156 output_file.close()

Subscribers

People subscribed via source and target branches

to all changes: