Bug fix - 948 | Added else if condition to get tables into InSync state

[TejasRGitHub]

Feature or Bugfix

• Bugfix

Detail

• Added else if condition for getting the glue tables which exist on the AWS account and also present in RDS dataset tables set the LastGlueTableStatus to InSync.

Testing

1. Created a dataset with tables
2. Marked the dataset tables LastGlueTableStatus to Deleted.
3. Checked if the table are now NOT displayed.
4. Deployed data.all with this fix
5. Now the dataset tables are present on the UI.

Relates

• Glue table not visible after upgrading to v2.2 with enable auto created pivot role #979

Security

Please answer the questions below briefly where applicable, or write N/A. Based on
OWASP 10.

• Does this PR introduce or modify any input fields or queries - this includes
  fetching data from storage outside the application (e.g. a database, an S3 bucket)? N/A
  • Is the input sanitized?
  • What precautions are you taking before deserializing the data you consume?
  • Is injection prevented by parametrizing queries?
  • Have you ensured no eval or similar functions are used?
• Does this PR introduce any functionality or component that requires authorization? No
  • How have you ensured it respects the existing AuthN/AuthZ mechanisms?
  • Are you logging failed auth attempts?
• Are you using or adding any cryptographic features? No
  • Do you use a standard proven implementations?
  • Are the used keys controlled by the customer? Where are they stored?
• Are you introducing any new policies/roles/users? No
  • Have you used the least-privilege principle? How?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[dlpzx]

Looks good I will test in AWS. Thanks @TejasRGitHub !!

[noah-paige]

Tested in AWS:

• CICD Pipeline Succeeds
• Removed glue:... permissions for pivot role + Synchronize on UI --> Tables update with Deleted Status and no show on UI
• Re-add glue:... permissions for pivot role + Synchronize on UI --> Tables update with InSync Status and show on UI

[noah-paige]

lgtm, approving!