[GitHub] nifi pull request #811: NIFI-2503 Backport PostHTTP SSL Protocol fix to 0.x ...

2016-08-09 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/811 --- 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] nifi issue #793: NIFI-2494 Remove extraneous text that shows on banner line

2016-08-05 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/793 Working on it now. --- 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

[GitHub] nifi issue #793: NIFI-2494 Remove extraneous text that shows on banner line

2016-08-05 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/793 @mcgilman, I restored the text and added the class to the div tag. My canvas is still fixed and the title is back on the Controller Settings page. Thanks for catching that. --- If your

[GitHub] nifi pull request #811: NIFI-2503 Backport PostHTTP SSL Protocol fix to 0.x ...

2016-08-08 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/811 NIFI-2503 Backport PostHTTP SSL Protocol fix to 0.x branch Merge changes from [NIFI-1688](https://issues.apache.org/jira/browse/NIFI-1688) / [PR-624](https://github.com/apache/nifi/pull/624/files

[GitHub] nifi pull request #806: Extend REST API docs template to handle read only pr...

2016-08-07 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/806 Extend REST API docs template to handle read only properties. * Add read only property handling to type.hbs template. * Add flag to ProcessorDTO.getRelationships to reflect read only nature

[GitHub] nifi pull request #1470: NIFI-3055 StandardRecordWriter Can Throw UTFDataFor...

2017-02-02 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1470 NIFI-3055 StandardRecordWriter Can Throw UTFDataFormatException (0.x) * Updated StandardRecordWriter to consider the encoding behavior of java.io.DataOutputStream.writeUTF() and truncate string

[GitHub] nifi pull request #1481: NIFI-3055 StandardRecordWriter Can Throw UTFDataFor...

2017-02-07 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1481 NIFI-3055 StandardRecordWriter Can Throw UTFDataFormatException (0.x) * Remove function based on JDK source. * Add new function to find bytes based on RFC3629. * Add field name to log entry

[GitHub] nifi pull request #1485: NIFI-2890 Provenance Repository Corruption (1.x)

2017-02-08 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1485 NIFI-2890 Provenance Repository Corruption (1.x) * Corrected handling of corrupt journal file records that prevents instance startup and loss of records from corrupt files. Specifically, exception

[GitHub] nifi pull request #1484: NIFI-2890 Provenance Repository Corruption (0.x)

2017-02-08 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1484 NIFI-2890 Provenance Repository Corruption (0.x) * Corrected handling of corrupt journal file records that prevents instance startup and loss of records from corrupt files. Specifically, exception

[GitHub] nifi pull request #1475: NIFI-3055 StandardRecordWriter Can Throw UTFDataFor...

2017-02-06 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1475 NIFI-3055 StandardRecordWriter Can Throw UTFDataFormatException (1.x) * Remove function based on JDK source. * Add new function to find bytes based on RFC3629. * Add field name to log entry

[GitHub] nifi issue #1470: NIFI-3055 StandardRecordWriter Can Throw UTFDataFormatExce...

2017-02-06 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1470 Closing this PR, committed on [4f72e34](https://git-wip-us.apache.org/repos/asf?p=nifi.git;a=commit;h=4f72e3491f2372c8c45afb96a765c1f5cdd2f07d). --- If your project is set up for it, you can reply

[GitHub] nifi pull request #1470: NIFI-3055 StandardRecordWriter Can Throw UTFDataFor...

2017-02-06 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/1470 --- 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] nifi issue #1469: NIFI-3055 StandardRecordWriter Can Throw UTFDataFormatExce...

2017-02-03 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1469 @mosermw I have updated the log message in both PRs. Thanks! --- 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 issue #1201: NIFI-2890 Provenance Repository Corruption

2017-02-14 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1201 Replaced by #1484. --- 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

[GitHub] nifi pull request #1203: NIFI-2890 Provenance Repository Corruption

2017-02-14 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/1203 --- 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] nifi pull request #1201: NIFI-2890 Provenance Repository Corruption

2017-02-14 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/1201 --- 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] nifi issue #1481: NIFI-3055 StandardRecordWriter Can Throw UTFDataFormatExce...

2017-02-14 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1481 Closed by commit [4f72e34](https://git-wip-us.apache.org/repos/asf?p=nifi.git;a=commit;h=4f72e3491f2372c8c45afb96a765c1f5cdd2f07d). Thanks @mosermw! --- If your project is set up for it, you can

[GitHub] nifi pull request #1481: NIFI-3055 StandardRecordWriter Can Throw UTFDataFor...

2017-02-14 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/1481 --- 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] nifi issue #1203: NIFI-2890 Provenance Repository Corruption

2017-02-14 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1203 Replaced by #1485 --- 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

