We're doing a bunch of joins and other weirdness that is unnecessary.

CommentFileSizeAuthor
#20 feeds-db-queries-2531828-20.patch6.22 KBtwistor
#19 feeds-db-queries-2531828-19.patch5.99 KBtwistor
#17 feeds-db-queries-2531828-17.patch6.15 KBtwistor
#15 feeds-db-queries-2531828-15.patch6.27 KBtwistor
#13 feeds-db-queries-2531828-13.patch6.28 KBtwistor
#11 feeds-db-queries-2531828-11.patch6.3 KBtwistor
#9 feeds-db-queries-2531828-9.patch6.31 KBtwistor
#7 feeds-db-queries-2531828-7.patch6.31 KBtwistor
#5 feeds-db-queries-2531828-5.patch6.29 KBtwistor
#3 feeds-db-queries-2531828-3.patch6.24 KBtwistor
#1 feeds-db-queries-2531828-1.patch5.61 KBtwistor

Comments

twistor’s picture

twistor commented
Status: Active » Needs review
StatusFileSize
new feeds-db-queries-2531828-1.patch5.61 KB

Status: Needs review » Needs work

The last submitted patch, 1: feeds-db-queries-2531828-1.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-3.patch6.24 KB

Status: Needs review » Needs work

The last submitted patch, 3: feeds-db-queries-2531828-3.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-5.patch6.29 KB

Status: Needs review » Needs work

The last submitted patch, 5: feeds-db-queries-2531828-5.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-7.patch6.31 KB

Status: Needs review » Needs work

The last submitted patch, 7: feeds-db-queries-2531828-7.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-9.patch6.31 KB

Status: Needs review » Needs work

The last submitted patch, 9: feeds-db-queries-2531828-9.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-11.patch6.3 KB

Status: Needs review » Needs work

The last submitted patch, 11: feeds-db-queries-2531828-11.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-13.patch6.28 KB

Fine! I guess we can't be fancy with our join :(

Status: Needs review » Needs work

The last submitted patch, 13: feeds-db-queries-2531828-13.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-15.patch6.27 KB

Status: Needs review » Needs work

The last submitted patch, 15: feeds-db-queries-2531828-15.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-17.patch6.15 KB

I should really put down the laptop.

Status: Needs review » Needs work

The last submitted patch, 17: feeds-db-queries-2531828-17.patch, failed testing.

twistor’s picture

twistor commented
Status: Needs work » Needs review
StatusFileSize
new feeds-db-queries-2531828-19.patch5.99 KB
twistor’s picture

twistor commented
StatusFileSize
new feeds-db-queries-2531828-20.patch6.22 KB

  • twistor committed ba3f009 on 7.x-2.x 
    Issue #2531828 by twistor: Simplify db queries in FeedsProcessor
twistor’s picture

twistor commented

For those who find this issue. It really wasn't that complicated, I was trying to get the db api to do some odd things, but ended up just doing what field api does.

twistor’s picture

twistor commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.