You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by yadavsaroj <gi...@git.apache.org> on 2015/12/12 01:09:44 UTC

[GitHub] avro pull request: Yadavsaroj/generic datum reader

GitHub user yadavsaroj opened a pull request:

    https://github.com/apache/avro/pull/64

    Yadavsaroj/generic datum reader

    As per [discussion here](https://github.com/apache/avro/pull/56#discussion_r46463829) with @rdblue, I have moved binary data related knowledge to the binary decoder and have made datum_reader a generic datum reader. 
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/yadavsaroj/avro yadavsaroj/generic_datum_reader

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/avro/pull/64.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #64
    
----
commit ed0be6c3af783a5c69d5abe3d8529ac4ab9657c8
Author: Saroj Yadav <sy...@zendesk.com>
Date:   2015-10-29T23:58:52Z

    AVRO-1746 - Support JSON encoding for Avro data.

commit 0218cbb05c35133361672ac20e0ff238cbfd7ccb
Author: Saroj Yadav <sy...@zendesk.com>
Date:   2015-11-19T18:29:17Z

    - Make json io visible to clients of the gem
    - Add gemspec

commit bcb72bcf85c995de2c2dceace8f6cc072e136e41
Author: Saroj Yadav <sy...@zendesk.com>
Date:   2015-11-24T21:39:51Z

    - Rename read_record_data to read_field_data
    - Use 80 as line length

commit c6661f4ca328612fb7ed71a82d084aae7ad1423b
Author: Saroj Yadav <sy...@zendesk.com>
Date:   2015-11-24T21:44:04Z

    Remove gemspec

commit 79afff8f0fcfde7913454fd584b1f8b6e7c48b64
Author: Saroj Yadav <sy...@zendesk.com>
Date:   2015-12-12T00:03:57Z

    DatumReader as a generic datum reader.

commit aac76b0d0878349c1878a1b1f7b8048b976a2d73
Author: Saroj Yadav <sy...@zendesk.com>
Date:   2015-12-12T00:06:38Z

    Remove reference to json_io

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] avro pull request: DatumReader as a generic datum reader

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/avro/pull/64


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---