wgtmac commented on PR #37313: URL: https://github.com/apache/arrow/pull/37313#issuecomment-1695302650
I don't think rerun will resolve the issue. Please check the report carefully. ``` FAILED: CMakeFiles/lint cd /tmp/arrow-lint-xdzm4fix/cpp-build && /usr/local/bin/python /arrow/cpp/build-support/run_cpplint.py --cpplint_binary /arrow/cpp/build-support/cpplint.py --exclude_globs /arrow/cpp/build-support/lint_exclusions.txt --source_dir /arrow/cpp/src --source_dir /arrow/cpp/examples --source_dir /arrow/cpp/tools --quiet /arrow/cpp/src/arrow/util/byte_stream_split.h:626: Namespace should be terminated with "// namespace arrow::util::internal" [readability/namespace] [5] /arrow/cpp/src/arrow/util/compression_brotli.cc:257: Namespace should be terminated with "// namespace arrow::util::internal" [readability/namespace] [5] /arrow/cpp/src/arrow/util/compression_lz4.cc:540: Namespace should be terminated with "// namespace arrow::util::internal" [readability/namespace] [5] Done processing /arrow/cpp/src/arrow/util/byte_stream_split.h Done processing /arrow/cpp/src/arrow/util/compression_brotli.cc Done processing /arrow/cpp/src/arrow/util/compression_lz4.cc Total errors found: 3 /arrow/cpp/src/arrow/util/compression_bz2.cc:283: Namespace should be terminated with "// namespace arrow::util::internal" [readability/namespace] [5] Done processing /arrow/cpp/src/arrow/util/compression_bz2.cc Total errors found: 1 /arrow/cpp/src/arrow/util/compression_snappy.cc:95: Namespace should be terminated with "// namespace arrow::util::internal" [readability/namespace] [5] /arrow/cpp/src/arrow/util/compression_snappy.cc:95: Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4] /arrow/cpp/src/arrow/util/compression_zlib.cc:520: Namespace should be terminated with "// namespace arrow::util::internal" [readability/namespace] [5] /arrow/cpp/src/arrow/util/utf8.cc:87: Namespace should be terminated with "// namespace internal" [readability/namespace] [5] /arrow/cpp/src/arrow/util/utf8.cc:195: Namespace should be terminated with "// namespace arrow::util" [readability/namespace] [5] Done processing /arrow/cpp/src/arrow/util/compression_snappy.cc Done processing /arrow/cpp/src/arrow/util/compression_zlib.cc Done processing /arrow/cpp/src/arrow/util/utf8.cc Total errors found: 5 /arrow/cpp/src/arrow/util/compression_zstd.cc:245: Namespace should be terminated with "// namespace arrow::util::internal" [readability/namespace] [5] Done processing /arrow/cpp/src/arrow/util/compression_zstd.cc Total errors found: 1 ``` -- 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]
