FearfulTomcat27 opened a new pull request, #15231: URL: https://github.com/apache/iotdb/pull/15231
This pull request includes several changes to update the error messages in various accumulator classes to use more specific aggregation names. The changes ensure that the error messages accurately reflect the aggregation type being processed. ### Updates to error messages in accumulator classes: * [`CountAllAccumulator.java`](diffhunk://#diff-ab1a955899e99e9fd127377787e46ea3cd983fb70b021b4e5076b7a8502e4803L53-R53): Updated error message to specify "CountAll" instead of "Count". * [`FirstByAccumulator.java`](diffhunk://#diff-908e670d1e5d6f64092c4208c1049b4d31b1d887292ee759c2a5f64c05efdf2cL173-R173): Updated error message to specify "FirstBy" instead of "First". * [`LastByAccumulator.java`](diffhunk://#diff-2dba4f7c3539cf481c64ff2e263385e19ecc6391698f2cd25148cd020b9f9738L197-R197): Updated error message to specify "LastBy" instead of "Last". * [`MaxAccumulator.java`](diffhunk://#diff-35d74ec378ba585d597ad12fe77fd7adc0e3d09397b4ecba67b554cb449ff4dbL84-R84): Updated error messages to specify "Max Aggregation" instead of "LastValue" or "Last". [[1]](diffhunk://#diff-35d74ec378ba585d597ad12fe77fd7adc0e3d09397b4ecba67b554cb449ff4dbL84-R84) [[2]](diffhunk://#diff-35d74ec378ba585d597ad12fe77fd7adc0e3d09397b4ecba67b554cb449ff4dbL233-R233) * [`MinAccumulator.java`](diffhunk://#diff-f5b514d97a70a943697de55b60761ee5e661c84240a7eea29df85b7b1a65b42aL84-R84): Updated error messages to specify "Min Aggregation" instead of "LastValue" or "Last". [[1]](diffhunk://#diff-f5b514d97a70a943697de55b60761ee5e661c84240a7eea29df85b7b1a65b42aL84-R84) [[2]](diffhunk://#diff-f5b514d97a70a943697de55b60761ee5e661c84240a7eea29df85b7b1a65b42aL233-R233) * [`SumAccumulator.java`](diffhunk://#diff-990d3ca558edc4d7877eec16a2fdbec2fc3009c7ca1f581106eaa703b4195b37L76-R76): Updated error messages to specify "Sum Aggregation" instead of "aggregation Sum". [[1]](diffhunk://#diff-990d3ca558edc4d7877eec16a2fdbec2fc3009c7ca1f581106eaa703b4195b37L76-R76) [[2]](diffhunk://#diff-990d3ca558edc4d7877eec16a2fdbec2fc3009c7ca1f581106eaa703b4195b37L104-R104) * [`TableVarianceAccumulator.java`](diffhunk://#diff-ce462ea36c9894a0f1f6101961c9388ba033267bd29da268d7b22ea042821a93L88-R88): Updated error messages to specify "Variance Aggregation" instead of "aggregation variance". [[1]](diffhunk://#diff-ce462ea36c9894a0f1f6101961c9388ba033267bd29da268d7b22ea042821a93L88-R88) [[2]](diffhunk://#diff-ce462ea36c9894a0f1f6101961c9388ba033267bd29da268d7b22ea042821a93L115-R115) * [`GroupedAvgAccumulator.java`](diffhunk://#diff-136bebef2dd74942c83dd4f87206a09edfe887c6cc95095eae51305438b95cb6L86-R86): Updated error message to specify "AVG Aggregation" instead of "aggregation AVG". * [`GroupedExtremeAccumulator.java`](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b074c8L70-R70): Updated error messages to specify "Extreme Aggregation" instead of generic "Unsupported data type". [[1]](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b074c8L70-R70) [[2]](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b074c8L100-R100) [[3]](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b074c8L129-R129) [[4]](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b074c8L157-R157) [[5]](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b074c8L190-R190) [[6]](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b074c8L221-R221) [[7]](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b074c8L252-R252) [[8]](diffhunk://#diff-6871e9be5fec9ba3cf082b7b4be9faaa14bafd213373175db1e0e54713b0 74c8L284-R284) * [`GroupedFirstAccumulator.java`](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46faL88-R88): Updated error messages to specify "First Aggregation" instead of generic "Unsupported data type". [[1]](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46faL88-R88) [[2]](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46faL120-R120) [[3]](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46faL153-R153) [[4]](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46faL185-R185) [[5]](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46faL239-R239) [[6]](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46faL286-R286) [[7]](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46faL322-R322) [[8]](diffhunk://#diff-6ffb16d462f44f266e1bba32520f30c5a32524fc8ea52b1bccd8794ccf6b46fa L374-R374) * [`GroupedFirstByAccumulator.java`](diffhunk://#diff-8d50630433ec9b73a81c321c00a4c983e566beba8599f27ecb73850feeba7d92L98-R98): Updated error messages to specify "FirstBy Aggregation" instead of generic "Unsupported data type". [[1]](diffhunk://#diff-8d50630433ec9b73a81c321c00a4c983e566beba8599f27ecb73850feeba7d92L98-R98) [[2]](diffhunk://#diff-8d50630433ec9b73a81c321c00a4c983e566beba8599f27ecb73850feeba7d92L130-R130) [[3]](diffhunk://#diff-8d50630433ec9b73a81c321c00a4c983e566beba8599f27ecb73850feeba7d92L166-R166) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