[GitHub] nifi issue #1484: NIFI-2890 Provenance Repository Corruption (0.x)

2017-02-16 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1484 Committed on [7d23bd7](https://git-wip-us.apache.org/repos/asf?p=nifi.git;a=commit;h=7d23bd7ac6af228a3e723d18253063231c919cf9). --- If your project is set up for it, you can reply to this email

[GitHub] nifi pull request #1484: NIFI-2890 Provenance Repository Corruption (0.x)

2017-02-16 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/1484 --- 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] nifi issue #1128: NIFI-2861 ControlRate should accept more than one flow fil...

2017-01-17 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1128 Closing per commit a3d95dc1582f2edfd7997c5d8a23105e88729d11 by @mosermw . --- 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

[GitHub] nifi pull request #1128: NIFI-2861 ControlRate should accept more than one f...

2017-01-17 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/1128 --- 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] nifi pull request #886: NIFI-1867 Improve ModifyBytes to make it easy to rem...

2016-08-19 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/886 --- 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] nifi issue #886: NIFI-1867 Improve ModifyBytes to make it easy to remove all...

2016-08-19 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/886 Closed by 0.x branch commit f89bc9efd8b8458b5bfd6a1b4045ce8230117ff4. Thanks @pvillard31 --- 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 #886: NIFI-1867 Improve ModifyBytes to make it easy to rem...

2016-08-19 Thread jskora
Github user jskora commented on a diff in the pull request: https://github.com/apache/nifi/pull/886#discussion_r75469531 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ModifyBytes.java --- @@ -74,6 +74,13

[GitHub] nifi issue #886: NIFI-1867 Improve ModifyBytes to make it easy to remove all...

2016-08-19 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/886 @pvillard31, I committed the .allowableValues change to this and the 1.x pull https://github.com/apache/nifi/pull/890. --- If your project is set up for it, you can reply to this email and have your

[GitHub] nifi pull request #904: NIFI-2411 ModifyBytes should use long instead of int...

2016-08-19 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/904 NIFI-2411 ModifyBytes should use long instead of int for offsets (1.x) * Update to support offsets larger than 2 gigabyte. You can merge this pull request into a Git repository by running: $ git

[GitHub] nifi pull request #903: NIFI-2411 ModifyBytes should use long instead of int...

2016-08-19 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/903 NIFI-2411 ModifyBytes should use long instead of int for offsets (0.x) * Update to support offsets larger than 2 gigabyte. You can merge this pull request into a Git repository by running: $ git

[GitHub] nifi pull request #890: NIFI-1867 Improve ModifyBytes to make it easy to rem...

2016-08-18 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/890 NIFI-1867 Improve ModifyBytes to make it easy to remove all FlowFile content (1.x) * Add "Remove All Content" property, related functionality, and tests to ModifyBytes processor to allo

[GitHub] nifi issue #913: NIFI-2440 - Add 'file.lastModifiedTime' attribute to ListSF...

2016-09-06 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/913 @kirkalicious, thanks! The changes look good and tests check out fine. I'd prefer to move the constants to a parent common to ListFileTransfer and ListFile, but the only common parent

[GitHub] nifi issue #913: NIFI-2440 - Add 'file.lastModifiedTime' attribute to ListSF...

2016-09-07 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/913 @joewitt Thanks, from what I can tell looking at JDK source the format spec drives the output more than the Locale, so I agree that the comments should cover it. @kirkalicious can you add

[GitHub] nifi pull request #1007: NIFI-2752 Correct ReplaceText default pattern and u...

2016-09-12 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1007 NIFI-2752 Correct ReplaceText default pattern and unit tests * Corrected the DEFAULT_REGEX pattern. * Added tests to isolate regex capture group problem and verify corrected functionality

[GitHub] nifi issue #913: NIFI-2440 - Add 'file.lastModifiedTime' attribute to ListSF...

2016-09-09 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/913 @kirkalicious Please close this, I committed it, but I transposed the PR number in the comment. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] nifi issue #1072: NIFI-2429 PersistentProvenanceRepository bug fixes

2016-10-05 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1072 Reviewing --- 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

[GitHub] nifi issue #1072: NIFI-2429 PersistentProvenanceRepository bug fixes

2016-10-06 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1072 Reviewed and committed. Thanks @mosermw --- 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] nifi pull request #886: NIFI-1867 Improve ModifyBytes to make it easy to rem...

2016-08-18 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/886 NIFI-1867 Improve ModifyBytes to make it easy to remove all FlowFile content (0.x) * Add "Remove All Content" property, related functionality, and tests to ModifyBytes processor to allo

[GitHub] nifi pull request #1201: NIFI-2890 Provenance Repository Corruption

