Re: [Wikitech-l] RevisionInsertComplete vs. RevisionRecordInserted

2018-02-01 Thread Stas Malyshev
Hi! On 2/1/18 7:39 AM, Andrew Otto wrote: > This is the first I’ve heard of it! So, we don’t have a plan to change it, > but I suppose we should if RevisionInsertComplete is deprecated. I haven’t > looked at RevisionRecordInserted yet so I can’t answer questions about > schema changes, but I

Re: [Wikitech-l] RevisionInsertComplete vs. RevisionRecordInserted

2018-02-01 Thread Andrew Otto
This is the first I’ve heard of it! So, we don’t have a plan to change it, but I suppose we should if RevisionInsertComplete is deprecated. I haven’t looked at RevisionRecordInserted yet so I can’t answer questions about schema changes, but I doubt it would change anything. Just created

[Wikitech-l] RevisionInsertComplete vs. RevisionRecordInserted

2018-01-29 Thread Stas Malyshev
Hi! I've noticed that RevisionInsertComplete hook is now deprecated in favor of RevisionRecordInserted. However, EventBus still uses RevisionInsertComplete. Is this going to change soon? If so, will the underlying event/topic change too? I couldn't find anything in Phabricator about this - is