Skip to content

Commit 99446ca

Browse files
authored
[Fix](compile) Fix BE compile failure (apache#60113)
introduced by apache#60042
1 parent 075c7da commit 99446ca

2 files changed

Lines changed: 3 additions & 5 deletions

File tree

be/src/vec/data_types/serde/data_type_timestamptz_serde.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -223,9 +223,7 @@ Status DataTypeTimeStampTzSerDe::write_column_to_orc(const std::string& timezone
223223
}
224224

225225
int64_t timestamp = 0;
226-
if (!col_data[row_id].unix_timestamp(&timestamp, UTC)) {
227-
return Status::InternalError("get unix timestamp error.");
228-
}
226+
col_data[row_id].unix_timestamp(&timestamp, UTC);
229227

230228
cur_batch->data[row_id] = timestamp;
231229
cur_batch->nanoseconds[row_id] = col_data[row_id].microsecond() * micro_to_nano_second;

be/src/vec/runtime/timestamptz_value.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -139,8 +139,8 @@ class TimestampTzValue {
139139

140140
void set_microsecond(uint64_t microsecond) { _utc_dt.set_microsecond(microsecond); }
141141

142-
bool unix_timestamp(int64_t* timestamp, const cctz::time_zone& ctz) const {
143-
return _utc_dt.unix_timestamp(timestamp, ctz);
142+
void unix_timestamp(int64_t* timestamp, const cctz::time_zone& ctz) const {
143+
_utc_dt.unix_timestamp(timestamp, ctz);
144144
}
145145

146146
// Convert UTC time to local time based on the given timezone

0 commit comments

Comments
 (0)