2016-11-10 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1201 NIFI-2890 Provenance Repository Corruption Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure the following

[GitHub] nifi pull request #1203: NIFI-2890 Provenance Repository Corruption

2016-11-10 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1203 NIFI-2890 Provenance Repository Corruption Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure the following

[GitHub] nifi-site pull request #7: NIFI-2911 Update Release Guide

2016-10-23 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi-site/pull/7 NIFI-2911 Update Release Guide * Added steps to configure GPG to default to strong digest algorithms (SHA512/SHA384/SHA256 over SHA1). * Added steps to produce SHA-256 checksum over artifacts

[GitHub] nifi-site pull request #6: NIFI-2910 Added Pony Mail archive links to mailin...

2016-10-17 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi-site/pull/6 NIFI-2910 Added Pony Mail archive links to mailing lists page. Adds Pony Mail links to the website mailing lists page. You can merge this pull request into a Git repository by running: $ git

[GitHub] nifi issue #1127: NIFI-2861 ControlRate should accept more than one flow fil...

2016-10-17 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1127 @markap14 except for removing the limit, review comments have been addressed. The FlowFile limit property has been renamed "Max FlowFiles per Batch" to reflect a batching metaph

[GitHub] nifi pull request #1240: NIFI-3039 Provenance Repository - Fix PurgeOldEvent...

2016-11-17 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1240 NIFI-3039 Provenance Repository - Fix PurgeOldEvent and Rollover Size Limits (1.x) Thank you for submitting a contribution to Apache NiFi. In order to streamline the review

[GitHub] nifi pull request #1241: NIFI-3039 Provenance Repository - Fix PurgeOldEvent...

2016-11-17 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1241 NIFI-3039 Provenance Repository - Fix PurgeOldEvent and Rollover Size Limits (0.x) Thank you for submitting a contribution to Apache NiFi. In order to streamline the review

[GitHub] nifi pull request #1127: NIFI-2861 ControlRate should accept more than one f...

2016-10-12 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1127 NIFI-2861 ControlRate should accept more than one flow file per execution (1.x) Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution

[GitHub] nifi pull request #1128: NIFI-2861 ControlRate should accept more than one f...

2016-10-13 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1128 NIFI-2861 ControlRate should accept more than one flow file per execution (0.x) Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution

[GitHub] nifi issue #1127: NIFI-2861 ControlRate should accept more than one flow fil...

2017-01-12 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1127 @mwoser, I will close and resubmit a clean request with squashed commits. --- 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

[GitHub] nifi pull request #1127: NIFI-2861 ControlRate should accept more than one f...

2017-01-12 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/1127 --- 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] nifi issue #1412: NIFI-2861 ControlRate should accept more than one flow fil...

2017-01-12 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1412 @mosermw this is the replacement for https://github.com/apache/nifi/pull/1127, rebased and squashed. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #1203: NIFI-2890 Provenance Repository Corruption

2017-01-06 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1203 @JPercivall This has @been updated to align with the changes in Apache/0.x. I couldn't squash without creating a new branch, but a diff with Apache/0.x only shows the PersistentProvenanceRepository

[GitHub] nifi issue #1201: NIFI-2890 Provenance Repository Corruption

2017-01-06 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1201 @JPercivall this has been updated to align with other changes in Master. I couldn't squash without creating a new branch, but a diff with Master only shows the PersistentProvenanceRepository

[GitHub] nifi pull request #1567: NIFI-3517 If HandleHttpResponse cannot write respon...

2017-03-06 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1567 NIFI-3517 If HandleHttpResponse cannot write response remove entry from HttpContextMap * Add test for failure not clear context map. * Add handler to remove context map entry if ProcessException

[GitHub] nifi pull request #1994: NIFI-4166 - Create toolkit module to generate and b...

2017-07-07 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1994 NIFI-4166 - Create toolkit module to generate and build Swagger API library for NiFi REST API Thank you for submitting a contribution to Apache NiFi. In order to streamline the review

[GitHub] nifi pull request #1993: NIFI-3677 - ExtractMediaMetadata should close TikaI...

2017-07-07 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1993 NIFI-3677 - ExtractMediaMetadata should close TikaInputStream * Added finally block to close TikaInputStream. Thank you for submitting a contribution to Apache NiFi. In order

[GitHub] nifi pull request #2008: NIFI-4183 - Fix handshake error logic in SocketFlow...

2017-07-13 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/2008 NIFI-4183 - Fix handshake error logic in SocketFlowFileServerProtocol * Added new unit tests to show how the suppressed handshake exceptions leave the server in a connected status. * Fixed

[GitHub] nifi pull request #1988: NIFI-3218: throw exception in MockProcessSession wh...

2017-07-06 Thread jskora
Github user jskora commented on a diff in the pull request: https://github.com/apache/nifi/pull/1988#discussion_r125982805 --- Diff: nifi-mock/src/main/java/org/apache/nifi/util/MockProcessSession.java --- @@ -756,6 +756,13 @@ public void transfer(FlowFile flowFile

[GitHub] nifi issue #1986: NIFI-2528: added support for SSLContextService protocols i...

2017-07-10 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1986 @trkurc Have you [enabled SSLv3](https://stackoverflow.com/questions/28236091/how-to-enable-ssl-3-in-java) in the JVM? It is disabled by default starting with [Java 8 Update 31](http

[GitHub] nifi issue #1986: NIFI-2528: added support for SSLContextService protocols i...

2017-07-25 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1986 I don't want to complicate this, but I feel like I must be missing something. As much as possible, the validation at configuration time should provide the user feedback, not failure upon

[GitHub] nifi issue #1666: NIFI-3018 removed flume-twitter-source due to usage of org...

2017-04-27 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1666 Reviewing. --- 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

[GitHub] nifi issue #1666: NIFI-3018 removed flume-twitter-source due to usage of org...

2017-04-27 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1666 @mwmoser Committed to 0.x on https://github.com/apache/nifi/commit/5072559156c613823ac554884dd9166acb0edd6b. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi issue #1667: NIFI-3019 cherry-picked from NIFI-3692 into 0.x branch

2017-04-27 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1667 Reviewing --- 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

[GitHub] nifi issue #1688: NIFI-3553 FetchFileTransfer fix

2017-04-27 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1688 Reviewing --- 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

[GitHub] nifi issue #1699: NIFI-3725 - validate processors only when they are in STOP...

2017-04-27 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1699 Reviewing --- 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

[GitHub] nifi issue #1667: NIFI-3019 cherry-picked from NIFI-3692 into 0.x branch

2017-04-27 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1667 @mosermw Committed on https://github.com/apache/nifi/commit/cd1cf68c9841d0cc892187ce0e510492cd9ddd48 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #1688: NIFI-3553 FetchFileTransfer fix

2017-04-27 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1688 Committed on https://github.com/apache/nifi/commit/b69b3b6c622d27544df70e157f8d8b1531a881c6 and https://github.com/apache/nifi/commit/c1e65f95bd9951390215c8ef7e2b25cf95f8b5f1 --- If your project

[GitHub] nifi issue #1699: NIFI-3725 - validate processors only when they are in STOP...

2017-04-27 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1699 @mosermw Committed on https://github.com/apache/nifi/commit/8a18d26c28bcb0e2df6b30359976638637d9. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi pull request #1732: NIFI-2752 Correct ReplaceText default pattern and u...

2017-05-02 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1732 NIFI-2752 Correct ReplaceText default pattern and unit tests (0.x) Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you

[GitHub] nifi pull request #1732: NIFI-2752 Correct ReplaceText default pattern and u...

2017-05-10 Thread jskora
Github user jskora closed the pull request at: https://github.com/apache/nifi/pull/1732 --- 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] nifi issue #1732: NIFI-2752 Correct ReplaceText default pattern and unit tes...

2017-05-10 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1732 @trkurc @pvillard Sorry, my bad for not closing this. --- 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] nifi pull request #1821: NIFI-3932 Symbols in API comments cause problems wi...

2017-05-18 Thread jskora
GitHub user jskora opened a pull request: https://github.com/apache/nifi/pull/1821 NIFI-3932 Symbols in API comments cause problems with Swagger Codegen * Update ApiModelProperty descriptions to avoid problematic characters. Thank you for submitting a contribution to Apache

[GitHub] nifi issue #2198: NIFI-4468: If an entire batch of Provenance Events are rea...

2017-10-08 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/2198 @markap14 I tested on S2S with and without Provenance filter to see the bug before and after fix and all checked out as expected. ---

[GitHub] nifi issue #1975: NIFI-3332: ListXXX to not miss files with the latest proce...

2017-08-29 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1975 @ijokarumawak @bbende Sorry I haven't kept up with this recently. I'm happy to say that I was able to do some preliminary testing and this appears to solve the problem! **_Thanks!_** ð

[GitHub] nifi issue #1241: NIFI-3039 Provenance Repository - Fix PurgeOldEvent and Ro...

2018-02-08 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1241 Overcome by events. ---

[GitHub] nifi issue #1241: NIFI-3039 Provenance Repository - Fix PurgeOldEvent and Ro...

2018-01-03 Thread jskora
Github user jskora commented on the issue: https://github.com/apache/nifi/pull/1241 @joewitt Since 0.x is end-of-life, this is OBE and should be closed. IIRC, the root problem was writing records first and then checking afterward to see if storage limits were exceeded