[BUG] timestamp is in 1000 sec with connector cloudwatch metrics

[pjoe]

Describe the bug
The docs says the metric_samples.timestamp should be in unix epoch seconds, but it look to be in 1000 seconds*

To Reproduce
Steps to reproduce the behavior:

1. Query a metric sample with the connector SELECT timestamp FROM metric_samples WHERE <some filtering> LIMIT 1
2. Notice the value looks to be in 1000 secs and not secs since unix epoch

Expected behavior
timestamp in unix epoch secs, as per docs

Screenshots / Exceptions / Errors

Connector Details (please complete the following information):

• Version: 2024.42.3
• Name cloudwatch-metrics

Additional context
Don't think you need the divide by 1000 here:

aws-athena-query-federation/athena-cloudwatch-metrics/src/main/java/com/amazonaws/athena/connectors/cloudwatch/metrics/MetricsRecordHandler.java

Line 266 in 3a306dd

long timestamp = timestamps.get(sampleNum).getEpochSecond() / 1000;

[colax94]

+1 for this, it is not possible to use the connector at the moment because of this