jhump commented on PR #3069:
URL: https://github.com/apache/avro/pull/3069#issuecomment-2546772954

   Before I stumbled upon this issue (and the associated JIRA issue), I had 
made a patch similar to what's proposed -- but instead of just renaming methods 
to make it more clear that they are JSON-encoded, I had actually added new 
methods. I just opened a PR with that change (#3266), however I'm more than 
happy to close that PR if the change in this one is preferred.
   
   Is this PR's merge being held up only for renaming the methods to have a 
"JSON" suffix? If we wanted to keep the old ones and just add new ones, that 
ends up looking a **lot** like the PR I just opened. I'd just need to mark the 
old methods deprecated in my branch.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to