Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[566437] Use IllegalStateExceptionQuery where it is necessary #341

Merged
merged 1 commit into from Oct 9, 2020
Merged

[566437] Use IllegalStateExceptionQuery where it is necessary #341

merged 1 commit into from Oct 9, 2020

Conversation

lredor
Copy link
Contributor

@lredor lredor commented Aug 27, 2020

These cases have been detected using Team For Capella after a shutdown
of the server with a session opened.

Bug: 566437
Change-Id: I9a4a3e292a0024efb9fdf7f13e1ce2c030003e0b
Signed-off-by: Laurent Redor laurent.redor@obeo.fr

@eclipse-capella-bot
Copy link

🚀 Build master-PR-341-1 started!

@eclipse-capella-bot
Copy link

😞 Build master-PR-341-1 failed!

pdulth
pdulth requested changes Aug 27, 2020
View reviewed changes
Copy link
Contributor

@pdulth pdulth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you look at your modified files, it seems that tab are used instead of spaces in the catch part.
otherwise looks ok

@lredor
Copy link
Contributor Author

lredor commented Aug 27, 2020

can you look at your modified files, it seems that tab are used instead of spaces in the catch part.
otherwise looks ok

I will look (the new change does not correspond to your remark but to a new discover case)

@eclipse-capella-bot
Copy link

🚀 Build master-PR-341-2 started!

@eclipse-capella-bot
Copy link

😞 Build master-PR-341-2 failed!

@eclipse-capella-bot
Copy link

🚀 Build master-PR-341-3 started!

@eclipse-capella-bot
Copy link

😞 Build master-PR-341-3 failed!

@lredor lredor requested a review from pdulth August 28, 2020 15:49
@eclipse-capella-bot
Copy link

🚀 Build master-PR-341-4 started!

@eclipse-capella-bot
Copy link

😞 Build master-PR-341-4 failed!

@eclipse-capella-bot
Copy link

🚀 Build master-PR-341-5 started!

@eclipse-capella-bot
Copy link

😞 Build master-PR-341-5 failed!

@lredor @pdulth
[566437] Use IllegalStateExceptionQuery where it is necessary
70f7972 
These cases have been detected using Team For Capella after a shutdown
of the server with a session opened.

Bug: 566437
Change-Id: I9a4a3e292a0024efb9fdf7f13e1ce2c030003e0b
Signed-off-by: Laurent Redor <laurent.redor@obeo.fr>
@eclipse-capella-bot
Copy link

🚀 Build master-PR-341-6 started!

@sonarcloud
Copy link

sonarcloud bot commented Oct 2, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@eclipse-capella-bot
Copy link

😟 Build master-PR-341-6 is unstable! The product is available here.

@pdulth pdulth merged commit d62b55e into master Oct 9, 2020
@pdulth pdulth deleted the 566437 branch October 9, 2020 13:49
@eclipse-capella-bot eclipse-capella-bot mentioned this pull request Jan 7, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pdulth pdulth pdulth approved these changes

Assignees
No one assigned
Labels
build-unstable
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@lredor @eclipse-capella-bot @pdulth