Revert "Update Agroal to 1.8"

[gsmet]

PLEASE DON'T MERGE AS MAYBE LUIS WILL COME WITH A QUICK RELEASE BEFORE I RELEASE FINAL

(and I don't use WIP as I want a proper CI run)

This reverts commit bd3016c.

1.8 introduces a log message at each startup:
2020-05-21 16:35:10,006 INFO [io.agr.pool] (main) Datasource '': Pool interceptors: []

While not a big issue, it's definitely a usability regression
consisdering we are working hard on keeping our logs as less verbose as
possible.

I think we should demote this log message to DEBUG.

/cc @barreiro

[machi1990]

1.8 introduces a log message at each startup:
2020-05-21 16:35:10,006 INFO [io.agr.pool] (main) Datasource '': Pool interceptors: []

While not a big issue, it's definitely a usability regression
consisdering we are working hard on keeping our logs as less verbose as
possible.

Was filtering out the log instead with a build item considered?

[gsmet]

That could have been a good idea but the start of the message (before you get the database name) is not easily identifiable unfortunately. And we don't use regexps, we compare the start of the message.

That's why I passed on this option.

[machi1990]

That could have been a good idea but the start of the message (before you get the database name) is not easily identifiable unfortunately. And we don't use regexps, we compare the start of the message.

That's why I passed on this option.

Could have been a good fix till an upstream release. Thanks for answering / looking.

[gsmet]

Superseded by #9596 .