qtorganizer5-eds

Merge lp://qastaging/~renatofilho/qtorganizer5-eds/thread-parse into lp://qastaging/qtorganizer5-eds

thread-parse
Merge into trunk

Proposed by Renato Araujo Oliveira Filho on 2015-04-10

Status:	Merged
Approved by:	Bill Filler on 2016-03-07
Approved revision:	98
Merged at revision:	91
Proposed branch:	lp://qastaging/~renatofilho/qtorganizer5-eds/thread-parse
Merge into:	lp://qastaging/qtorganizer5-eds
Prerequisite:	lp://qastaging/~renatofilho/qtorganizer5-eds/filter-collection-query
Diff against target:	785 lines (+359/-34) 13 files modified organizer/CMakeLists.txt (+2/-0) organizer/qorganizer-eds-engine.cpp (+42/-21) organizer/qorganizer-eds-engine.h (+9/-0) organizer/qorganizer-eds-fetchrequestdata.cpp (+59/-8) organizer/qorganizer-eds-fetchrequestdata.h (+25/-1) organizer/qorganizer-eds-parseeventthread.cpp (+48/-0) organizer/qorganizer-eds-parseeventthread.h (+61/-0) organizer/qorganizer-eds-requestdata.cpp (+15/-1) organizer/qorganizer-eds-requestdata.h (+6/-1) tests/unittest/cancel-operation-test.cpp (+2/-0) tests/unittest/eds-base-test.cpp (+1/-0) tests/unittest/fetchitem-test.cpp (+6/-2) tests/unittest/parseecal-test.cpp (+83/-0)
To merge this branch:	bzr merge lp://qastaging/~renatofilho/qtorganizer5-eds/thread-parse
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
PS Jenkins bot	continuous-integration		Approve on 2016-01-11
Ubuntu Phablet Team		2015-04-10	Pending
Review via email: mp+255873@code.qastaging.launchpad.net

Commit message

Implemented event parse EDS -> QOrganizer in a thread to avoid main thread be blocked.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-04-10:

#

FAILED: Continuous integration, rev:90
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/211/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-amd64-ci/18/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/18/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-i386-ci/18/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/211/rebuild

review: Needs Fixing (continuous-integration)

lp://qastaging/~renatofilho/qtorganizer5-eds/thread-parse updated on 2015-04-10

91. By Renato Araujo Oliveira Filho on 2015-04-10: Added minssing files.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-04-10:

#

FAILED: Continuous integration, rev:91
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/212/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-amd64-ci/19
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/19
        deb: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/19/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-i386-ci/19/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/212/rebuild

review: Needs Fixing (continuous-integration)

lp://qastaging/~renatofilho/qtorganizer5-eds/thread-parse updated on 2015-04-11

92. By Renato Araujo Oliveira Filho on 2015-04-11

Make sure to delete request-data only after emit the finish signal.

We need to wait for the parse thread before emit the signal.

93. By Renato Araujo Oliveira Filho on 2015-04-11

Make sure that the request data is not destroyed immediately after call finish function.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-04-11:

#

FAILED: Continuous integration, rev:92
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/213/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-amd64-ci/20/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/20/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-i386-ci/20/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/213/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-04-11:

#

FAILED: Continuous integration, rev:93
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/214/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-amd64-ci/21/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/21
        deb: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/21/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-i386-ci/21

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/214/rebuild

review: Needs Fixing (continuous-integration)

lp://qastaging/~renatofilho/qtorganizer5-eds/thread-parse updated on 2015-04-13

94. By Renato Araujo Oliveira Filho on 2015-04-13: Added debug helper functions.
95. By Renato Araujo Oliveira Filho on 2015-04-13: Code cleanup.
96. By Renato Araujo Oliveira Filho on 2015-04-13: Added check to make sure that all request get destroyed.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-04-13:

#

FAILED: Continuous integration, rev:96
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/215/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-amd64-ci/22/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/22/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-i386-ci/22

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/215/rebuild

review: Needs Fixing (continuous-integration)

lp://qastaging/~renatofilho/qtorganizer5-eds/thread-parse updated on 2015-05-13

97. By Renato Araujo Oliveira Filho on 2015-05-13: Parent merged.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-05-13:

#

FAILED: Continuous integration, rev:97
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/218/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-amd64-ci/25/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/25
        deb: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/25/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-i386-ci/25

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/218/rebuild

review: Needs Fixing (continuous-integration)

lp://qastaging/~renatofilho/qtorganizer5-eds/thread-parse updated on 2016-01-11

98. By Renato Araujo Oliveira Filho on 2016-01-11: Parent merged.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2016-01-11:

#

PASSED: Continuous integration, rev:98
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/240/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-amd64-ci/47
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/47
        deb: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/47/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-i386-ci/47

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/240/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes:

Renato Araujo Oliveira Filho

Ubuntu Phablet Team