Fix Parquet's fallback field id assignment

[rdsr]

As discussed in #580, we need to fix how Parquet assigns fallback ids. Today, it checks if the schema has at least one id missing, it assigns fallback ids to all fields, but as pointed out by @rdblue it should check if any id is set, we should not use fallback assignment.

[rdblue]

For context, I think the idea was to catch when fallback was needed as quickly as possible. That works for cases where the schema either has IDs for every field or has no IDs, as in our environment when I wrote the logic. But that isn't necessarily the case for all data files, so we should update the logic to be more careful.

[github-actions]

This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occurs. To permanently prevent this issue from being considered stale, add the label 'not-stale', but commenting on the issue is preferred when possible.

[github-actions]

This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale'