Revert "Allow alien value in MVEL-based derivations (#1120) and remov… - Githubissues

feathr-ai / feathr

Feathr – A scalable, unified data and AI engineering platform for enterprise

https://join.slack.com/t/feathrai/shared_invite/zt-1ffva5u6v-voq0Us7bbKAw873cEzHOSg

Apache License 2.0

1.99k stars 260 forks source link

Revert "Allow alien value in MVEL-based derivations (#1120) and remov… #1139

Closed anirudhagar13 closed 1 year ago

anirudhagar13 commented 1 year ago

…e stdout statements"

This reverts commit 55290e7824c217bd563447ff2df04ed238e5dd4f.

Description

Reverted commit for allowing alien values within Mvel as it caused failure for many flows, specifically around being able to use derived features.
The commit also removes some of the bigger stdout statements as they were overloading spark resources, causing client jobs to fail.

Resolves #XXX

How was this PR tested?

User MP flows were triggered with this change.

Does this PR introduce any user-facing changes?

[ ^] No. You can skip the rest of this section.
[ ] Yes. Make sure to clarify your proposed changes.