qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From alex-rufous <...@git.apache.org>
Subject [GitHub] qpid-broker-j pull request #9: [QPID-8208] Fix Sybase support for the link-s...
Date Thu, 05 Jul 2018 16:11:33 GMT
Github user alex-rufous commented on a diff in the pull request:

    https://github.com/apache/qpid-broker-j/pull/9#discussion_r200407725
  
    --- Diff: broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/store/jdbc/JDBCDetails.java
---
    @@ -379,7 +403,8 @@ public boolean isOverridden()
                     return contextMap.containsKey(CONTEXT_JDBCSTORE_USEBYTESFORBLOB)
                             || contextMap.containsKey(CONTEXT_JDBCSTORE_BIGINTTYPE)
                             || contextMap.containsKey(CONTEXT_JDBCSTORE_VARBINARYTYPE)
    -                        || contextMap.containsKey(CONTEXT_JDBCSTORE_BLOBTYPE);
    +                        || contextMap.containsKey(CONTEXT_JDBCSTORE_BLOBTYPE)
    --- End diff --
    
    Please undo the change as context variable for timestamp type should not be used to indicate
to use bytes methods for blob.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


Mime
View raw message