Skip to content

fix namespace - #80

Merged
stbischof merged 1 commit into
eclipse-daanse:mainfrom
dbulahov:main141026
Oct 14, 2026
Merged

stbischof merged 1 commit into
eclipse-daanse:mainfrom
dbulahov:main141026

Conversation

@dbulahov

Copy link
Copy Markdown
Contributor

What does this PR do?

fix(io): write mddataset cube timestamps in the engine namespace

SSAS answers OlapInfo/CubeInfo/Cube with LastDataUpdate and
LastSchemaUpdate in http://schemas.microsoft.com/analysisservices/2003/engine,
declared on the element itself, while CubeName beside them stays in the
mddataset namespace. We wrote both timestamps in the mddataset namespace.

A value element can now name its own namespace through the daanse
annotation elementNamespace; EcoreXmlWriter declares it as the default
namespace on that element alone. ExtendedMetaData's namespace could not
carry this: every rowset column states one, and honouring it in general
would also change the ASSL ddl* extension elements in
DISCOVER_XML_METADATA. Only the two Cube timestamps are marked.

SSAS answers OlapInfo/CubeInfo/Cube with LastDataUpdate and
LastSchemaUpdate in http://schemas.microsoft.com/analysisservices/2003/engine,
declared on the element itself, while CubeName beside them stays in the
mddataset namespace. We wrote both timestamps in the mddataset namespace.

A value element can now name its own namespace through the daanse
annotation elementNamespace; EcoreXmlWriter declares it as the default
namespace on that element alone. ExtendedMetaData's namespace could not
carry this: every rowset column states one, and honouring it in general
would also change the ASSL ddl* extension elements in
DISCOVER_XML_METADATA. Only the two Cube timestamps are marked.
Signed-off-by: dbulahov <bulahovdenis@gmail.com>
@sonarqubecloud

Copy link
Copy Markdown

@stbischof
stbischof merged commit 157bae4 into eclipse-daanse:main Oct 14, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants