better default error message for WriteArrowTimestamps

kszucs · kszucs · commit 0bf997ecee09 · 2025-03-10T13:54:10.000+01:00
diff --git a/cpp/src/parquet/column_writer.cc b/cpp/src/parquet/column_writer.cc
@@ -1399,7 +1399,7 @@ class TypedColumnWriterImpl : public ColumnWriterImpl,
   Status WriteArrowTimestamps(const int16_t* def_levels, const int16_t* rep_levels,
                               int64_t num_levels, const ::arrow::Array& values,
                               ArrowWriteContext* ctx, bool maybe_parent_nulls) {
-    return Status::NotImplemented("Timestamps writing not implemented");
+    return Status::NotImplemented("Timestamps writing is only implemented for Int64Type");
   }
 
   void WriteDictionaryPage() override {

Original file line number	Diff line number	Diff line change
`@@ -1399,7 +1399,7 @@ class TypedColumnWriterImpl : public ColumnWriterImpl,`
`1399`	`1399`	`Status WriteArrowTimestamps(const int16_t* def_levels, const int16_t* rep_levels,`
`1400`	`1400`	`int64_t num_levels, const ::arrow::Array& values,`
`1401`	`1401`	`ArrowWriteContext* ctx, bool maybe_parent_nulls) {`
`1402`		`- return Status::NotImplemented("Timestamps writing not implemented");`
	`1402`	`+ return Status::NotImplemented("Timestamps writing is only implemented for Int64Type");`
`1403`	`1403`	`}`
`1404`	`1404`
`1405`	`1405`	`void WriteDictionaryPage() override {`