[Mondrian] Re: ParserSym error in 184.108.40.20671
jhyde at pentaho.com
Mon Oct 6 12:52:52 EDT 2008
I introduced that test when I fixed mondrian bug 2046318,
35302&atid=414613> olap4j driver should throw OlapException on validate
It is fixed in mondrian, just not released yet.
From: mondrian-bounces at pentaho.org [mailto:mondrian-bounces at pentaho.org] On
Behalf Of Luc Boudreau
Sent: Monday, October 06, 2008 9:30 AM
To: mondrian at pentaho.org
Subject: [Mondrian] Re: ParserSym error in 220.127.116.1171
Nevermind that. I got the source from the distribution instead of directly
from perforce and now it works.
But now, I confirmed that there were some changes in this release that
affects the Olap4j test suite. The Mondrian Olap4j driver throws
MondrianException where an OlapException is expected by the API. This
happens in Olap4j's ConnectionTest.testValidateError, line 1787. It waits
for an OlapException but gets a MondrianException instead. Since
MondrianException is a subclass of RuntimeException, it gets through all
Olap4jMondrianDriver error handling facilities right back to the end-user.
This is a no-no and an infringement of the current Olap4j API.
The violation exact point is in MondrianOlap4jonnection.validateSelect(),
line 614. The method signature says it throws only OlapExceptions, but as I
said before, the call to parseQuery() at line 614 triggers a subclass of
RuntimeException to be thrown, thus violating the API.
This is reproductible with Mondrian 3.0.4 and Olap4j head or whatever
recision you want as a matter of fact.
On Mon, Oct 6, 2008 at 11:59 AM, Luc Boudreau <lucboudreau at gmail.com> wrote:
I'm trying to build Mondrian changeset 18.104.22.16871 from source but i get
"ParserSym.BANG cannot be resolved" in
mondrian-3.0/src/main/mondrian/olap/Scanner.java, line 721.
Since this file is generated at build time, i'm asking the developer's list.
What did I do wrong?! Howcome it didn't get generated properly? What are the
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Mondrian