Merge changes from main to native_cql_changes

[tatu-at-datastax]

What this PR does:

Merges changes from main to native_cql_changes branch

Which issue(s) this PR fixes:
N/A

Checklist

• Changes manually tested
• Automated Tests added/updated
• Documentation added/updated
• CLA Signed: DataStax CLA

[Yuqi-Du]

I am seeing MeteredCommandProcessorTest 3 unit tests failing. But it seems that they also fail in native_cql_changes branch.

Integration Tests all passed.

[tatu-at-datastax]

I am seeing MeteredCommandProcessorTest 3 unit tests failing. But it seems that they also fail in native_cql_changes branch.

Integration Tests all passed.

I see 4th failure which may be due to merge:

[ERROR] Failures: 
[ERROR]   ThrowableCommandResultSupplierTest$Get.authenticationError:104 
[List check single element] (1 failure)
-- failure 1 --
expected: "UNAUTHENTICATED: Invalid token"
 but was: "UNAUTHENTICATED"

[Yuqi-Du]

I am seeing MeteredCommandProcessorTest 3 unit tests failing. But it seems that they also fail in native_cql_changes branch.
Integration Tests all passed.

I see 4th failure which may be due to merge:

[ERROR] Failures: 
[ERROR]   ThrowableCommandResultSupplierTest$Get.authenticationError:104 
[List check single element] (1 failure)
-- failure 1 --
expected: "UNAUTHENTICATED: Invalid token"
 but was: "UNAUTHENTICATED"

Could you please pull again. I think I fixed this yesterday

[tatu-at-datastax]

I am seeing MeteredCommandProcessorTest 3 unit tests failing. But it seems that they also fail in native_cql_changes branch.
Integration Tests all passed.

I see 4th failure which may be due to merge:

[ERROR] Failures: 
[ERROR]   ThrowableCommandResultSupplierTest$Get.authenticationError:104 
[List check single element] (1 failure)
-- failure 1 --
expected: "UNAUTHENTICATED: Invalid token"
 but was: "UNAUTHENTICATED"

Could you please pull again. I think I fixed this yesterday

Ah. Yes, I see that one, thank you!

[tatu-at-datastax]

@maheshrajamani @kathirsvn Any concerns? Not sure why checks are not running (I guess CI does not pick up this PR somehow), but there are 3 unit test failures:

[ERROR] Failures: 
[ERROR]   MeteredCommandProcessorTest$CustomMetrics.errorMetrics:177 
Multiple Failures (1 failure)
-- failure 1 --
expected: 3
 but was: 76
[ERROR]   MeteredCommandProcessorTest$CustomMetrics.errorMetricsWithNoErrorCode:123 
Multiple Failures (1 failure)
-- failure 1 --
expected: 3
 but was: 76
[ERROR]   MeteredCommandProcessorTest$CustomMetrics.metrics:72 
Multiple Failures (1 failure)
-- failure 1 --
expected: 3
 but was: 76
[INFO] 
[ERROR] Tests run: 645, Failures: 3, Errors: 0, Skipped: 18

[maheshrajamani]

errorMetrics
@tatu-at-datastax
We have added histogram percentile metrics for command_processor_process, that adds entries to the meters. We need to add additional filter like !line.contains("quantile")

[tatu-at-datastax]

errorMetrics
@tatu-at-datastax
We have added histogram percentile metrics for command_processor_process, that adds entries to the meters. We need to add additional filter like !line.contains("quantile")

Looks like one possible filter that removes new bucketed entries is:

    && !line.contains("seconds_bucket")

Thanks!

[tatu-at-datastax]

Ok, so:

• Unit tests pass (locally and on CI)
• Integration tests pass locally
• Integration tests fail on CI

since CI was not yet enabled for native_cql_changes branch I think there's no new breakage so I will go ahead and merge (couldn't see an obvious fix to solve the issue, need more time).