[GitHub] avro pull request: AVRO-1497: Add Java logical type implementation

2015-03-29 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/29 AVRO-1497: Add Java logical type implementation You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/avro AVRO-1497-logical-types Alternatively

[GitHub] avro pull request: AVRO-607. Make SpecificData#getSchema thread-sa...

2015-03-30 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/30 AVRO-607. Make SpecificData#getSchema thread-safe. This adds a version of WeakIdentityHashMap that is thread-safe by replacing the backing Map implementation with ConcurrentHashMap. Both

[GitHub] avro pull request: AVRO-1497: Add Java logical type implementation

2015-05-26 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/avro/pull/29 --- 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

[GitHub] avro pull request: AVRO-1672: Add time logical types and conversio...

2015-05-28 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/36 AVRO-1672: Add time logical types and conversions. This uses Joda classes to represent the new logical types: * date => LocalDate * time-millis => LocalTime * timestamp-millis =>

[GitHub] avro pull request: AVRO-1684: Add support for date, time, and time...

2015-06-17 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/37 AVRO-1684: Add support for date, time, and timestamp to specific This is based on AVRO-1672, which adds the date, time, and timestamp conversions. The specific IDL has been updated with 3 new

[GitHub] avro pull request: AVRO-1692: Allow multiple logical types for a J...

2015-07-06 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/39 AVRO-1692: Allow multiple logical types for a Java class. This changes the class to conversion lookup to store multiple conversions and choose based on the logical type name. This still assumes

[GitHub] avro pull request: AVRO-1672: Add time logical types and conversio...

2015-07-09 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/avro/pull/36 --- 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

[GitHub] avro pull request: AVRO-1692: Allow multiple logical types for a J...

2015-07-14 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/avro/pull/39 --- 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

[GitHub] avro pull request: AVRO-1799: Fix GenericRecord#toString ByteBuffe...

2016-02-21 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/73 AVRO-1799: Fix GenericRecord#toString ByteBuffer bug. Reading the ByteBuffer to return a string representaton modified the buffer's position. The solution is to duplicate the buffer b

[GitHub] avro pull request: AVRO-1799: Fix GenericRecord#toString ByteBuffe...

2016-02-29 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/avro/pull/73 --- 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

[GitHub] avro pull request: AVRO-1684: Add time types to the specific compi...

2016-04-16 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/86 AVRO-1684: Add time types to the specific compiler. This adds a dependency on Joda time for both the compiler and the compiled classes. When generating Java classes, any conversion that is

[GitHub] avro pull request: Add .travis.yml for Java tests

2016-05-24 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/98 Add .travis.yml for Java tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/avro travis-test Alternatively you can review and apply

[GitHub] avro pull request #102: AVRO-1869: Java: Fix Decimal conversion from ByteBuf...

2016-06-27 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/102 AVRO-1869: Java: Fix Decimal conversion from ByteBuffer. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/avro AVRO-1869-fix-bytebuffer

[GitHub] avro pull request #103: AVRO-1704: Add DatumEncoder API

2016-06-27 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/103 AVRO-1704: Add DatumEncoder API You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/avro AVRO-1704-add-datum-encoder-decoder Alternatively

[GitHub] avro pull request #102: AVRO-1869: Java: Fix Decimal conversion from ByteBuf...

2016-06-28 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/avro/pull/102 --- 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

[GitHub] avro pull request #106: AVRO-1878: Add support for field aliases to Resolvin...

2016-07-09 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/106 AVRO-1878: Add support for field aliases to ResolvingGrammarGenerator. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/avro AVRO-1878-fix

[GitHub] avro pull request #117: AVRO-1908: Fix TestSpecificCompiler reference to pri...

2016-09-04 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/117 AVRO-1908: Fix TestSpecificCompiler reference to private method. AVRO-1884 changed makePath to a private method from a package-private static method. This broke the test that references the method

[GitHub] avro pull request #118: AVRO-1891: Fix specific nested logical types

2016-09-05 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/118 AVRO-1891: Fix specific nested logical types This changes the datum readers to use `getConversionFor` and `getConversionByClass` instance methods that delegate to the data models. This allows

