This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 81893 - stackOverflowError when trying to rename a message part
Summary: stackOverflowError when trying to rename a message part
Status: CLOSED FIXED
Alias: None
Product: soa
Classification: Unclassified
Component: BPEL (show other bugs)
Version: 5.x
Hardware: All All
: P2 blocker (vote)
Assignee: Vladimir Yaroslavskiy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-08-04 01:01 UTC by lchang
Modified: 2006-08-28 10:37 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
StackFlowError (80.25 KB, application/octet-stream)
2006-08-04 01:02 UTC, lchang
Details
BPEL project with multiple bpels and wsdls (564.51 KB, application/octet-stream)
2006-08-04 17:10 UTC, lchang
Details

Note You need to log in before you can comment on or make changes to this bug.
Description lchang 2006-08-04 01:01:56 UTC
Steps to reproduce:

1.  create .wsdl from scratch.
2.  Add new message
3.  Select part1, and select refactor > rename
4.  In the window, enter a new name and click "Next"

You will get the attached stackflowerror after some time.
Comment 1 lchang 2006-08-04 01:02:15 UTC
Created attachment 32520 [details]
StackFlowError
Comment 2 Chris Webster 2006-08-04 01:51:53 UTC
The majority of the stack is in BPEL refactoring. Assigning to BPEL team for
evaluation. 
Comment 3 Vladimir Yaroslavskiy 2006-08-04 08:58:52 UTC
I tried the following:

1. Create Bpel module
2. Create wsdl file
3. Create new message
4. Invoke Refactor > Rename for part1
5. Press Next, weait, press Do Refactoring

Everything is ok, exception is not reproduced.
I tried with coke soa 060803_19 and coke beta 060803_7 builds.

Please verify on the latest build (soa and beta) and in case of exception,
please attached original project and describe detailed steps for reproducing.
Comment 4 lchang 2006-08-04 17:10:22 UTC
Created attachment 32552 [details]
BPEL project with multiple bpels and wsdls
Comment 5 lchang 2006-08-04 17:11:43 UTC
Attached is a BPEL module with many wsdls and bpels.  If you open this and add 
a new wsdl and follow the steps above to "refactor", you will get this 
stackFlowError.

I have lowered the priority to P3, since the workaround is:
1.  create bpel and wsdl in another BPEL Module, and copy/paste it over.
Comment 6 Vladimir Yaroslavskiy 2006-08-08 08:17:31 UTC
The problem is that QueryWithNamespace1.wsdl contains recursive definition of
addressType

<xsd:complexType name="addressType">
  <xsd:sequence>
    <xsd:element name="street" type="tns:addressType"/>
    <xsd:element name="city" type="xsd:string"/>
    <xsd:element name="state" type="xsd:string"/>
    <xsd:element name="zip" type="xsd:string"/>
  </xsd:sequence>
</xsd:complexType>     

Additional check should be done in bpel refactoring provider.
Comment 7 Michael Frisino 2006-08-08 11:12:00 UTC
Vladimir. Do you agree that workaround listed here is a valid workaround? I was
surprised to read this workaround. It does not seem like it would help. 

If this is not valid workaround, maybe we need to raise bug priority again.
Comment 8 Vladimir Yaroslavskiy 2006-08-08 12:39:20 UTC
Mike, what workaround do you mean? I suggested additional check - passed type
should be skipped, it keeps from infinity loop.
Comment 9 Vladimir Yaroslavskiy 2006-08-08 12:57:02 UTC
fixed.
Comment 10 Michael Frisino 2006-08-08 13:01:27 UTC
I meant the workaround listed by lchang:

"I have lowered the priority to P3, since the workaround is:
1.  create bpel and wsdl in another BPEL Module, and copy/paste it over."

She had originally created this bug as a P1 or P2 and lowered it to P3 because
of "workaround". But I am not sure workaround is workaround. 

The "workaround" is only significant for Beta since you have now fixed this in
release55.
Comment 11 Mikhail Kondratyev 2006-08-08 16:12:14 UTC
The workaround is not valid. Reopening the bug and setting priority back to P2

The problem is already fixed in trunk we suggest it to be fixed in beta branch.
Comment 12 Vladimir Yaroslavskiy 2006-08-08 16:16:40 UTC
will be fixed in cokebeta branch on 9 Aug 2006
Comment 13 Vladimir Yaroslavskiy 2006-08-09 07:57:25 UTC
fixed in coke beta.
Comment 14 lchang 2006-08-11 18:57:48 UTC
verified