Deprecate twisted.persisted.journal

[Thijs Triemstra]

twisted.persisted.journal is one of the not-quite-reliable storage systems still floating around from Twisted's sordid past. No one has any intention of finishing it (clearly, as it's been 4 years since its last change), and by current standards we probably wouldn't want it in Twisted anyway due to how far afield from Twisted's primary goals it is.

It should be deprecated so we can remove it in a few more releases.

[Itamar Turner-Trauring]

I'd rather not spend typing time on this, but it's certainly worth doing.

[Drew Smathers]

Unit test from first patch is stupid and causes breakage in test suite anyhow. I'm not sure how to correctly test deprecation warning in module import or if this is even necessary.

[Thijs Triemstra]

This is blocked by #4111 and doesn't need a higher priority, apps aren't breaking or anything.

[Jean-Paul Calderone]

Thanks! Some simple comments:

1. Missing docstrings on the test class and test method
2. The test will fail if it is run twice in one process. You could either:
   1. pop it out of sys.modules before the test runs
   2. deprecate the journal attribute of twisted.persisted instead, using deprecateModuleAttribute (although I've never done it this way and I don't know what will happen).

[Allister MacLeod]

(In [28690]) Branching to 'deprecatepersistedjournal-4298'

[Allister MacLeod]

Updated changes in [28590:28694].

[Allister MacLeod]

Er.. I mean [28690:28694].

[Allister MacLeod]

Here's a better revision link: ​http://twistedmatrix.com/trac/changeset?new=/branches/deprecatepersistedjournal-4298@28694&old=/branches/deprecatepersistedjournal-4298@28690

[Allister MacLeod]

Ready for re-review.

[Ying Li]

Deprecation should be "as of 10.1", since the current version of Twisted is 10.0.

Everything else seems to be great though, and the tests pass. This can probably be merged into trunk after that change.

[Allister MacLeod]

Incremented deprecated-at version in r28714

[Ying Li]

Sorry, I should have spotted this last time, but it's missing an entry in the relevant topfile directory listing what change this branch will make (see ​http://twistedmatrix.com/trac/wiki/ReviewProcess#Newsfiles).

[Allister MacLeod]

Thanks for the heads-up cyli. I will add a news file next time I'm home and have free time.

[Thijs Triemstra]

Thanks. Could you also:

• update the copyright in twisted/persisted/journal/init.py
• use 2010 instead of 2001-2010 in the new test_journal.py

Merge after that.

[Glyph]

Poke, poke, amacleod: this is almost done! Are you going to take it across the finish line or what? :)

[Allister MacLeod]

Finally got off my duff and did it: [29264].

[Jean-Paul Calderone]

Hi amacleod,

Please read ​http://twistedmatrix.com/trac/wiki/ReviewProcess before your next trunk commit. :)

[Jean-Paul Calderone]

This was reverted in r29265 because it was missing a news file. Should be easy for anyone to finish up.

[Ying Li]

(In [30451]) Branching to 're-deprecate-persisted-journal-4298'

[Ying Li]

(In [30458]) Changed some spacing and whitespace to comply with coding standard.

refs #4298

[Ying Li]

Oops - majority of the changes were in ​r30457

[Ying Li]

Here are some ​build results

[Thijs Triemstra]

Merge after updating the copyright in twisted/test/test_journal.

[Thijs Triemstra]

Replying to thijs:

It should be deprecated so we can remove it in a few more releases.

Opened #4805 for removing it in a future release.

[Ying Li]

Replying to thijs:

Merge after updating the copyright in twisted/test/test_journal.

I did so in ​r30458. It's just when I entered the commit message for ​r30457, I used the word "ref" instead of "refs", so trac didn't pick it up. Hence I linked to ​r30457 manually, but ​r30458 has some additional changes.

Anyway, thanks for the review. Merging. :)

[Ying Li]

(In [30460]) Merge re-deprecate-persisted-journal-4298: Deprecate twisted.persisted.journal

Author: djfroofy, amacleod, cyli

Reviewer: thijs, exarkun, cyli

Fixes: #4298

Re-deprecate twisted.persisted.journal as of 11.0, after reversion (of deprecation as of 10.1) due to lack of newsfile.