Re: Modifying join syntax generated by Cayenne's EJBQL

2016-02-28 Thread Andrus Adamchik
Yeah, you may open a Jira, and then reference Jira name in pull request. This will ensure all PR conversations are recorded in Jira. Andrus > On Feb 27, 2016, at 6:22 PM, Hugi Thordarson wrote: > > Sounds great :). Since it’s a simple fix I’d like to make this my first pull > request if that’

Re: Modifying join syntax generated by Cayenne's EJBQL

2016-02-27 Thread Hugi Thordarson
Sounds great :). Since it’s a simple fix I’d like to make this my first pull request if that’s okay? Should I start with a JIRA issue? Cheers, - hugi > On 26. feb. 2016, at 07:19, Andrus Adamchik wrote: > >> if I use the same expression with a regular SelectQuery, Cayenne puts “AND” >> be

Re: Modifying join syntax generated by Cayenne's EJBQL

2016-02-25 Thread Andrus Adamchik
> if I use the same expression with a regular SelectQuery, Cayenne puts “AND” > between the joins rather than commas and everything works fine. So this means AND works on other DBs and we should simply align Select transaltor with EJBQL translator. Maybe let's create a pull request for that and