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

CORE: #1661

Merged
merged 2 commits into from Jun 3, 2019

Conversation

Projects
None yet
2 participants
@manolama
Copy link
Member

commented Jun 2, 2019

  • Avoid a potential NPE in AQPC when closing if the query never parsed or
    made it to the planning stage.
  • Throw QEEs in the planner instead of illegals so we can mark them as 400s.

HTTP:

  • Change the health metrics from host to remote to avoid collisions in the
    reporter.

manolama added some commits Jun 2, 2019

CORE:
- Avoid a potential NPE in AQPC when closing if the query never parsed or
  made it to the planning stage.
- Throw QEEs in the planner instead of illegals so we can mark them as 400s.

HTTP:
- Change the health metrics from `host` to `remote` to avoid collisions in the
  reporter.
HTTP:
- Add a timer to run through the remote hosts and report their status as 0
  being down an 1 being up for the health checker.

@manolama manolama merged commit cef1d95 into OpenTSDB:3.0 Jun 3, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls First build on 3x_tweaks at 59.368%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.