P3-Core-Dev-Team / P3-Q-A

This repo is for tracking features
Other
1 stars 0 forks source link

ADS-COC - COC Report Clarity from Erik - (Dell) #51

Closed kilam005 closed 7 months ago

kilam005 commented 8 months ago

In COC Report, For Table with 0 Expected Record Count, it shows data not ingested and the validation messages need to have additional details for the warnings with Spaces

image

Adding this ticket for tacking purpose. Originally raised by Erik (Dell) via Rani on 11 Jan 2024

kilam005 commented 8 months ago

Rani explained the scenerio on 24th Jan2024. Info presented on report needs to be reworded.

As per Harsha & others in call, the message should indicate the columns name alone that do not match the leading or trailing space conditions.

🚩Additionally, if the count of columns impacted is provided, it is good to have.

ClementJosh21 commented 8 months ago

@kilam005 We have made changes as above mentioned. Attaching the sample screenshot for reference.

Image

kilam005 commented 8 months ago

Suggestions for statement has been asked for confirmation from Implementation team.

image

kilam005 commented 8 months ago

This fixes are ready and verified in the expected version 2024.01.x. @selvakumarchandran to check priority needs from Rani / Kavitha (Dell) to check if this needs to be patched in 3.1.7_x (effectv build version)

Udhayanila8 commented 8 months ago

Issue fixed. Tested in 146 server. "Column names" following the - "has data with leading/trailing spaces" image

COC report COC_REPORT_088f1b2c-8ad0-4df3-b116-25481692b133.pdf

kilam005 commented 7 months ago

The issue was deployed in Effectv region. Expecting closure from Rani & team

raniangamuthu commented 7 months ago

verified and ok to close this issue

kilam005 commented 7 months ago

Based on above comment from @raniangamuthu, closing this issue