-
Notifications
You must be signed in to change notification settings - Fork 27
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
Feature/cdr 473 contains interpreter #379
Conversation
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
see CDR-473
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The noted issues are somewhat trivial.
The bigger issue is that the interpreter contains a whole lot of recursion, which makes the code harder to read/understand/debug. For the current goal of a prototype new engine this is fine, but i suspect there is some performance and readability/understandability to be gained by refactoring this later on.
aql/src/main/java/org/ehrbase/aql/dto/path/predicate/PredicateHelper.java
Show resolved
Hide resolved
...ata/src/main/java/org/ehrbase/test_data/operationaltemplate/OperationalTemplateTestData.java
Outdated
Show resolved
Hide resolved
web-template/src/main/java/org/ehrbase/webtemplate/interpreter/Interpreter.java
Show resolved
Hide resolved
web-template/src/main/java/org/ehrbase/webtemplate/interpreter/Interpreter.java
Show resolved
Hide resolved
web-template/src/main/java/org/ehrbase/webtemplate/interpreter/Interpreter.java
Outdated
Show resolved
Hide resolved
web-template/src/main/java/org/ehrbase/webtemplate/interpreter/Interpreter.java
Outdated
Show resolved
Hide resolved
web-template/src/main/java/org/ehrbase/webtemplate/interpreter/Interpreter.java
Outdated
Show resolved
Hide resolved
web-template/src/main/java/org/ehrbase/webtemplate/interpreter/InterpreterInput.java
Outdated
Show resolved
Hide resolved
web-template/src/main/java/org/ehrbase/webtemplate/interpreter/MatcherUtil.java
Show resolved
Hide resolved
web-template/src/test/java/org/ehrbase/webtemplate/interpreter/InterpreterTest.java
Show resolved
Hide resolved
Also please add an entry to CHANGELOG.md |
see CDR-473
see CDR-473
see CDR-473
see CDR-473
Kudos, SonarCloud Quality Gate passed! |
No description provided.