[
https://issues.apache.org/jira/browse/GEODE-6481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16784642#comment-16784642
]
Darrel Schneider commented on GEODE-6481:
-----------------------------------------
This ticket makes some incorrect statements in the original comment.
create jdbc-mapping does not require that the class named in --pdx-name
implement PdxSerializable. If the class can not be serialized as PDX given how
the server's cache is configured, then it will attempt to serialize it by using
the ReflectionBasedAutoSerializer.
In the example given "income" and "id" were correctly made pdx OBJECT fields
because the instance variable was a java.lang.Object (Long and Float) instead
of a primitive (long and float).
> add tests to verify create jdbc mapping should get correct field type
> ---------------------------------------------------------------------
>
> Key: GEODE-6481
> URL: https://issues.apache.org/jira/browse/GEODE-6481
> Project: Geode
> Issue Type: Bug
> Reporter: xiaojian zhou
> Assignee: xiaojian zhou
> Priority: Major
> Time Spent: 10m
> Remaining Estimate: 0h
>
> Add 2 test cases here:
> 1) when using numeric fields in domain class, create mapping and describe
> mapping should get correct fieldMapping between field and column, such as:
> PDX Field | PDX Type | JDBC Column | JDBC Type | Nullable
> --------- | -------- | ----------- | --------- | --------
> income | FLOAT | INCOME | REAL | true
> id | STRING | ID | VARCHAR | false
> refid | LONG | REFID | BIGINT | true
> name | STRING | NAME | VARCHAR | true
> age | INT | AGE | INTEGER | true
> It should NOT be:
> PDX Field | PDX Type | JDBC Column | JDBC Type | Nullable
> --------- | -------- | ----------- | ----------- | --------
> income | OBJECT | income | REAL | true
> name | STRING | name | LONGVARCHAR | true
> id | OBJECT | id | BIGINT | false
> 2) since we enforce the --pdx-name should be a PdxSerializable. We also
> support to convert a regular class into PdxSerializable using
> AutoSerializableManager.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
