You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@metron.apache.org by nickwallen <gi...@git.apache.org> on 2018/03/16 13:18:14 UTC

[GitHub] metron pull request #966: METRON-1493 Unhelpful Error Message When Assignmen...

GitHub user nickwallen opened a pull request:

    https://github.com/apache/metron/pull/966

    METRON-1493 Unhelpful Error Message When Assignment Expressions Fail

    When executing an assignment expression that fails, the error message is less than helpful.  Prior to this PR the error message looks something like this.
    
    ```
    [Stellar]>>> p := 0/0
    [!] Assignment expression failed
    java.lang.IllegalArgumentException: Assignment expression failed
    	at org.apache.metron.stellar.common.shell.StellarResult.error(StellarResult.java:115)
    	at org.apache.metron.stellar.common.shell.specials.AssignmentCommand.execute(AssignmentCommand.java:82)
    	at org.apache.metron.stellar.common.shell.DefaultStellarShellExecutor.execute(DefaultStellarShellExecutor.java:252)
    	at org.apache.metron.stellar.common.shell.cli.StellarShell.execute(StellarShell.java:357)
    	at org.jboss.aesh.console.AeshProcess.run(AeshProcess.java:53)
    	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
    	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
    	at java.lang.Thread.run(Thread.java:748)
    ```
    
    This has now been fixed so that you can understand exactly what happened.
    
    ```
    [Stellar]>>> x := 0/0
    [!] / by zero
    java.lang.ArithmeticException: / by zero
    	at org.apache.metron.stellar.common.evaluators.ArithmeticEvaluator$ArithmeticEvaluatorFunctions.lambda$division$3(ArithmeticEvaluator.java:98)
    	at org.apache.metron.stellar.common.evaluators.ArithmeticEvaluator.evaluate(ArithmeticEvaluator.java:39)
    	at org.apache.metron.stellar.common.StellarCompiler.lambda$exitArithExpr_div$2(StellarCompiler.java:316)
    	at org.apache.metron.stellar.common.StellarCompiler$Expression.apply(StellarCompiler.java:190)
    	at org.apache.metron.stellar.common.BaseStellarProcessor.parse(BaseStellarProcessor.java:145)
    	at org.apache.metron.stellar.common.shell.DefaultStellarShellExecutor.executeStellar(DefaultStellarShellExecutor.java:401)
    	at org.apache.metron.stellar.common.shell.DefaultStellarShellExecutor.execute(DefaultStellarShellExecutor.java:257)
    	at org.apache.metron.stellar.common.shell.specials.AssignmentCommand.execute(AssignmentCommand.java:66)
    	at org.apache.metron.stellar.common.shell.DefaultStellarShellExecutor.execute(DefaultStellarShellExecutor.java:252)
    	at org.apache.metron.stellar.common.shell.cli.StellarShell.execute(StellarShell.java:357)
    	at org.jboss.aesh.console.AeshProcess.run(AeshProcess.java:53)
    	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    	at java.lang.Thread.run(Thread.java:745)
    ```
    
    To test this, launch the REPL and try it out.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/nickwallen/metron METRON-1493

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/metron/pull/966.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #966
    
----
commit 88ffa5be69e0901ac6c79952ffe164d2de4ef2a7
Author: Nick Allen <ni...@...>
Date:   2018-03-16T13:15:59Z

    METRON-1493 Unhelpful Error Message When Assignment Expressions Fail

----


---

[GitHub] metron pull request #966: METRON-1493 Unhelpful Error Message When Assignmen...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/metron/pull/966


---

[GitHub] metron issue #966: METRON-1493 Unhelpful Error Message When Assignment Expre...

Posted by cestella <gi...@git.apache.org>.
Github user cestella commented on the issue:

    https://github.com/apache/metron/pull/966
  
    +1 by inspection.


---