You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by GitBox <gi...@apache.org> on 2021/02/10 14:07:19 UTC
[GitHub] [nifi] timeabarna opened a new pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
timeabarna opened a new pull request #4815:
URL: https://github.com/apache/nifi/pull/4815
…xception was thrown in uploadContent()
https://issues.apache.org/jira/browse/NIFI-8200
#### Description of PR
delete temp file if exception was thrown in uploadContent()
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
### For all changes:
- [ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?
- [ ] Does your PR title start with **NIFI-XXXX** where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
- [ ] Has your PR been rebased against the latest commit within the target branch (typically `main`)?
- [ ] Is your initial contribution a single, squashed commit? _Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not `squash` or use `--force` when pushing to allow for clean monitoring of changes._
### For code changes:
- [ ] Have you ensured that the full suite of tests is executed via `mvn -Pcontrib-check clean install` at the root `nifi` folder?
- [ ] Have you written or updated unit tests to verify your changes?
- [ ] Have you verified that the full build is successful on JDK 8?
- [ ] Have you verified that the full build is successful on JDK 11?
- [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?
- [ ] If applicable, have you updated the `LICENSE` file, including the main `LICENSE` file under `nifi-assembly`?
- [ ] If applicable, have you updated the `NOTICE` file, including the main `NOTICE` file found under `nifi-assembly`?
- [ ] If adding new Properties, have you added `.displayName` in addition to .name (programmatic access) for each of the new properties?
### For documentation related changes:
- [ ] Have you ensured that format looks appropriate for the output in which it is rendered?
### Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] timeabarna commented on a change in pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
timeabarna commented on a change in pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#discussion_r576091826
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/java/org/apache/nifi/processors/azure/storage/PutAzureDataLakeStorage.java
##########
@@ -126,6 +126,9 @@ public void onTrigger(final ProcessContext context, final ProcessSession session
if (length > 0) {
try (final InputStream rawIn = session.read(flowFile); final BufferedInputStream bufferedIn = new BufferedInputStream(rawIn)) {
uploadContent(fileClient, bufferedIn, length);
+ } catch (Exception e) {
+ fileClient.delete();
+ throw e;
Review comment:
Thanks for your help, corrected in the next commit
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] timeabarna commented on pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
timeabarna commented on pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#issuecomment-779900021
> @timeabarna What happens when an existing file is being overwritten with the Conflict Resolution Strategy set to replace?
@jfrazee Thanks for pointing this out. I've checked the original version and the behaviour is the following:
- The original file has been replaced with a 0 byte file
- uploading content started
- if uploading failed the original data was gone and the 0 byte file remained
With this current modification the behaviour:
- file replaced with a 0 byte file
- uploading content started
- if failed the 0 byte file was removed (if deletion fails, the 0 byte file remains)
I'm exploring the possibility using temp files instead of inline replacement however I think this should be covered by another Jira ticket.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] pgyori commented on a change in pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
pgyori commented on a change in pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#discussion_r573941975
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/java/org/apache/nifi/processors/azure/storage/PutAzureDataLakeStorage.java
##########
@@ -126,6 +126,9 @@ public void onTrigger(final ProcessContext context, final ProcessSession session
if (length > 0) {
try (final InputStream rawIn = session.read(flowFile); final BufferedInputStream bufferedIn = new BufferedInputStream(rawIn)) {
uploadContent(fileClient, bufferedIn, length);
+ } catch (Exception e) {
+ fileClient.delete();
+ throw e;
Review comment:
Exception e is suppressed if fileClient.delete() also throws an exception. My recommendation is to put a try-catch-finally around delete(). In the catch only an error message needs to be logged, and in the finally the original exception 'e' can be thrown forward.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] jfrazee commented on pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
jfrazee commented on pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#issuecomment-779926853
@timeabarna That's what I figured. It's not ideal, but this isn't a regression. Can we clarify the delete behavior in the docs?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] jfrazee commented on pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
jfrazee commented on pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#issuecomment-782929616
Thanks for this @timeabarna +1
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] timeabarna commented on pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
timeabarna commented on pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#issuecomment-780339935
> @timeabarna That's what I figured. It's not ideal, but this isn't a regression. Can we clarify the delete behavior in the docs?
@jfrazee Sure thing, will update the docs accordingly
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] jfrazee commented on pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
jfrazee commented on pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#issuecomment-779444703
@timeabarna What happens when an existing file is being overwritten with the Conflict Resolution Strategy set to replace?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] jfrazee commented on pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
jfrazee commented on pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#issuecomment-779932920
@timeabarna Oh, also, yes it makes sense to have another JIRA for some more sophisticated handling, which should probably be done. Ideally we should try to have the same behavior we can get with HDFS, which I think is achievable since renames are atomic.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] timeabarna commented on a change in pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
timeabarna commented on a change in pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#discussion_r578969886
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/resources/docs/org.apache.nifi.processors.azure.storage.PutAzureDataLakeStorage/additionalDetails.html
##########
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html lang="en">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+ http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+<head>
+ <meta charset="utf-8" />
+ <title>QuerySplunkIndexingStatus</title>
+ <link rel="stylesheet" href="/nifi-docs/css/component-usage.css" type="text/css" />
+</head>
+
+<body>
+<h2>PutAzureDataLakeStorage</h2>
+
+<p>
+ This processor is responsible for uploading files to Azure Data Lake Storage servers.
+</p>
+
+<h3>File uploading and cleanup process</h3>
+
+<h4>New file:</h4>
+
+<ol>
+ <li>An empty file is created.</li>
+ <li>Content is appended to file.</li>
+ <li>In case append failure the file is deleted.</li>
+ <li>In case file deletion failure the empty file remains on the server.</li>
+</ol>
+
+<h4>Existing file:</h4>
+
+<ul>
+ <li>Processors with "Fail" conflict resolution strategy will be directed to "Failure" relationship.</li>
+ <li>Processors with "Ignore" conflict resolution strategy will be directed to "Success" relationship.</li>
+ <li>Processors with "replace" conflict resolution strategy:</li>
+
+ <ol>
+ <li>An empty file overwrites the existing file, the original file is lost.</li>
+ <li>Content is appended to file.</li>
+ <li>In case append failure the file is deleted.</li>
+ <li>In case file deletion failure the empty file remains on the server.</li>
+ </ol>
+</ul>
+
+</body>
+</html>
Review comment:
All suggestions have been applied. Thanks Joey for your help.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] jfrazee commented on a change in pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
jfrazee commented on a change in pull request #4815:
URL: https://github.com/apache/nifi/pull/4815#discussion_r578630400
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/resources/docs/org.apache.nifi.processors.azure.storage.PutAzureDataLakeStorage/additionalDetails.html
##########
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html lang="en">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+ http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+<head>
+ <meta charset="utf-8" />
+ <title>QuerySplunkIndexingStatus</title>
+ <link rel="stylesheet" href="/nifi-docs/css/component-usage.css" type="text/css" />
+</head>
+
+<body>
+<h2>PutAzureDataLakeStorage</h2>
Review comment:
```suggestion
```
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/resources/docs/org.apache.nifi.processors.azure.storage.PutAzureDataLakeStorage/additionalDetails.html
##########
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html lang="en">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+ http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+<head>
+ <meta charset="utf-8" />
+ <title>QuerySplunkIndexingStatus</title>
+ <link rel="stylesheet" href="/nifi-docs/css/component-usage.css" type="text/css" />
+</head>
+
+<body>
+<h2>PutAzureDataLakeStorage</h2>
+
+<p>
+ This processor is responsible for uploading files to Azure Data Lake Storage servers.
+</p>
+
+<h3>File uploading and cleanup process</h3>
+
+<h4>New file:</h4>
+
+<ol>
+ <li>An empty file is created.</li>
+ <li>Content is appended to file.</li>
+ <li>In case append failure the file is deleted.</li>
+ <li>In case file deletion failure the empty file remains on the server.</li>
+</ol>
+
+<h4>Existing file:</h4>
+
+<ul>
+ <li>Processors with "Fail" conflict resolution strategy will be directed to "Failure" relationship.</li>
+ <li>Processors with "Ignore" conflict resolution strategy will be directed to "Success" relationship.</li>
+ <li>Processors with "replace" conflict resolution strategy:</li>
+
+ <ol>
+ <li>An empty file overwrites the existing file, the original file is lost.</li>
+ <li>Content is appended to file.</li>
+ <li>In case append failure the file is deleted.</li>
+ <li>In case file deletion failure the empty file remains on the server.</li>
+ </ol>
+</ul>
+
+</body>
+</html>
Review comment:
Can you add a trailing newline?
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/resources/docs/org.apache.nifi.processors.azure.storage.PutAzureDataLakeStorage/additionalDetails.html
##########
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html lang="en">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+ http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+<head>
+ <meta charset="utf-8" />
+ <title>QuerySplunkIndexingStatus</title>
+ <link rel="stylesheet" href="/nifi-docs/css/component-usage.css" type="text/css" />
+</head>
+
+<body>
+<h2>PutAzureDataLakeStorage</h2>
+
+<p>
+ This processor is responsible for uploading files to Azure Data Lake Storage servers.
+</p>
+
+<h3>File uploading and cleanup process</h3>
+
+<h4>New file:</h4>
Review comment:
```suggestion
<h4>New file</h4>
```
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/resources/docs/org.apache.nifi.processors.azure.storage.PutAzureDataLakeStorage/additionalDetails.html
##########
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html lang="en">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+ http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+<head>
+ <meta charset="utf-8" />
+ <title>QuerySplunkIndexingStatus</title>
Review comment:
```suggestion
<title>PutAzureDataLakeStorage</title>
```
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/resources/docs/org.apache.nifi.processors.azure.storage.PutAzureDataLakeStorage/additionalDetails.html
##########
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html lang="en">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+ http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+<head>
+ <meta charset="utf-8" />
+ <title>QuerySplunkIndexingStatus</title>
+ <link rel="stylesheet" href="/nifi-docs/css/component-usage.css" type="text/css" />
+</head>
+
+<body>
+<h2>PutAzureDataLakeStorage</h2>
+
+<p>
+ This processor is responsible for uploading files to Azure Data Lake Storage servers.
+</p>
+
+<h3>File uploading and cleanup process</h3>
+
+<h4>New file:</h4>
+
+<ol>
+ <li>An empty file is created.</li>
+ <li>Content is appended to file.</li>
+ <li>In case append failure the file is deleted.</li>
+ <li>In case file deletion failure the empty file remains on the server.</li>
+</ol>
+
+<h4>Existing file:</h4>
+
+<ul>
+ <li>Processors with "Fail" conflict resolution strategy will be directed to "Failure" relationship.</li>
+ <li>Processors with "Ignore" conflict resolution strategy will be directed to "Success" relationship.</li>
Review comment:
```suggestion
<li>Processors with "fail" conflict resolution strategy will be directed to "Failure" relationship.</li>
<li>Processors with "ignore" conflict resolution strategy will be directed to "Success" relationship.</li>
```
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/resources/docs/org.apache.nifi.processors.azure.storage.PutAzureDataLakeStorage/additionalDetails.html
##########
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html lang="en">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+ http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+<head>
+ <meta charset="utf-8" />
+ <title>QuerySplunkIndexingStatus</title>
+ <link rel="stylesheet" href="/nifi-docs/css/component-usage.css" type="text/css" />
+</head>
+
+<body>
+<h2>PutAzureDataLakeStorage</h2>
+
+<p>
+ This processor is responsible for uploading files to Azure Data Lake Storage servers.
+</p>
+
+<h3>File uploading and cleanup process</h3>
+
+<h4>New file:</h4>
+
+<ol>
+ <li>An empty file is created.</li>
+ <li>Content is appended to file.</li>
+ <li>In case append failure the file is deleted.</li>
+ <li>In case file deletion failure the empty file remains on the server.</li>
+</ol>
+
+<h4>Existing file:</h4>
Review comment:
```suggestion
<h4>Existing file</h4>
```
##########
File path: nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/resources/docs/org.apache.nifi.processors.azure.storage.PutAzureDataLakeStorage/additionalDetails.html
##########
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html lang="en">
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+ http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+<head>
+ <meta charset="utf-8" />
+ <title>QuerySplunkIndexingStatus</title>
+ <link rel="stylesheet" href="/nifi-docs/css/component-usage.css" type="text/css" />
+</head>
+
+<body>
+<h2>PutAzureDataLakeStorage</h2>
+
+<p>
+ This processor is responsible for uploading files to Azure Data Lake Storage servers.
Review comment:
```suggestion
This processor is responsible for uploading files to Azure Data Lake Storage Gen2.
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nifi] jfrazee closed pull request #4815: NIFI-8200: Modifying PutAzureDataLakeStorage to delete temp file if e…
Posted by GitBox <gi...@apache.org>.
jfrazee closed pull request #4815:
URL: https://github.com/apache/nifi/pull/4815
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org