[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-18 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13974241#comment-13974241 ] Hudson commented on AVRO-1402: -- FAILURE: Integrated in AvroJava #451 (See [https://builds.apa

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-18 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13974174#comment-13974174 ] ASF subversion and git services commented on AVRO-1402: --- Commit 15884

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-17 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13973367#comment-13973367 ] Ryan Blue commented on AVRO-1402: - Created AVRO-1497 to track the logical type support in s

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-17 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13973233#comment-13973233 ] Doug Cutting commented on AVRO-1402: Yes, it sounds like a separate issue for implement

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-17 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13973173#comment-13973173 ] Ryan Blue commented on AVRO-1402: - [~cutting], thanks for taking a look. I'll update the pa

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-17 Thread Jarek Jarcec Cecho (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13973158#comment-13973158 ] Jarek Jarcec Cecho commented on AVRO-1402: -- Yes [~cutting], I do feel that it woul

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-17 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13972981#comment-13972981 ] Doug Cutting commented on AVRO-1402: Ryanb: I'd prefer that Decimal is not a nested sub

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-16 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13971638#comment-13971638 ] Ryan Blue commented on AVRO-1402: - I don't think we need to work with decimal objects yet,

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-16 Thread Jarek Jarcec Cecho (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13971615#comment-13971615 ] Jarek Jarcec Cecho commented on AVRO-1402: -- Even though that code changes are supp

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-15 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13970408#comment-13970408 ] Doug Cutting commented on AVRO-1402: Does this need to be in Schema.java? That class is

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-15 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13970302#comment-13970302 ] Ryan Blue commented on AVRO-1402: - I've pushed a branch to github, which may be more readab

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-15 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13970093#comment-13970093 ] Doug Cutting commented on AVRO-1402: The changes to the spec look fine, but I'd prefer

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-15 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13969637#comment-13969637 ] Ryan Blue commented on AVRO-1402: - bq. [D]o we put upper bounds for the precision/scale? T

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-14 Thread Xuefu Zhang (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13969109#comment-13969109 ] Xuefu Zhang commented on AVRO-1402: --- Fair enough. I just liked to put all the options on

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-14 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13969061#comment-13969061 ] Ryan Blue commented on AVRO-1402: - [~xuefuz], you're right that this is between those two a

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-14 Thread Xuefu Zhang (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13969022#comment-13969022 ] Xuefu Zhang commented on AVRO-1402: --- Since the logical type approach proposed here isn't

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-14 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13968981#comment-13968981 ] Ryan Blue commented on AVRO-1402: - The rules that can be enforced, like consistent metadata

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-14 Thread Xuefu Zhang (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13968962#comment-13968962 ] Xuefu Zhang commented on AVRO-1402: --- Thanks for the clarification, [~rdblue]. That makes

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-14 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13968798#comment-13968798 ] Ryan Blue commented on AVRO-1402: - [~xuefuz], what I'm trying to say is that it's an error

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-14 Thread Xuefu Zhang (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13968708#comment-13968708 ] Xuefu Zhang commented on AVRO-1402: --- bq, Because the maximum precision varies with the le

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-14 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13968559#comment-13968559 ] Ryan Blue commented on AVRO-1402: - bq. I think we should allow writing unspecified logical

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-11 Thread Xuefu Zhang (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13967214#comment-13967214 ] Xuefu Zhang commented on AVRO-1402: --- {quote} precision, a JSON integer representing the (

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-11 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13966795#comment-13966795 ] Doug Cutting commented on AVRO-1402: > fixed may also be annotated with a decimal logic

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-10 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13966159#comment-13966159 ] Hudson commented on AVRO-1402: -- SUCCESS: Integrated in AvroJava #450 (See [https://builds.apa

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-10 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13966052#comment-13966052 ] Ryan Blue commented on AVRO-1402: - bq. A logical type is always encoded in exactly the same

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-10 Thread Skye Wanderman-Milne (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13965887#comment-13965887 ] Skye Wanderman-Milne commented on AVRO-1402: Ah, makes sense. Thanks for the cl

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-10 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13965862#comment-13965862 ] Ryan Blue commented on AVRO-1402: - bq. "Language implementations may choose to represent lo

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-10 Thread Skye Wanderman-Milne (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13965808#comment-13965808 ] Skye Wanderman-Milne commented on AVRO-1402: This looks great Tom! If I underst

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-10 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13965206#comment-13965206 ] ASF subversion and git services commented on AVRO-1402: --- Commit 15862

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-09 Thread Jarek Jarcec Cecho (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13964840#comment-13964840 ] Jarek Jarcec Cecho commented on AVRO-1402: -- I really like the idea of the logical

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-09 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13964588#comment-13964588 ] Doug Cutting commented on AVRO-1402: > At this point I think we should revert this patc

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-09 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13964337#comment-13964337 ] Tom White commented on AVRO-1402: - I think the unique name limitation is a problem. At this

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-08 Thread Skye Wanderman-Milne (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13963738#comment-13963738 ] Skye Wanderman-Milne commented on AVRO-1402: I strongly prefer the logical type

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-08 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13963257#comment-13963257 ] Ryan Blue commented on AVRO-1402: - No problem. Going from int to long or float to double is

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-08 Thread Nong Li (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13963209#comment-13963209 ] Nong Li commented on AVRO-1402: --- I see your use case as the same as going from int -> long or

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-08 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13963200#comment-13963200 ] Ryan Blue commented on AVRO-1402: - I don't think evolution would cover the use case because

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-08 Thread Nong Li (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13963180#comment-13963180 ] Nong Li commented on AVRO-1402: --- Ryan, your example seems like a schema evolution use case. E

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-08 Thread Ryan Blue (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13963146#comment-13963146 ] Ryan Blue commented on AVRO-1402: - bq. There have been objections to the approach implement

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-08 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13963029#comment-13963029 ] Tom White commented on AVRO-1402: - There have been objections to the approach implemented h

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-08 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13962954#comment-13962954 ] Tom White commented on AVRO-1402: - There has been some further (offline) discussion about w

[jira] [Commented] (AVRO-1402) Support for DECIMAL type

2014-04-04 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13959889#comment-13959889 ] Hudson commented on AVRO-1402: -- SUCCESS: Integrated in AvroJava #448 (See [https://builds.apa