[GitHub] avro pull request #117: AVRO-1908: Fix TestSpecificCompiler reference to pri...

2016-09-08 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/avro/pull/117 --- 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

[GitHub] avro pull request #121: AVRO-1873: Add CRC32 checksum to Snappy-compressed b...

2016-09-10 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/121 AVRO-1873: Add CRC32 checksum to Snappy-compressed blocks. Java and other implementations require this CRC32 checksum of the uncompressed content in order to read the data. This implements the

[GitHub] avro pull request #122: AVRO-1900: Remove dev-tools to fix Java build.

2016-09-10 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/avro/pull/122 AVRO-1900: Remove dev-tools to fix Java build. The dev-tools module has to be built before working in lang/java. This breaks the build.sh script and is a confusing failure for new Java

[GitHub] avro pull request #122: AVRO-1900: Remove dev-tools to fix Java build.

2016-09-13 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/avro/pull/122 --- 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

[GitHub] nifi pull request: AbstractKiteProcessor.getSchema leaks file hand...

2015-10-12 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/101#issuecomment-147560797 +1, thanks for fixing this @jackowaya! --- 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

[GitHub] nifi pull request: NIFI-1193: Add Hive support to Kite storage pro...

2015-11-18 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/nifi/pull/128 NIFI-1193: Add Hive support to Kite storage processor. This adds a Hive dataset implementation that shades and minimizes the classes needed to connect to the Hive MetaStore. You can merge this

[GitHub] nifi pull request: NIFI-1193: Add Hive support to Kite storage pro...

2015-11-19 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/128#issuecomment-158137120 @joey, that's exactly the problem. If you have a solution that avoids hive-exec, that would be great! --- If your project is set up for it, you can reply to this

[GitHub] nifi pull request: NIFI-1193: Add Hive support to Kite storage pro...

2015-12-23 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/nifi/pull/128 --- 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

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2015-12-23 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/nifi/pull/147 NIFI-1193: Add support for storing data in Hive tables. This adds support for storing data in Hive tables to the Kite processor using @joey's suggestion on #128. I've excluded as many depen

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-25 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/147#issuecomment-174752878 Yep! --- 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

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-26 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/147#issuecomment-175186662 Now that this is in master, I'm going to close this PR. Thanks @rickysaltzer! --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-26 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/nifi/pull/147 --- 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

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-26 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/147#issuecomment-175230521 I'll find where that leak is coming from, sorry about this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-27 Thread rdblue
GitHub user rdblue reopened a pull request: https://github.com/apache/nifi/pull/147 NIFI-1193: Add support for storing data in Hive tables. This adds support for storing data in Hive tables to the Kite processor using @joey's suggestion on #128. I've excluded as many de

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-27 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/147#issuecomment-175765398 @apiri I've reopened this and will fix it in this PR. Thanks for rolling back the changes in master. --- If your project is set up for it, you can reply to this emai

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-28 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/147#issuecomment-176281907 I just pushed a fix that @rickysaltzer came up with. The jars being pulled in aren't actually needed for the tests so we can exclude them. --- If your project is s

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-28 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/147#issuecomment-176297371 Looks like the build is working after the new commit. Thanks Ricky! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-28 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/147#issuecomment-176373718 Squashed. --- 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

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-29 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/nifi/pull/147 --- 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

[GitHub] nifi pull request: NIFI-1193: Add support for storing data in Hive...

2016-01-29 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/nifi/pull/147#issuecomment-176866130 Thanks, @rickysaltzer! I'll close this again. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-nifi pull request: NIFI-238: Add Kite processors.

2015-02-15 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-nifi/pull/24 NIFI-238: Add Kite processors. Includes: * KiteStorageProcessor - store Avro files in a Kite dataset * KiteCSVToAvroProcessor - convert CSV to Avro for storage

[GitHub] incubator-nifi pull request: NIFI-238: Add Kite processors.

2015-02-17 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/24#discussion_r24836760 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/JSONFileReader.java --- @@ -0,0

[GitHub] incubator-nifi pull request: NIFI-238: Add Kite processors.

2015-02-17 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/24#discussion_r24836941 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/apache/nifi/processors/kite/KiteStorageProcessor.java

