On 08/24/2011 07:09 PM, Vincent Ladeuil wrote:
> I've discussed it with Martin, the disagreement was two-fold:
>
> - he felt the bug was too vague,
>
> - I felt setting the importance to low wasn't the right way to
> convey that the priority is low (which I agree with, I don't
> intend to fix it until the migration to the new design is
> completed, including addressing the performance issue)
>
> I answered his concern about vagueness by explaining that I think
> there are ~10 at worst 20 such constants and that I don't know
> them all.
>
> Here are the ones I can find after a quick grepping:
>
> bzrlib.workingtree.worth_saving_limit
^- This is configurable. See bzrlib.workingtree_4._worth_saving_limit
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 08/24/2011 07:09 PM, Vincent Ladeuil wrote: workingtree. worth_saving_ limit
> I've discussed it with Martin, the disagreement was two-fold:
>
> - he felt the bug was too vague,
>
> - I felt setting the importance to low wasn't the right way to
> convey that the priority is low (which I agree with, I don't
> intend to fix it until the migration to the new design is
> completed, including addressing the performance issue)
>
> I answered his concern about vagueness by explaining that I think
> there are ~10 at worst 20 such constants and that I don't know
> them all.
>
> Here are the ones I can find after a quick grepping:
>
> bzrlib.
^- This is configurable. See bzrlib. workingtree_ 4._worth_ saving_ limit
> bzrlib. groupcompress. BATCH_SIZE groupcompress. GroupCompressVe rsionedFiles. _DEFAULT_ MAX_BYTES_ TO_INDEX
> bzrlib.
^- Similarly: groupcompress. _LazyGroupConte ntManager. _get_compressor _settings
bzrlib.
> bzrlib. lockdir. _DEFAULT_ TIMEOUT_ SECONDS lockdir. __DEFAULT_ POLL_SECONDS add._DEFAULT_ MAX_FILE_ SIZE
> bzrlib.
> bzrlib.
Similarly bzrlib. add.AddWithSkip LargeAction. skip_file
>
> I'm sure you know more, feel free to add them (we have time to
> list them before this bug is fixed anyway).
>
Most are already configurable when we felt it would be useful. We just
didn't have the new syntax to be able to do so.
John enigmail. mozdev. org/
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAk5 WVQ8ACgkQJdeBCY SNAAMCxQCeLjf68 xWHJ4TBTW6mF38v 29uN 9902sX0ye1VxbAz lm
OykAoMu75ZGuXai
=y7cO
-----END PGP SIGNATURE-----