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

Importing groovy.time.* results in an error #1129

Closed
jjbrosnan opened this issue Aug 25, 2021 · 1 comment · Fixed by #3395
Closed

Importing groovy.time.* results in an error #1129

jjbrosnan opened this issue Aug 25, 2021 · 1 comment · Fixed by #3395
Assignees
Labels
bug Something isn't working console
Milestone

Comments

@jjbrosnan
Copy link
Contributor

jjbrosnan commented Aug 25, 2021

The following code (in a Deephaven Groovy console):

import groovy.time.*

Results in the following error and stack trace:

r-Scheduler-Serial-1 | .u.GroovyDeephavenSession | Invalid import: "import groovy.time.*;"
r-Scheduler-Serial-1 | i.d.g.s.SessionState      | Internal Error '12e4617c-5ec4-41d3-a2c5-219105f4df9b' java.lang.RuntimeException: Attempting to import a path that does not exist: import groovy.time.*;
        at io.deephaven.db.util.GroovyDeephavenSession.updateScriptImports(GroovyDeephavenSession.java:487)
        at io.deephaven.db.util.GroovyDeephavenSession.grepScriptImports(GroovyDeephavenSession.java:498)
        at io.deephaven.db.util.GroovyDeephavenSession.evaluate(GroovyDeephavenSession.java:229)
        at io.deephaven.db.util.AbstractScriptSession.evaluateScript(AbstractScriptSession.java:115)
        at io.deephaven.db.util.ScriptSession.evaluateScript(ScriptSession.java:71)
        at io.deephaven.grpc_api.console.ConsoleServiceGrpcImpl.lambda$null$8(ConsoleServiceGrpcImpl.java:176)
        at io.deephaven.grpc_api.session.SessionState$ExportBuilder.lambda$submit$1(SessionState.java:1303)
        at io.deephaven.grpc_api.session.SessionState$ExportObject.doExport(SessionState.java:866)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at io.deephaven.grpc_api.runner.DeephavenApiServerModule$ThreadFactory.lambda$newThread$0(DeephavenApiServerModule.java:146)
        at java.lang.Thread.run(Thread.java:748)

A clear and concise description of what the bug is.

Steps to reproduce

  1. Launch deephaven-core with Groovy
  2. import groovy.time.*
  3. An error will occur

Expected results

Not much to happen. The import should work, and you should be able to use the groovy time library.

Actual results

An error with the stack trace given above.

Additional details and attachments

If applicable, add any additional screenshots, logs, or other attachments to help explain your problem.

Versions

  • Deephaven: community
  • OS: Big Sur
  • Browser: Chrome
  • Docker: 20.10.7
@jjbrosnan jjbrosnan added bug Something isn't working triage labels Aug 25, 2021
@nbauernfeind nbauernfeind added this to the Sept 2021 milestone Sep 8, 2021
@mattrunyon
Copy link
Contributor

mattrunyon commented Sep 21, 2021

Some more info. If you do the following, there is no error (no idea if it is importing everything). If you start with the wildcard import then there's an error.

import groovy.time.TimeCategory
import groovy.time.*

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working console
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants