fix: Revert "feat: Enable stream feature view materialization (#2798)"#2806
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: felixwang9817 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/lgtm |
)" This reverts commit a06700d. Signed-off-by: Felix Wang <[email protected]>
felixwang9817
force-pushed
the
revert_sfv_materialization
branch
from
6690ccc to
26e016e
Compare
|
New changes are detected. LGTM label has been removed. |
Codecov Report
@@ Coverage Diff @@
## master #2806 +/- ##
==========================================
- Coverage 59.67% 59.62% -0.06%
==========================================
Files 172 173 +1
Lines 15311 15271 -40
==========================================
- Hits 9137 9105 -32
+ Misses 6174 6166 -8
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
4 participants
This reverts commit a06700d.
What this PR does / why we need it: This PR reverts #2798, which passed but causes tests to be extremely flaky, and is therefore currently blocking other PRs. I will take a closer look at #2798 and rewrite the testing logic so we can avoid causing more test flakiness.
Which issue(s) this PR fixes:
Fixes #