amousavigourabi commented on code in PR #1253:
URL: https://github.com/apache/parquet-mr/pull/1253#discussion_r1452323505
##########
pom.xml:
##########
@@ -713,13 +713,12 @@
</build>
</profile>
- <!-- Profile for CI tests to have less output -->
+ <!-- Profile for tests to have more output -->
<profile>
- <id>ci-test</id>
+ <id>verbose-test</id>
Review Comment:
As of now our output is verbose by default, with the `ci-test` profile being
more compact. I tried looking into enabling `ci-test` by default, but when a
profile is `activeByDefault`, it gets deactivated whenever another profile
becomes active. The problem then is that `parquet-mr` has a few profiles that
get activated automatically depending on the environment Maven is run in, so we
cannot rely on it in that way. Because of this, the PR gets rid of the slimmed
down `ci-test` profile and replaces it with a verbose one, such that the
(currently default) verbose output becomes opt-in, with the more lean output
just being the default setting.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
