Skip to content

Commit

Permalink
Add raise and explanatory comment so if you try to use the v1 retriev…
Browse files Browse the repository at this point in the history
…al service in this branch, it fails
  • Loading branch information
Noah Gibbs committed Jan 21, 2013
1 parent 3cd1838 commit f1cd1dd
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions lib/hastur-server/service/retrieval.rb
Expand Up @@ -13,6 +13,17 @@
require "multi_json"
require "termite"

#
# Do you need a v1 retrieval API? Then you should rewind the git
# repo back to when it existed, worked, and ran on MRI Ruby rather
# than JRuby. This is no longer tested, and in several ways is
# guaranteed to not work, or just have various differences.
#
# Since it exists at all for API compatibility reasons, various
# differences is not a good thing.
#
raise "This no longer works in current JRuby Hastur!"

# TODO(noah): Add parameter validation - alert on bad params?

# TODO(noah): Override for JRuby
Expand Down

0 comments on commit f1cd1dd

Please sign in to comment.