Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

FADI user guide update #108

Open
leprincev opened this issue Mar 5, 2020 · 2 comments · Fixed by #135
Open

FADI user guide update #108

leprincev opened this issue Mar 5, 2020 · 2 comments · Fixed by #135
Assignees
Milestone

Comments

@leprincev
Copy link

leprincev commented Mar 5, 2020

FADI User guide update

  1. PutDatabaseRecord processor

In the FADI User Guide, in the chapter 3 Nifi "measurement ingestion" I suggest to make a reference to the arrow when mentioning the first "Go To" int the "PutDatabaseRecord processor" section.
image

For example by inserting a line or a graphical reference to the arrow:
right-click > Configure > Properties tab > Record Reader > Create a new service > CSV Reader
Open the "Go To" section on the arrow on the right: image
Go To > Configure > Properties >
Treat First Line as Header: true

  1. PutDatabaseRecord processor
    If it is the first use / creation on Nifi, the "DBCPConnectionPool " is not yet created, therefore you first need to create it.

On the 4th dot, I suggest to replace the:
right-click > Configure > Properties tab > Database Connection Pooling Service >
with:
right-click > Configure > Properties tab > Database Connection Pooling Service > Create a new service > DBCPConnectionPool

@leprincev leprincev added the bug Something isn't working label Mar 5, 2020
@fzalila fzalila self-assigned this Mar 5, 2020
@fzalila fzalila removed the bug Something isn't working label Mar 5, 2020
@fzalila
Copy link
Contributor

fzalila commented Mar 9, 2020

@leprincev
Thank you for your feedback.
I suggest that you improve the userguide with your remarks and do then a PR to integrate them in the FADI repository.

Best,

Faiez.

@fzalila fzalila assigned leprincev and unassigned fzalila Mar 9, 2020
@banzo banzo added this to the 0.1.2 milestone May 27, 2020
@banzo banzo self-assigned this Jul 14, 2020
@banzo
Copy link
Contributor

banzo commented Apr 21, 2021

@banzo banzo mentioned this issue May 4, 2021
@banzo banzo linked a pull request May 4, 2021 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants