Github user franz1981 commented on the issue:
https://github.com/apache/activemq-artemis/pull/1866
@graben @mtaylor was the right person that knows why the ID was configured
in that way; I will check on the Oracle 12c doc about it in order to give you
feedback ASAP :+1:
---
Github user graben commented on the issue:
https://github.com/apache/activemq-artemis/pull/1866
@mtaylor: Any feedback on this?
---
Github user jmesnil commented on the issue:
https://github.com/apache/activemq-artemis/pull/1866
FWIW, I copied the SQL statement from Artemis 2.4.0
Github user franz1981 commented on the issue:
https://github.com/apache/activemq-artemis/pull/1866
@graben
> Well, I'm actually not able to test against oracle 12 only have a oracle
11 express instance
It would be enough: it is just to validate the changes.
>
Github user graben commented on the issue:
https://github.com/apache/activemq-artemis/pull/1866
Well, I'm actually not able to test against oracle 12 only have a oracle 11
express instance. But a it is obvious that the autoincrement of id is wrong,
since it is 1st managed by the
Github user franz1981 commented on the issue:
https://github.com/apache/activemq-artemis/pull/1866
@graben
> Please run the JDBC tests with Oracle 12 (now possible using the right
sys properties)
Currently we're not supporting Oracle into the Jenkins CI hence please
Github user graben commented on the issue:
https://github.com/apache/activemq-artemis/pull/1866
@franz1981 : sorry I don't understand what you would like to tell? Do you
think the title is wrong? Do you think my patch is wrong?
---
Github user franz1981 commented on the issue:
https://github.com/apache/activemq-artemis/pull/1866
Please run the JDBC tests with Oracle 12 (now possible using the right sys
properties) and fix the PR tile/message
---