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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to