[jira] [Updated] (JAMES-2929) JDBCMailRepository broken: SQL and JDBC code out of sync

2019-10-22 Thread ConSol


 [ 
https://issues.apache.org/jira/browse/JAMES-2929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jörg Thomas (ConSol) updated JAMES-2929:

Description: 
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} and 
{{dockerfiles/run/spring/destination/conf/sqlResources.xml}} were left 
unmodified
 ** [EDIT] `sqlResources-mail.xml` need to be changed as well
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which -I will bring up when the above issues are 
adressed.- is already filed here under 
[JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].

  was:
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} and 
{{dockerfiles/run/spring/destination/conf/sqlResources.xml}} were left 
unmodified
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which -I will bring up when the above issues are 
adressed.- is already filed here under 
[JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].


> JDBCMailRepository broken: SQL and JDBC code out of sync
> 
>
> Key: JAMES-2929
> URL: https://issues.apache.org/jira/browse/JAMES-2929
> Project: James Server
>  Issue Type: Bug
>  Components: JDBC, MailStore & MailRepository
>Affects Versions: 3.2.0, 3.3.0, 3.4.0, 3.3.1
>Reporter: Jörg Thomas (ConSol)
>Priority: Major
> Fix For: 3.5.0
>
>
> The table structure as defined in {{sqlResources.xml}} and the corresponding 
> JDBC code in 
> {{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
>  have diverged since the fix for 
> [JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
> issues:
>  * {{server/app/src/main/resources/sqlResources.xml}} and 
> {{dockerfiles/run/spring/destination/conf/sqlResources.xml}} were left 
> unmodified
>  ** [EDIT] `sqlResources-mail.xml` need to be changed as well
>  * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
> is wrong:
>  ** message_body (SQL index: 9, JDBC index: 11)
>  ** message_attributes (SQL index: 10, JDBC index: 12)
>  ** per_recipient_headers (SQL index: 11, JDBC index: 9)
>  ** last_updated timestamp (SQL index: 12, JDBC index: 10)
>  * Since this went completely unnoticed for quite some time, adding tests 
> would be appropriate?
> There is a fourth issue which -I will bring up when the above issues are 
> adressed.- is already filed here under 
> [JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



[jira] [Updated] (JAMES-2929) JDBCMailRepository broken: SQL and JDBC code out of sync

2019-10-22 Thread ConSol


 [ 
https://issues.apache.org/jira/browse/JAMES-2929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jörg Thomas (ConSol) updated JAMES-2929:

Description: 
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * these SQL resources were left unmodified:
 ** {{dockerfiles/run/spring/destination/conf/sqlResources.xml}}
 ** {{server/app/src/main/resources/sqlResources.xml}}
 ** {{server/data/data-jdbc/src/test/resources/sqlResources-mail.xml}}
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which -I will bring up when the above issues are 
adressed.- is already filed here under 
[JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].

  was:
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} and 
{{dockerfiles/run/spring/destination/conf/sqlResources.xml}} were left 
unmodified
 ** [EDIT] `sqlResources-mail.xml` need to be changed as well
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which -I will bring up when the above issues are 
adressed.- is already filed here under 
[JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].


> JDBCMailRepository broken: SQL and JDBC code out of sync
> 
>
> Key: JAMES-2929
> URL: https://issues.apache.org/jira/browse/JAMES-2929
> Project: James Server
>  Issue Type: Bug
>  Components: JDBC, MailStore & MailRepository
>Affects Versions: 3.2.0, 3.3.0, 3.4.0, 3.3.1
>Reporter: Jörg Thomas (ConSol)
>Priority: Major
> Fix For: 3.5.0
>
>
> The table structure as defined in {{sqlResources.xml}} and the corresponding 
> JDBC code in 
> {{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
>  have diverged since the fix for 
> [JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
> issues:
>  * these SQL resources were left unmodified:
>  ** {{dockerfiles/run/spring/destination/conf/sqlResources.xml}}
>  ** {{server/app/src/main/resources/sqlResources.xml}}
>  ** {{server/data/data-jdbc/src/test/resources/sqlResources-mail.xml}}
>  * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
> is wrong:
>  ** message_body (SQL index: 9, JDBC index: 11)
>  ** message_attributes (SQL index: 10, JDBC index: 12)
>  ** per_recipient_headers (SQL index: 11, JDBC index: 9)
>  ** last_updated timestamp (SQL index: 12, JDBC index: 10)
>  * Since this went completely unnoticed for quite some time, adding tests 
> would be appropriate?
> There is a fourth issue which -I will bring up when the above issues are 
> adressed.- is already filed here under 
> [JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



[jira] [Updated] (JAMES-2929) JDBCMailRepository broken: SQL and JDBC code out of sync

2019-10-22 Thread ConSol


 [ 
https://issues.apache.org/jira/browse/JAMES-2929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jörg Thomas (ConSol) updated JAMES-2929:

Description: 
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} and 
{{dockerfiles/run/spring/destination/conf/sqlResources.xml}} were left 
unmodified
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which -I will bring up when the above issues are 
adressed.- is already filed here under 
[JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].

  was:
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which -I will bring up when the above issues are 
adressed.- is already filed here under 
[JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].


> JDBCMailRepository broken: SQL and JDBC code out of sync
> 
>
> Key: JAMES-2929
> URL: https://issues.apache.org/jira/browse/JAMES-2929
> Project: James Server
>  Issue Type: Bug
>  Components: JDBC, MailStore & MailRepository
>Affects Versions: 3.2.0, 3.3.0, 3.4.0, 3.3.1
>Reporter: Jörg Thomas (ConSol)
>Priority: Major
> Fix For: 3.5.0
>
>
> The table structure as defined in {{sqlResources.xml}} and the corresponding 
> JDBC code in 
> {{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
>  have diverged since the fix for 
> [JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
> issues:
>  * {{server/app/src/main/resources/sqlResources.xml}} and 
> {{dockerfiles/run/spring/destination/conf/sqlResources.xml}} were left 
> unmodified
>  * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
> is wrong:
>  ** message_body (SQL index: 9, JDBC index: 11)
>  ** message_attributes (SQL index: 10, JDBC index: 12)
>  ** per_recipient_headers (SQL index: 11, JDBC index: 9)
>  ** last_updated timestamp (SQL index: 12, JDBC index: 10)
>  * Since this went completely unnoticed for quite some time, adding tests 
> would be appropriate?
> There is a fourth issue which -I will bring up when the above issues are 
> adressed.- is already filed here under 
> [JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



[jira] [Updated] (JAMES-2929) JDBCMailRepository broken: SQL and JDBC code out of sync

2019-10-22 Thread ConSol


 [ 
https://issues.apache.org/jira/browse/JAMES-2929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jörg Thomas (ConSol) updated JAMES-2929:

Description: 
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which -I will bring up when the above issues are 
adressed.- is already filed here under 
[JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].

  was:
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which I will bring up when the above issues are 
adressed.


> JDBCMailRepository broken: SQL and JDBC code out of sync
> 
>
> Key: JAMES-2929
> URL: https://issues.apache.org/jira/browse/JAMES-2929
> Project: James Server
>  Issue Type: Bug
>  Components: JDBC, MailStore & MailRepository
>Affects Versions: 3.2.0, 3.3.0, 3.4.0, 3.3.1
>Reporter: Jörg Thomas (ConSol)
>Priority: Major
> Fix For: 3.5.0
>
>
> The table structure as defined in {{sqlResources.xml}} and the corresponding 
> JDBC code in 
> {{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
>  have diverged since the fix for 
> [JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
> issues:
>  * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
>  * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
> is wrong:
>  ** message_body (SQL index: 9, JDBC index: 11)
>  ** message_attributes (SQL index: 10, JDBC index: 12)
>  ** per_recipient_headers (SQL index: 11, JDBC index: 9)
>  ** last_updated timestamp (SQL index: 12, JDBC index: 10)
>  * Since this went completely unnoticed for quite some time, adding tests 
> would be appropriate?
> There is a fourth issue which -I will bring up when the above issues are 
> adressed.- is already filed here under 
> [JAMES-2828|https://issues.apache.org/jira/browse/JAMES-2828].



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



[jira] [Updated] (JAMES-2929) JDBCMailRepository broken: SQL and JDBC code out of sync

2019-10-22 Thread ConSol


 [ 
https://issues.apache.org/jira/browse/JAMES-2929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jörg Thomas (ConSol) updated JAMES-2929:

Description: 
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)

Since this went completely unnoticed for quite some time, adding tests would be 
appropriate?

There is a third issue which I will bring up when these first two fixes are 
done.

  was:
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * `server/app/src/main/resources/sqlResources.xml` was left unmodified
 * the order in `server/data/data-jdbc/src/test/resources/sqlResources.xml` is 
wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)

Since this went completely unnoticed for quite some time, adding tests would be 
appropriate?

There is a third issue which I will bring up when these first two fixes are 
done.


> JDBCMailRepository broken: SQL and JDBC code out of sync
> 
>
> Key: JAMES-2929
> URL: https://issues.apache.org/jira/browse/JAMES-2929
> Project: James Server
>  Issue Type: Bug
>  Components: JDBC, MailStore & MailRepository
>Affects Versions: 3.2.0, 3.3.0, 3.4.0, 3.3.1
>Reporter: Jörg Thomas (ConSol)
>Priority: Major
> Fix For: 3.5.0
>
>
> The table structure as defined in {{sqlResources.xml}} and the corresponding 
> JDBC code in 
> {{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
>  have diverged since the fix for 
> [JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
> issues:
>  * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
>  * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
> is wrong:
>  ** message_body (SQL index: 9, JDBC index: 11)
>  ** message_attributes (SQL index: 10, JDBC index: 12)
>  ** per_recipient_headers (SQL index: 11, JDBC index: 9)
>  ** last_updated timestamp (SQL index: 12, JDBC index: 10)
> Since this went completely unnoticed for quite some time, adding tests would 
> be appropriate?
> There is a third issue which I will bring up when these first two fixes are 
> done.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



[jira] [Updated] (JAMES-2929) JDBCMailRepository broken: SQL and JDBC code out of sync

2019-10-22 Thread ConSol


 [ 
https://issues.apache.org/jira/browse/JAMES-2929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jörg Thomas (ConSol) updated JAMES-2929:

Description: 
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)
 * Since this went completely unnoticed for quite some time, adding tests would 
be appropriate?

There is a fourth issue which I will bring up when the above issues are 
adressed.

  was:
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
 * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
is wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)

Since this went completely unnoticed for quite some time, adding tests would be 
appropriate?

There is a third issue which I will bring up when these first two fixes are 
done.


> JDBCMailRepository broken: SQL and JDBC code out of sync
> 
>
> Key: JAMES-2929
> URL: https://issues.apache.org/jira/browse/JAMES-2929
> Project: James Server
>  Issue Type: Bug
>  Components: JDBC, MailStore & MailRepository
>Affects Versions: 3.2.0, 3.3.0, 3.4.0, 3.3.1
>Reporter: Jörg Thomas (ConSol)
>Priority: Major
> Fix For: 3.5.0
>
>
> The table structure as defined in {{sqlResources.xml}} and the corresponding 
> JDBC code in 
> {{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
>  have diverged since the fix for 
> [JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
> issues:
>  * {{server/app/src/main/resources/sqlResources.xml}} was left unmodified
>  * the order in {{server/data/data-jdbc/src/test/resources/sqlResources.xml}} 
> is wrong:
>  ** message_body (SQL index: 9, JDBC index: 11)
>  ** message_attributes (SQL index: 10, JDBC index: 12)
>  ** per_recipient_headers (SQL index: 11, JDBC index: 9)
>  ** last_updated timestamp (SQL index: 12, JDBC index: 10)
>  * Since this went completely unnoticed for quite some time, adding tests 
> would be appropriate?
> There is a fourth issue which I will bring up when the above issues are 
> adressed.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



[jira] [Updated] (JAMES-2929) JDBCMailRepository broken: SQL and JDBC code out of sync

2019-10-22 Thread ConSol


 [ 
https://issues.apache.org/jira/browse/JAMES-2929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jörg Thomas (ConSol) updated JAMES-2929:

Description: 
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
issues:
 * `server/app/src/main/resources/sqlResources.xml` was left unmodified
 * the order in `server/data/data-jdbc/src/test/resources/sqlResources.xml` is 
wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)

Since this went completely unnoticed for quite some time, adding tests would be 
appropriate?

There is a third issue which I will bring up when these first two fixes are 
done.

  was:
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303](https://issues.apache.org/jira/browse/JAMES-2303). There's three 
issues:
 * `server/app/src/main/resources/sqlResources.xml` was left unmodified
 * the order in `server/data/data-jdbc/src/test/resources/sqlResources.xml` is 
wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)

Since this went completely unnoticed for quite some time, adding tests would be 
appropriate?

There is a third issue which I will bring up when these first two fixes are 
done.


> JDBCMailRepository broken: SQL and JDBC code out of sync
> 
>
> Key: JAMES-2929
> URL: https://issues.apache.org/jira/browse/JAMES-2929
> Project: James Server
>  Issue Type: Bug
>  Components: JDBC, MailStore & MailRepository
>Affects Versions: 3.2.0, 3.3.0, 3.4.0, 3.3.1
>Reporter: Jörg Thomas (ConSol)
>Priority: Major
> Fix For: 3.5.0
>
>
> The table structure as defined in {{sqlResources.xml}} and the corresponding 
> JDBC code in 
> {{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
>  have diverged since the fix for 
> [JAMES-2303|https://issues.apache.org/jira/browse/JAMES-2303]. There's three 
> issues:
>  * `server/app/src/main/resources/sqlResources.xml` was left unmodified
>  * the order in `server/data/data-jdbc/src/test/resources/sqlResources.xml` 
> is wrong:
>  ** message_body (SQL index: 9, JDBC index: 11)
>  ** message_attributes (SQL index: 10, JDBC index: 12)
>  ** per_recipient_headers (SQL index: 11, JDBC index: 9)
>  ** last_updated timestamp (SQL index: 12, JDBC index: 10)
> Since this went completely unnoticed for quite some time, adding tests would 
> be appropriate?
> There is a third issue which I will bring up when these first two fixes are 
> done.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



[jira] [Updated] (JAMES-2929) JDBCMailRepository broken: SQL and JDBC code out of sync

2019-10-22 Thread ConSol


 [ 
https://issues.apache.org/jira/browse/JAMES-2929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jörg Thomas (ConSol) updated JAMES-2929:

Description: 
The table structure as defined in {{sqlResources.xml}} and the corresponding 
JDBC code in 
{{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
 have diverged since the fix for 
[JAMES-2303](https://issues.apache.org/jira/browse/JAMES-2303). There's three 
issues:
 * `server/app/src/main/resources/sqlResources.xml` was left unmodified
 * the order in `server/data/data-jdbc/src/test/resources/sqlResources.xml` is 
wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)

Since this went completely unnoticed for quite some time, adding tests would be 
appropriate?

There is a third issue which I will bring up when these first two fixes are 
done.

  was:
The table structure as defined in `sqlResources.xml` and the corresponding JDBC 
code in 
`server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java`
 have diverged since the fix for 
JAMES-2303(https://issues.apache.org/jira/browse/JAMES-2303). There's three 
issues:
 * `server/app/src/main/resources/sqlResources.xml` was left unmodified
 * the order in `server/data/data-jdbc/src/test/resources/sqlResources.xml` is 
wrong:
 ** message_body (SQL index: 9, JDBC index: 11)
 ** message_attributes (SQL index: 10, JDBC index: 12)
 ** per_recipient_headers (SQL index: 11, JDBC index: 9)
 ** last_updated timestamp (SQL index: 12, JDBC index: 10)

Since this went completely unnoticed for quite some time, adding tests would be 
appropriate?

There is a third issue which I will bring up when these first two fixes are 
done.


> JDBCMailRepository broken: SQL and JDBC code out of sync
> 
>
> Key: JAMES-2929
> URL: https://issues.apache.org/jira/browse/JAMES-2929
> Project: James Server
>  Issue Type: Bug
>  Components: JDBC, MailStore & MailRepository
>Affects Versions: 3.2.0, 3.3.0, 3.4.0, 3.3.1
>Reporter: Jörg Thomas (ConSol)
>Priority: Major
> Fix For: 3.5.0
>
>
> The table structure as defined in {{sqlResources.xml}} and the corresponding 
> JDBC code in 
> {{server/data/data-jdbc/src/main/java/org/apache/james/mailrepository/jdbc/JDBCMailRepository.java}}
>  have diverged since the fix for 
> [JAMES-2303](https://issues.apache.org/jira/browse/JAMES-2303). There's three 
> issues:
>  * `server/app/src/main/resources/sqlResources.xml` was left unmodified
>  * the order in `server/data/data-jdbc/src/test/resources/sqlResources.xml` 
> is wrong:
>  ** message_body (SQL index: 9, JDBC index: 11)
>  ** message_attributes (SQL index: 10, JDBC index: 12)
>  ** per_recipient_headers (SQL index: 11, JDBC index: 9)
>  ** last_updated timestamp (SQL index: 12, JDBC index: 10)
> Since this went completely unnoticed for quite some time, adding tests would 
> be appropriate?
> There is a third issue which I will bring up when these first two fixes are 
> done.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org