[GitHub] [nifi] ravinarayansingh commented on pull request #7604: NIFI-11946 Added preserveSourceProperties query parameter to create flow API

2023-08-24 Thread via GitHub
ravinarayansingh commented on PR #7604: URL: https://github.com/apache/nifi/pull/7604#issuecomment-1692325610 Hi @exceptionfactory let me know if I have to do any changes from my side -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [nifi] ravinarayansingh commented on pull request #7604: NIFI-11946 Added preserveSourceProperties query parameter to create flow API

2023-08-23 Thread via GitHub
ravinarayansingh commented on PR #7604: URL: https://github.com/apache/nifi/pull/7604#issuecomment-1690738085 > Thanks @ravinarayansingh, making adjustments to only retain the created timestamp, but always update the modified timestamp, seems like the best way forward. I also noted a

[GitHub] [nifi] ravinarayansingh commented on pull request #7604: NIFI-11946 Added preserveSourceProperties query parameter to create flow API

2023-08-22 Thread via GitHub
ravinarayansingh commented on PR #7604: URL: https://github.com/apache/nifi/pull/7604#issuecomment-1689023712 > Is there some particular reason for not wanting to reflect the current timestamp when importing flows? Thanks @exceptionfactory for quick response No, there is no

[GitHub] [nifi] ravinarayansingh commented on pull request #7604: NIFI-11946 Added preserveSourceProperties query parameter to create flow API

2023-08-22 Thread via GitHub
ravinarayansingh commented on PR #7604: URL: https://github.com/apache/nifi/pull/7604#issuecomment-1688954316 > > > Thanks for the contribution @ravinarayansingh. > > > On initial review, supporting this feature raises some concerns. The created and modified timestamps are intended to

[GitHub] [nifi] ravinarayansingh commented on pull request #7604: NIFI-11946 Added preserveSourceProperties query parameter to create flow API

2023-08-22 Thread via GitHub
ravinarayansingh commented on PR #7604: URL: https://github.com/apache/nifi/pull/7604#issuecomment-1688779867 > Thanks for the contribution @ravinarayansingh. > > On initial review, supporting this feature raises some concerns. The created and modified timestamps are intended to