[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16689909#comment-16689909
]
ASF subversion and git services commented on AVRO-1605:
---
Commit a7fb
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673661#comment-16673661
]
ASF subversion and git services commented on AVRO-1605:
---
Commit d799
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673666#comment-16673666
]
ASF subversion and git services commented on AVRO-1605:
---
Commit d799
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673664#comment-16673664
]
ASF subversion and git services commented on AVRO-1605:
---
Commit d799
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673667#comment-16673667
]
ASF subversion and git services commented on AVRO-1605:
---
Commit d799
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673665#comment-16673665
]
ASF subversion and git services commented on AVRO-1605:
---
Commit d799
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673662#comment-16673662
]
ASF subversion and git services commented on AVRO-1605:
---
Commit d799
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673663#comment-16673663
]
ASF subversion and git services commented on AVRO-1605:
---
Commit d799
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16542364#comment-16542364
]
Ahmet Altay commented on AVRO-1605:
---
What is the status of this issue? This issue was me
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16493769#comment-16493769
]
Romain Manni-Bucau commented on AVRO-1605:
--
Hi,
it can be neat to move to javax.
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16409861#comment-16409861
]
Mike Yoder commented on AVRO-1605:
--
I wanted to chime in on this issue from a security per
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16282158#comment-16282158
]
Ryan Blue commented on AVRO-1605:
-
I don't think Avro should use accessors or friend packag
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16281727#comment-16281727
]
Davide Gesino commented on AVRO-1605:
-
This PR has been opened for a long time. @Ryan B
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15856011#comment-15856011
]
Gabor Szadovszky commented on AVRO-1605:
[~rdblue], do you have any additional comm
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15728298#comment-15728298
]
Tom White commented on AVRO-1605:
-
I agree that the second approach is not feasible for thi
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15726078#comment-15726078
]
Gabor Szadovszky commented on AVRO-1605:
Currently the classes {{ResolvingGrammarGe
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15725964#comment-15725964
]
Ryan Blue commented on AVRO-1605:
-
I think the second option is the right one. Accessors ca
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15725006#comment-15725006
]
Gabor Szadovszky commented on AVRO-1605:
Thanks for the reply, [~rdblue].
I comple
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15722949#comment-15722949
]
Ryan Blue commented on AVRO-1605:
-
To keep a patch small, we can break it across multiple i
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15708893#comment-15708893
]
Gabor Szadovszky commented on AVRO-1605:
What do you think, [~rdblue]?
> Remove Ja
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15628478#comment-15628478
]
Gabor Szadovszky commented on AVRO-1605:
- {{Accessor.defaultValue(Field)}} is used
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15620519#comment-15620519
]
Ryan Blue commented on AVRO-1605:
-
This is looking better, but I'd like to see more justifi
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15580234#comment-15580234
]
Gabor Szadovszky commented on AVRO-1605:
Removed some unnecessary calls of the Acce
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15567956#comment-15567956
]
Gabor Szadovszky commented on AVRO-1605:
Thanks for the response, Ryan. Sure, it is
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15565892#comment-15565892
]
Ryan Blue commented on AVRO-1605:
-
Because I don't think accessors are a good practice, I t
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15565867#comment-15565867
]
Ryan Blue commented on AVRO-1605:
-
[~gszadovszky], sorry about not getting back to this bef
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15565661#comment-15565661
]
Gabor Szadovszky commented on AVRO-1605:
Any updates on this one? I would appreciat
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15544956#comment-15544956
]
Tom White commented on AVRO-1605:
-
I'm not sure how long the accessors would need to be the
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15544756#comment-15544756
]
Gabor Szadovszky commented on AVRO-1605:
My understanding about this issue was to n
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15542684#comment-15542684
]
Ryan Blue commented on AVRO-1605:
-
My concern isn't that this is public; I see that these c
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15542678#comment-15542678
]
Tom White commented on AVRO-1605:
-
The point behind the accessors is to remove the Jackson
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15542652#comment-15542652
]
Ryan Blue commented on AVRO-1605:
-
If I understand correctly, the motivation behind adding
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15542497#comment-15542497
]
Tom White commented on AVRO-1605:
-
+1. I ran all the tests and they pass. I'll commit this
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15533271#comment-15533271
]
Gabor Szadovszky commented on AVRO-1605:
Good point.
Found two more references lef
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15533165#comment-15533165
]
Tom White commented on AVRO-1605:
-
This looks great! If you build the javadoc are there any
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15529342#comment-15529342
]
Gabor Szadovszky commented on AVRO-1605:
Thanks a lot, [~tomwhite].
The friend pa
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15529260#comment-15529260
]
Tom White commented on AVRO-1605:
-
I see your point. The motivation behind this change is t
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15529070#comment-15529070
]
Gabor Szadovszky commented on AVRO-1605:
I completely agree that the best way would
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15528994#comment-15528994
]
Tom White commented on AVRO-1605:
-
I'm not sure it gains much to make incompatible changes
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15528821#comment-15528821
]
Gabor Szadovszky commented on AVRO-1605:
There are many public methods leaking Jack
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15526331#comment-15526331
]
Tom White commented on AVRO-1605:
-
I think we should remove the method. How much code needs
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15525473#comment-15525473
]
Gabor Szadovszky commented on AVRO-1605:
Thanks, [~tomwhite]. The purpose is clear.
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15525366#comment-15525366
]
Tom White commented on AVRO-1605:
-
The goal with this is to remove the Jackson classes from
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15522922#comment-15522922
]
Gabor Szadovszky commented on AVRO-1605:
Thanks for the link, [~tomwhite]. Now, it'
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15522733#comment-15522733
]
Tom White commented on AVRO-1605:
-
Although very rare, backward incompatible changes to the
[
https://issues.apache.org/jira/browse/AVRO-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15522493#comment-15522493
]
Gabor Szadovszky commented on AVRO-1605:
I do not know (and failed to find info abo
46 matches
Mail list logo