[GitHub] incubator-nifi pull request: NIFI-238: Add Kite processors.

2015-02-17 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/24#discussion_r24837051 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/src/main/java/org/kitesdk/data/spi/filesystem/CSVFileReaderFixed.java

[GitHub] incubator-nifi pull request: NIFI-238: Add Kite processors.

2015-02-17 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/24#discussion_r24837243 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/pom.xml --- @@ -0,0 +1,60 @@ +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://

[GitHub] incubator-nifi pull request: NIFI-238: Add Kite processors.

2015-02-17 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/24#discussion_r24837278 --- Diff: nifi/nifi-nar-bundles/nifi-kite-bundle/pom.xml --- @@ -0,0 +1,60 @@ +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://

[GitHub] incubator-nifi pull request: NIFI-238: Add Kite processors.

2015-02-17 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/24#issuecomment-74789039 Thanks for the reviews! I've update the PR with the needed changes. I also noticed that I was using the deprecated annotations, so I updated those as well. -

[GitHub] incubator-nifi pull request: NIFI-238: Add Kite processors.

2015-02-19 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/24#issuecomment-75162252 I've updated the pull request after Mark's comments. Thanks for the great suggestions and help! --- If your project is set up for it, you can reply to

[GitHub] incubator-nifi pull request: NIFI-428: Update to Kite 1.0.0.

2015-03-16 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-nifi/pull/36 NIFI-428: Update to Kite 1.0.0. Kite 1.0.0 fixes the JSON and CSV readers so they no longer need to be in NiFi. You can merge this pull request into a Git repository by running

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-05 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r31855489 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSinkProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-05 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r31855669 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSinkProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-05 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r31855713 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSinkProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-05 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r31856990 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSourceProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-05 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r31857255 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSourceProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-05 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r31857427 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSinkProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-05 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r31858031 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSourceProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-09 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/51#issuecomment-110475354 @joey did you find a good way to have a reference to a `ProcessSession` outside of the `onTrigger` method? --- If your project is set up for it, you can reply to

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-10 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r32166926 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSinkProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-10 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r32167493 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSinkProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-10 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r32168462 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSourceProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-10 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r32168616 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSourceProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-10 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r32168957 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSourceProcessor.java

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-10 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/51#issuecomment-110921016 +1 Looks great! --- 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

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-10 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-nifi/pull/51#discussion_r32173669 --- Diff: nifi/nifi-nar-bundles/nifi-flume-bundle/nifi-flume-processors/src/main/java/org/apache/nifi/processors/flume/FlumeSinkProcessor.java

[GitHub] incubator-nifi pull request: NIFI-738: Improve error handling for ...

2015-06-27 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-nifi/pull/69 NIFI-738: Improve error handling for CSV and JSON conversion. Changes: * Send bad record information on the "incompatible" relationship * Use an attribute, "errors&

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-27 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/51#issuecomment-116191838 > Is there any service provider construct or similar mechanism whereby we could provide a dropdown of identified sources and sinks I think this

[GitHub] incubator-nifi pull request: NIFI-738: Improve error handling for ...

2015-06-28 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/69#issuecomment-116218005 Thanks for taking a look, Joe. I implemented the changes as you suggested. I kept the incompatible relationship because when we can recover the original

[GitHub] incubator-nifi pull request: NIFI-738: Improve error handling for ...

2015-06-29 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/69#issuecomment-116809640 @joewitt, I get an error that it converted 3 rows successfully instead of 2. I'm not sure I understand why that isn't the correct behavior. Shouldn'

[GitHub] incubator-nifi pull request: NIFI-738: Improve error handling for ...

2015-06-29 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/69#issuecomment-116813026 Nevermind, I was in the wrong branch. I've fixed the bug and will have this updated shortly. Thanks! --- If your project is set up for it, you can reply to

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-06-29 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/51#issuecomment-116823666 I think thread-local would be okay because the thread doesn't change while the process is running. In onTrigger, you would first set the thread's sess

[GitHub] incubator-nifi pull request: NIFI-738: Improve error handling for ...

2015-06-29 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/69#issuecomment-116838575 I think this is ready to go. I'm not concerned with the error tracking because we keep to a pretty reasonable format, "General error: specific data"

