Added TECP to the Main RSS Feed – now RSS feed is blank

[Jen Russo]

My guess from error log is that the query is timing out trying to return tribe events.
https://cloudup.com/c5ZGxgwROXS

So, I’d open a ticket with tribe and point to that link, these are the customizations we have added:
https://gist.github.com/jb510/4a4e03abb0467fe7f71b

http://mauitime.com/feed/

[Geoff]

Hi there, Jen!

I’m seeing several jQuery and JS errors in the console when I inspect your feed URL. Are you able to clean those up to see if that makes a difference?

Cheers!
Geoff

[Jen Russo]

Geoff – This is Jon, one of Jen’s developers. Just to clarify, are you saying you’re seeing JS console errors on the RSS feed? Maybe I’m missing something but there is no JS loaded on the XML feed URL, it’s just an XML doc.

All I see is: https://cloudup.com/cfXr3Rv7nLx

[Geoff]

Hi there, Jon. Yeah, DevTools is seeing something there–but perhaps it’s not crucial if the source code itself isn’t referencing it. Strange it comes up, though.

Interestingly, I tried adding your snippet to my setup and it worked. It seems there is something else at play here that suggests a conflict coming from another source.

Geoff

[Jen Russo]

This reply is private.

[Jen Russo]

FYI.

On staging with all plugins disabled _except_:
1) that code to add events to RSS (in a core functionality plugin)
2) all TEC plugins

The feed still fails to generate 🙁

On staging with all plugins disabled _except:
1) all TEC plugins
ie. that code to add events to RSS (in the core functionality plugin) is commented out

The feed generate just fine for posts.

Hence, that code _is_ causing the problem.

[Geoff]

Do you have any other customizations in place on the calendar that could be causing the issue? Again, the code appears to work on my end with a clean install on Twenty Fourteen.

Cheers!
Geoff

[Jen Russo]

(This is Jon)

Geoff – I understand the code works on a clean 2014 install, but with what size data set? Do you have 1000’s of events and 1000’s of recurring events?

The code isn’t throwing PHP errors, I think MySQL is just timing out because that query would take minutes to run and would return thousands of results.

[Geoff]

Hi Jon, thanks for following up!

Yeah, I certainly do believe that a timeout is a solid possibility here considering we’re dealing with so many events.

I wonder if there is enough memory allocated to PHP. Have you tried increasing the limit? Some hosts do not allow you to set that limit so, if that’s the case, contacting them for assistance is a good route.

Let me know if that’s something you’ve already tried. 🙂

Geoff

[Jen Russo]

Hey Geoff –
We’re on WPE, but I tried cranking PHP memory to 512MB and PHP timeout to 300s. Still fails with a error log that reads:

[Mon Mar 16 21:39:02 2015] [error] [client 23.241.209.91] WordPress database error Lost connection to MySQL server during query for query SELECT SQL_CALC_FOUND_ROWS DISTINCT wp_posts.*, IF (wp_posts.post_type = 'tribe_events', wp_postmeta.meta_value, wp_posts.post_date) AS post_date, tribe_event_end_date.meta_value as EventEndDate FROM wp_posts LEFT JOIN wp_postmeta as wp_postmeta on wp_posts.ID = wp_postmeta.post_id AND wp_postmeta.meta_key = '_EventStartDate' LEFT JOIN wp_postmeta as tribe_event_end_date ON ( wp_posts.ID = tribe_event_end_date.post_id AND tribe_event_end_date.meta_key = '_EventEndDate' ) WHERE 1=1 AND wp_posts.post_type IN ('post', 'tribe_events') AND (wp_posts.post_status = 'publish') GROUP BY IF( wp_posts.post_parent = 0, wp_posts.ID, wp_posts.post_parent ) ORDER BY wp_posts.post_date DESC LIMIT 0, 50 /* From [mauitime.com/feed/] in [/nas/wp/www/cluster-10442/mauitime/index.php:17] */ made by require('wp-blog-header.php'), wp, WP->main, WP->query_posts, WP_Query->query, WP_Query->get_posts

I ran an EXPLAIN on that.

It’s querying 100,000+ rows. It’s not a PHP issue, it’s MySQL

Those LEFT JOINs are blowing things up, but I think their needed or you’d only get back event posts.

Seems like it might be more efficient to do something like getting the last 20 posts by date and last 20 events by date separately, then take those 2 arrays for 20 posts and merge them.

Postmeta just isn’t made for querying on like this and I’m not sure how it could be indexed.

[Geoff]

Hey Jon, thanks for following up and nice work! I do think you’re on to something here with MySQL and size of the query being run on post_meta.

I went ahead and logged a ticket for our team to discuss it further. There’s nothing I can do at the moment, but I know you have other threads and emails going on with others on the team around the same topic, so hopefully this will provide some good traction for a larger conversation on how and where events are stored in the tables.

I’m going to go ahead and close this thread for now, but please feel free to start another if other questions pop up and we’d be happy to help.

Cheers,
Geoff

[Leah]

Hi there,

Thanks again for your post. We wanted to get in touch and let you know that although we weren’t able to address this issue in our upcoming 3.10 release, it is still very much a priority. We have a ticket in our system and will be investigating a solution to be released in a future version. Thank you for your patience and support while we work on this!

Cheers,
The Events Calendar Team

[Author]

Posts