Very good Andres, Congratulations for the great work.
Cheers 2011/7/4 Andres Taylor <[email protected]> > Obrigado Adriano! > > You are, of course, correct. My JVM inexperience shows... > > I think I've made the > changes< > https://github.com/neo4j/community/commit/e4622cbbcad8210babb01d861a5ef9df823c054b > >you > suggested. > > Thanks for your help! > > Andrés > > On Mon, Jul 4, 2011 at 4:35 PM, Adriano Henrique de Almeida < > [email protected]> wrote: > > > Hi, > > > > I've been doing some stuffs with the Cypher API on M4 and found a couple > > strange issues at the API, that I'd like to discuss. > > > > - CypherParser.parse("") method throws SyntaxError, which is a checked > > exception. 2 issues here. > > > > > > a) a checked exception called Error, seems weird and going against > > conventions. The name suggests it is an Error, thus, not recoverable. > > > > b) I don't know about you guys, but I think that a SyntaxError should be > > unchecked, I don't see it as recoverable. Take Hibernate API for > instance, > > if you write an invalid HQL, you'll get the exception, but without the > > extra > > work of having to handle the exception that should not be handled, the > > query > > should be fixed. Or maybe there is a scenario for it to be checked, but > I'm > > not aware of, I think the default scenario would be to not handle the > > exception. And BTW, of course, I could just wrap it inside an unchecked > > exception in my code. > > > > > > Cheers, > > > > -- > > Adriano Almeida > > Caelum | Ensino e Inovação > > www.caelum.com.br > > _______________________________________________ > > Neo4j mailing list > > [email protected] > > https://lists.neo4j.org/mailman/listinfo/user > > > _______________________________________________ > Neo4j mailing list > [email protected] > https://lists.neo4j.org/mailman/listinfo/user > -- Adriano Almeida Caelum | Ensino e Inovação www.caelum.com.br _______________________________________________ Neo4j mailing list [email protected] https://lists.neo4j.org/mailman/listinfo/user