[GitHub] incubator-nifi pull request: NIFI-738: Improve error handling for ...

2015-06-29 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/69#issuecomment-116841432 No problem, I wanted to get it into the release. Thanks for your help! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-nifi pull request: NIFI-738: Improve error handling for ...

2015-06-29 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/69#issuecomment-116860830 @joewitt, looks like the problem is that the accumulated error messages are out of order. This is a result of moving from the multimap to a plain hashmap. I&#x

[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

[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

[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

[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

[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

[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

[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

[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_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_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_r34175286 --- 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_r34175566 --- 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_r34176166 --- 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_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_r34178232 --- 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-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

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-07-12 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/51#issuecomment-120764330 +1. Thanks Joey! --- 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

[GitHub] incubator-nifi pull request: NIFI-589: Add processors that can run...

2015-07-12 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/incubator-nifi/pull/51#issuecomment-120764934 Another way to fix the threading issue is to validate the sinks and sources are safe and wrap them in first-class processors that allow threading even if the

[GitHub] incubator-toree pull request #116: [TOREE-395] disable cell result output

2017-03-21 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-toree/pull/116#discussion_r107310980 --- Diff: scala-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/scala/ScalaInterpreter.scala --- @@ -48,7 +48,7 @@ class

[GitHub] incubator-toree pull request #116: [TOREE-395] disable cell result output fr...

2017-03-22 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/incubator-toree/pull/116#discussion_r107525449 --- Diff: scala-interpreter/src/main/scala/org/apache/toree/kernel/interpreter/scala/ScalaInterpreter.scala --- @@ -48,7 +48,7 @@ class

[GitHub] incubator-toree pull request #120: TOREE-412: Update AddDeps magic

2017-06-10 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-toree/pull/120 TOREE-412: Update AddDeps magic * Removes non-Jar files (POM files) from downloaded artifacts before adding dependencies to the kernel * Updates to Coursier 1.0.0-M15-7 * Uses

[GitHub] incubator-toree pull request #121: TOREE-413: Add exit commands and kernel i...

2017-06-10 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-toree/pull/121 TOREE-413: Add exit commands and kernel idle timeout. This adds support for "exit", "quit", and ":q" to exit Toree. It also adds a session idle timeout that

[GitHub] incubator-toree pull request #122: TOREE-414: Use ThreadFactoryBuilder to se...

2017-06-10 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-toree/pull/122 TOREE-414: Use ThreadFactoryBuilder to set name and daemon status. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/incubator

[GitHub] incubator-toree pull request #104: TOREE-380: Allow interpreters to format o...

2017-06-10 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/incubator-toree/pull/104 --- 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

[GitHub] incubator-toree pull request #104: TOREE-380: Allow interpreters to format o...

2017-06-10 Thread rdblue
GitHub user rdblue reopened a pull request: https://github.com/apache/incubator-toree/pull/104 TOREE-380: Allow interpreters to format output. This branch has the changes that I made to our Toree distribution to allow interpreters to format output. The goal is to enable better

[GitHub] incubator-toree pull request #123: TOREE-415: Cancel Spark jobs when interru...

2017-06-10 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-toree/pull/123 TOREE-415: Cancel Spark jobs when interrupted. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/incubator-toree TOREE-415

[GitHub] incubator-toree pull request #124: TOREE-407: Add support for hdfs and s3 to...

2017-06-10 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-toree/pull/124 TOREE-407: Add support for hdfs and s3 to AddJar. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/incubator-toree TOREE-407-add

[GitHub] incubator-toree pull request #125: TOREE-408: Add support for hdfs and s3 to...

2017-06-10 Thread rdblue
GitHub user rdblue opened a pull request: https://github.com/apache/incubator-toree/pull/125 TOREE-408: Add support for hdfs and s3 to AddJar. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rdblue/incubator-toree TOREE-408-add

[GitHub] incubator-toree pull request #124: TOREE-407: Add support for hdfs and s3 to...

2017-06-10 Thread rdblue
Github user rdblue closed the pull request at: https://github.com/apache/incubator-toree/pull/124 --- 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

  1   2   3   4   5   6   7   8   9   10   >