[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-nifi/pull/70 --- 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

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-14 Thread joewitt
Github user joewitt commented on the pull request: https://github.com/apache/incubator-nifi/pull/70#issuecomment-121218552 i just want to add how easy this was to test. We're definitely going to build an avro viewer like we have for JSON, XML, etc.. because of this cool use case.

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-14 Thread joewitt
Github user joewitt commented on the pull request: https://github.com/apache/incubator-nifi/pull/70#issuecomment-121215127 @rdblue You might want to double check the CSV and JSON to Avro processors and add documentation to them like Alan has done here (additional documentation). Prov

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-13 Thread jackowaya
Github user jackowaya commented on the pull request: https://github.com/apache/incubator-nifi/pull/70#issuecomment-120983883 rebased to upstream/develop and squashed into one commit. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-13 Thread joewitt
Github user joewitt commented on the pull request: https://github.com/apache/incubator-nifi/pull/70#issuecomment-120979194 Alan - excellent and thanks! Do you mind providing a rebased/squashed commit? --- If your project is set up for it, you can reply to this email and have your re

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-13 Thread jackowaya
Github user jackowaya commented on the pull request: https://github.com/apache/incubator-nifi/pull/70#issuecomment-120950010 Made many changes here, including I believe all of Joe's suggestions and the majority of Ryan's. A few little notes: * I wound up using Scanner to do th

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-10 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34380128 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-10 Thread jackowaya
Github user jackowaya commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34363327 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34178232 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34176343 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/ConvertAvroSchema.java --- @@ -0,0

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34176166 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34175566 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34175286 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34175017 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/ConvertAvroSchema.java --- @@ -0,0

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34174772 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/ConvertAvroSchema.java --- @@ -0,0

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34174588 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/ConvertAvroSchema.java --- @@ -0,0

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34174435 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34174088 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34173977 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread jackowaya
Github user jackowaya commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34173958 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34173699 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34173302 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34173083 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-08 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/70#discussion_r34172888 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/AvroRecordConverter.java --- @@ -0,

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-07 Thread joewitt
Github user joewitt commented on the pull request: https://github.com/apache/incubator-nifi/pull/70#issuecomment-119416855 Alan, Thanks for contributing! We'll work with you to get this promptly merged. Here are some findings of an initial review: * Please run the b

[GitHub] incubator-nifi pull request: NIFI-751 Add Processor To Convert Avr...

2015-07-07 Thread jackowaya
GitHub user jackowaya opened a pull request: https://github.com/apache/incubator-nifi/pull/70 NIFI-751 Add Processor To Convert Avro Formats Implemented a new NiFi processor that allows avro records to be converted from one Avro schema to another. This supports.. * Flattenin