Github user JamesRTaylor commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/48#discussion_r26729460
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/schema/types/PUnsignedLong.java 
---
    @@ -95,6 +95,11 @@ public Object toObject(byte[] b, int o, int l, PDataType 
actualType, SortOrder s
       }
     
       @Override
    +    public boolean isCastableTo(PDataType targetType) {
    --- End diff --
    
    You'd never define a column with a type of integer and cast it to a 
DATE/TIME. The "constant to DATE/TIME/TIMESTAMP" isn't a real use case. Given 
there's a workaround already even for unsigned timestamp (casting it to a 
BIGINT first), I'm somewhat reluctant to do anything.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to