[AOCO] remove the useless if condition
according to the comment, when `GetAllAOCSFileSegInfo_pg_aocsseg_rel`
was called from `gp_aoseg_history` after an upgrade, the `state` and
`modcount` might not be set, but no `formatversion`, the if conditon
should always be true, so just remove it to make the code more clean.
@@ -387,7 +387,6 @@ GetAllAOCSFileSegInfo_pg_aocsseg_rel(int numOfColumns,
aocs_seginfo->modcount = DatumGetInt64(d[Anum_pg_aocs_modcount - 1]);
Assert(!null[Anum_pg_aocs_formatversion - 1]);
- if (!null[Anum_pg_aocs_formatversion - 1])
aocs_seginfo->formatversion = DatumGetInt16(d[Anum_pg_aocs_formatversion - 1]);
Assert(!null[Anum_pg_aocs_state - 1] || snapshot == SnapshotAny);
