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 81735 - Code review AXIOM Sync
Summary: Code review AXIOM Sync
Status: RESOLVED FIXED
Alias: None
Product: xml
Classification: Unclassified
Component: Schema Tools (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: Samaresh Panda
URL:
Keywords:
Depends on:
Blocks: 81731
  Show dependency tree
 
Reported: 2006-08-02 19:59 UTC by Chris Webster
Modified: 2006-11-15 21:14 UTC (History)
1 user (show)

See Also:
Issue Type: TASK
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Webster 2006-08-02 19:59:07 UTC 
Host code review.
Comment 1 Samaresh Panda 2006-08-23 21:44:16 UTC 
Least dependent content models need be synced first.
Checking in AXIModelUpdater.java; new revision: 1.1.2.18;
Comment 2 Samaresh Panda 2006-08-29 19:32:25 UTC 
ContentModel can potentially have element refs, in which case the global element
should be synced first.

Checking in AXIModelUpdater.java; new revision: 1.1.2.21;
Comment 3 Samaresh Panda 2006-09-01 21:23:42 UTC 
Proxy should play no role during sync. The new sync algorithm syncs all global
element and attributes. Then all content models.

Checking in AXIComponent.java;
new revision: 1.1.2.57; previous revision: 1.1.2.56
Checking in impl/AXIModelBuilder.java;
new revision: 1.1.2.23; previous revision: 1.1.2.22
Checking in impl/ElementImpl.java;
new revision: 1.1.2.32; previous revision: 1.1.2.31
Checking in impl/ElementRef.java;
new revision: 1.1.2.22; previous revision: 1.1.2.21
Checking in impl/AXIModelImpl.java;
new revision: 1.1.2.29; previous revision: 1.1.2.28
Checking in impl/AttributeImpl.java;
new revision: 1.1.2.14; previous revision: 1.1.2.13
Checking in impl/Util.java;
new revision: 1.1.2.58; previous revision: 1.1.2.57
Checking in impl/AXIModelUpdater.java;
new revision: 1.1.2.24;
Comment 4 Samaresh Panda 2006-09-07 09:03:10 UTC 
Multifile sync was broken:
AXIModelUpdater.java: new revision: 1.1.2.27;
Adding new unit test MultiFileSyncTest.java: new revision: 1.1.2.1;
Comment 5 Samaresh Panda 2006-11-14 07:18:18 UTC 
This was created for coke release and all review items have been fixed and
integrated.
Comment 6 Samaresh Panda 2006-11-14 07:20:15 UTC 
Looks like we want to keep this open for 6.0 as well.
Comment 7 Samaresh Panda 2006-11-14 07:22:25 UTC 
Marking it fixed. Will create separate issue for 6.0
Comment 8 Samaresh Panda 2006-11-15 21:14:43 UTC 
These changes were made in release55 during development and hence all branches
have these changes.