exceptionfactory commented on PR #8903:
URL: https://github.com/apache/nifi/pull/8903#issuecomment-2145580852
> @exceptionfactory In `PutElasticsearchHttp` on line 252 `logger.error("No
value for index in for {}, transferring to failure", id_attribute, file);`
there is only one parameter (`{}`) specified in the logging statement yet there
are two parameter arguments ( `id_attribute, file`) given.
>
> should the log statement be `logger.error("No value for index in {} for
{}, transferring to failure", id_attribute, file);` ?
@dan-s1 On further consideration, I'm not sure it is worth back-porting this
change. Especially for this type of syntax change, there is very little value
in bringing it over, given the current focus on getting 2.0 finalized.
--
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.
To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
