Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a different repository to download sigar artifacts. #12561

Merged
merged 1 commit into from
May 24, 2022

Conversation

abhishekagarwal87
Copy link
Contributor

JBoss repository seems to be down again and we have run into this before as well. This PR switches to a different repository in hope that availability is better for that repository. Eventually, we will have to get rid of sugar itself since it has not been upgraded in awhile.

This PR has:

  • been self-reviewed.

Copy link
Contributor

@cryptoe cryptoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

@abhishekagarwal87
Copy link
Contributor Author

it fixed the build on LGTM and doesn't need the whole CI to pass so I am going to merge it. Thank you @clintropolis and @cryptoe for review.

@abhishekagarwal87 abhishekagarwal87 merged commit 32fe4d1 into apache:master May 24, 2022
abhishekagarwal87 added a commit to abhishekagarwal87/druid that referenced this pull request Jun 3, 2022
@abhishekagarwal87 abhishekagarwal87 added this to the 0.23.0 milestone Jun 3, 2022
abhishekagarwal87 added a commit that referenced this pull request Jun 3, 2022
)

* Service stdout log files, move logs to log/. (#12570)

* Service stdout log files, move logs to log/.

Two changes that make log behavior cleaner:

1) Redirect messages from the Java runtime to their own log files.
   Otherwise, they would get jumbled up in the output of the all-in-one
   start command.

2) Use log/ instead of bin/log/ for the default log directory. Makes them
   easier to find.

Additionally, add documentation about how to avoid the reflective
access warnings in Java 11.

* Spelling.

* See if code formatting affects spelling.

* Update .travis.yml

* Use a different repository to download sigar artifacts. (#12561)

* Revert "Update .travis.yml"

This reverts commit ddd7be2.

Co-authored-by: Gian Merlino <gianmerlino@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants