[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-06-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

Jon  changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 CC||jrob...@wikimedia.org
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-06-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #12 from Gerrit Notification Bot  ---
Change 134089 merged by jenkins-bot:
(bug 61939) Topic-titles not displaying in some topics (disappearing?)

https://gerrit.wikimedia.org/r/134089

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-05-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #11 from Gerrit Notification Bot  ---
Change 119304 abandoned by Matthias Mullie:
(bug 61939) Topic-titles not displaying in some topics (disappearing?)

Reason:
Up-to-date patch at https://gerrit.wikimedia.org/r/134089

https://gerrit.wikimedia.org/r/119304

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-05-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

Gerrit Notification Bot  changed:

   What|Removed |Added

 Status|RESOLVED|PATCH_TO_REVIEW
 Resolution|FIXED   |---

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-05-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #10 from Gerrit Notification Bot  ---
Change 134089 had a related patch set uploaded by Matthias Mullie:
(bug 61939) Topic-titles not displaying in some topics (disappearing?)

https://gerrit.wikimedia.org/r/134089

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-03-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

Maryana Pinchuk  changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #9 from Maryana Pinchuk  ---
The short-term issue looks to be fixed; moving the longer term solution stuff
to Trello: https://trello.com/c/4Oel8VTK/150-improve-caching

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-03-18 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #8 from Erik Bernhardson  ---
Dug into this with benny last night, fairly sure what happened is:


* Some data models were not in the cache
* Visiting a contributions page loaded cross-wiki data and cached it
* ExternalStore uses per-wiki databases, but flow doesn't take that into
account when querying
* As such the data that got cached included blank content


Things we should do:

As matthias mentioned, we need to either remove the bad data from the result
set or somehow mark that row as uncachable. 

We should probably also look into having a flowdb created on cluster25 and
cluster26 (the insert clusters for ES) and moving all our content over so we
have a valid cross-wiki content storage.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-03-18 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

Gerrit Notification Bot  changed:

   What|Removed |Added

 Status|NEW |PATCH_TO_REVIEW

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-03-18 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #7 from Gerrit Notification Bot  ---
Change 119304 had a related patch set uploaded by Matthias Mullie:
(bug 61939) Topic-titles not displaying in some topics (disappearing?)

https://gerrit.wikimedia.org/r/119304

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-03-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #6 from Quiddity  ---
New example at
https://en.wikipedia.org/w/index.php?title=Wikipedia_talk:Flow/Developer_test_page&topic_postId=rqjunzu26n44zemu&workflow=rqjunzu26n44zemu
in case that helps.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-03-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

Matthias Mullie  changed:

   What|Removed |Added

 CC||mmul...@wikimedia.org

--- Comment #5 from Matthias Mullie  ---
The last ones here seem ok to me. Only the original one still seems to have the
problem.

I've done some digging and I assume we failed to fetch the content from
ExternalStorage, and cached the incorrect result.

$ mwscript eval.php --wiki enwiki:

> $uuid = Flow\Model\UUID::create( 'rp2pfb6ete6pq6fg' );
> $collection = Flow\Model\PostCollection::newFromId( $uuid );
> $revision = $collection->getLastRevision();
> var_dump( $revision );

$revision->content seems to have a false value here. Judging from
AbstractRevision.php, the value should either be the full text or an empty
string ''.

Since the content is stored in ExternalStorage, it should be fetched via
RevisionStorage::mergeExternalContent, which will fetch the content via
ExternalStore::batchFetchFromURLs, which can return false. So I'm pretty sure
that's where the false comes from. And it looks like we're only caching the
value after that.

I've looked at clearing the cache, but has less luck there. It looks like the
key the revision is saved at should be
enwiki:flow_revision:v4:descendant:rp2pfb6ete6pq6fg:3.0 (I've also tried :pk:
instead of :descendant: and :2.0 instead of :3.0, but no luck). Trying to
either fetch those keys from cache returns a false, and trying to delete them
doesn't change anything. I may be assuming an incorrect cache key here, though.
Clearing the cache would only be a stopgap for this particular instance though,
and not properly fix the issue.

Any way; to properly fix this, we should (unless someone has a better idea):
* Make it possible to let our backend storage know to our cache-layers to not
cache certain data -> the proper fix IMO, but a fair bit of work

Inferior alternatives:
* Cache the real DB return (prior to ExternalStore::batchFetchFromURLs) and do
ExternalStore::batchFetchFromURLs afterwards -> probably the easiest fix, but
would result in a lot more calls to resolve data from ExternalStore.
* Implement an 2nd ExternalStore::batchFetchFromURLs in AbstractRevision, to
retry fetching content in case $revision->content is false -> easy too, but
duplicate code, and still unneccessary calls to ExternalStore for the duration
of the corrupt cache

Oh, and also: should we still have occurences of $revision->content being
false, we should probably make AbstractRevision::getContent return a 'We could
not fetch this content' message instead of displaying nothing.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-02-28 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

Maryana Pinchuk  changed:

   What|Removed |Added

   Priority|Normal  |High

--- Comment #4 from Maryana Pinchuk  ---
Oh, I see it here, too, now:
https://en.wikipedia.org/w/index.php?title=Wikipedia_talk:WikiProject_Breakfast&topic_postId=rpbc3zcjhrrwtaza&workflow=rpbc3zcjhrrwtaza

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-02-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #3 from Quiddity  ---
(In reply to Maryana Pinchuk from comment #2)
> Has this been observed anywhere other than that test page? 

Yes, but only in the second topic currently at
https://en.wikipedia.org/wiki/Wikipedia_talk:WikiProject_Hampshire 
specifically:
https://en.wikipedia.org/w/index.php?title=Wikipedia_talk:Flow/Developer_test_page&workflow=rpgca5e9yyrhoqq5

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-02-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

Maryana Pinchuk  changed:

   What|Removed |Added

   Priority|Unprioritized   |Normal

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-02-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #2 from Maryana Pinchuk  ---
Has this been observed anywhere other than that test page? If not, it's likely
some bit of CSS or JS added by a user in the course of testing that's
overriding Flow styles.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 61939] Flow: Topic-titles not displaying in some topics (disappearing?)

2014-02-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=61939

--- Comment #1 from Quiddity  ---
If we click "edit title", for any of them, it shows "false" in the edit-area.
eg.
https://en.wikipedia.org/w/index.php?title=Wikipedia_talk:Flow/Developer_test_page&topic_postId=royxd2frzdy34tcg&workflow=royxd2frzdy34tcg

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l