avshenuk commented on PR #16387: URL: https://github.com/apache/pinot/pull/16387#issuecomment-3133810964
> > > To maintain backward compatible and also fix the issue, we should consider adding back `originalValues` when the key doesn't exist in the unnested record. Could you give it a try and see if it works for the existing test and new added one? > > > > > > Why would it not exist in the unnested record? > > When we unnest a record, the original unnested fields are not included in the record. #13490 added the original fields to the record because the following transformer might use them. We don't want the original unnested fields to override the unnested fields if the key is the same Well. I’ve pushed the changed you asked me to. The test is still failing. Please take a look. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
