fresh-borzoni commented on code in PR #313:
URL: https://github.com/apache/fluss-rust/pull/313#discussion_r2806260625
##########
bindings/cpp/include/fluss.hpp:
##########
@@ -306,19 +306,19 @@ enum class DatumType {
constexpr int64_t EARLIEST_OFFSET = -2;
constexpr int64_t LATEST_OFFSET = -1;
-enum class OffsetSpec {
+enum class OffsetType {
Earliest = 0,
Latest = 1,
Timestamp = 2,
};
-struct OffsetQuery {
- OffsetSpec spec;
+struct OffsetSpec {
+ OffsetType type;
int64_t timestamp{0};
- static OffsetQuery Earliest() { return {OffsetSpec::Earliest, 0}; }
- static OffsetQuery Latest() { return {OffsetSpec::Latest, 0}; }
- static OffsetQuery FromTimestamp(int64_t ts) { return
{OffsetSpec::Timestamp, ts}; }
+ static OffsetSpec Earliest() { return {OffsetType::Earliest, 0}; }
+ static OffsetSpec Latest() { return {OffsetType::Latest, 0}; }
+ static OffsetSpec Timestamp(int64_t ts) { return {OffsetType::Timestamp,
ts}; }
Review Comment:
Well, it's a tricky one: in order to have full cross compatibility with java
configs - we need to have client prefix as well. But i guess it's not
complicated, it's just adding more options in dict config layer. Let me add an
example, it's non-blocker for python, but C++/Rust - it's better to leave as it
is, unless we have rsFlink xD
--
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]
