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

Jruby profiler raises exception when using the holidays gem #4586

Closed
vassilios opened this Issue May 3, 2017 · 2 comments

Comments

Projects
None yet
3 participants
@vassilios

vassilios commented May 3, 2017

Environment

  • JRuby version (jruby -v)
    jruby 9.1.8.0 (2.3.1) 2017-03-06 90fc7ab OpenJDK 64-Bit Server VM 25.72-b15 on 1.8.0_72-internal-b15 +jit [linux-x86_64]

  • command line (flags, JRUBY_OPTS, etc)
    JRUBY_OPTS="--profile.api"

  • Operating system and platform (e.g. uname -a)
    Linux ip-172-30-3-223 3.13.0-74-generic #118-Ubuntu SMP Thu Dec 17 22:52:10 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Other relevant info

  • gem install holidays -v 5.5.0

Expected Behavior

Using jruby 1.7.22 (on same server)

$ jruby -v
jruby 1.7.22 (1.9.3p551) 2015-08-20 c28f492 on OpenJDK 64-Bit Server VM 1.8.0_72-internal-b15 +jit [linux-amd64]

$ jruby -rdate -rholidays -e 'puts Date.civil(2017,2,20).holiday?(:us)'
Profiling enabled; ^C shutdown will now dump profile info
true

Actual Behavior

$ jruby -v
jruby 9.1.8.0 (2.3.1) 2017-03-06 90fc7ab OpenJDK 64-Bit Server VM 25.72-b15 on 1.8.0_72-internal-b15 +jit [linux-x86_64]

$ jruby -rdate -rholidays -e 'puts Date.civil(2017,2,20).holiday?(:us)'
Profiling enabled; ^C shutdown will now dump profile info
LoadError: load error: holidays -- java.lang.UnsupportedOperationException: We do not know enough about the arity to convert it to a signature
  require at org/jruby/RubyKernel.java:961
  require at /opt/jruby-9.1.8.0/lib/ruby/stdlib/rubygems/core_ext/kernel_require.rb:133

@enebo enebo added this to the JRuby 9.1.9.0 milestone May 10, 2017

@enebo enebo closed this in 7f6e378 May 11, 2017

@vassilios

This comment has been minimized.

Show comment
Hide comment
@vassilios

vassilios May 11, 2017

Thank you!

vassilios commented May 11, 2017

Thank you!

@dewani-saurabh

This comment has been minimized.

Show comment
Hide comment
@dewani-saurabh

dewani-saurabh Oct 10, 2017

How did you solve the issue??

dewani-saurabh commented Oct 10, 2017

How did you solve the issue??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment