[kudu-CR] KUDU-3543 Fix Content-Type headers

2024-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20868 ) Change subject: KUDU-3543 Fix Content-Type headers .. Patch Set 3: (12 comments)

[kudu-CR] Use GTEST SKIP() to skip tests

2024-01-09 Thread Yingchun Lai (Code Review)
Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20875 ) Change subject: Use GTEST_SKIP() to skip tests .. Use GTEST_SKIP() to skip tests GTEST_SKIP() is better than manually logging

[kudu-CR] Use GTEST SKIP() to skip tests

2024-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20875 ) Change subject: Use GTEST_SKIP() to skip tests .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/20875

[kudu-CR] Use GTEST SKIP() to skip tests

2024-01-09 Thread Yingchun Lai (Code Review)
Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/20875 ) Change subject: Use GTEST_SKIP() to skip tests .. Patch Set 3: Verified+1 Remove the un-related failed test:

[kudu-CR] Use GTEST SKIP() to skip tests

2024-01-09 Thread Yingchun Lai (Code Review)
Yingchun Lai has removed a vote on this change. Change subject: Use GTEST_SKIP() to skip tests .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/20875 To unsubscribe, visit

[kudu-CR] Use GTEST SKIP() to skip tests

2024-01-09 Thread Yingchun Lai (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20875 to look at the new patch set (#3). Change subject: Use GTEST_SKIP() to skip tests .. Use

[kudu-CR] Use GTEST SKIP() to skip tests

2024-01-09 Thread Yingchun Lai (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20875 to look at the new patch set (#2). Change subject: Use GTEST_SKIP() to skip tests .. Use

[kudu-CR] Use GTEST SKIP() to skip tests

2024-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20875 ) Change subject: Use GTEST_SKIP() to skip tests .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/20875

[kudu-CR] KUDU-3543 Fix Content-Type headers

2024-01-09 Thread Marton Greber (Code Review)
Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/20868 ) Change subject: KUDU-3543 Fix Content-Type headers .. Patch Set 3: (3 comments)

[kudu-CR] KUDU-3543 Fix Content-Type headers

2024-01-09 Thread Marton Greber (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20868 to look at the new patch set (#3). Change subject: KUDU-3543 Fix Content-Type headers

[kudu-CR] Use GTEST SKIP() to skip tests

2024-01-09 Thread Yingchun Lai (Code Review)
Yingchun Lai has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20875 Change subject: Use GTEST_SKIP() to skip tests .. Use GTEST_SKIP() to skip tests GTEST_SKIP() is better than manually logging and

[kudu-CR] Add CLion build directories to .gitignore

2024-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20873 ) Change subject: Add CLion build directories to .gitignore .. Add CLion build directories to .gitignore Change-Id:

[kudu-CR] Add CLion build directories to .gitignore

2024-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20873 ) Change subject: Add CLion build directories to .gitignore .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] Add CLion build directories to .gitignore

2024-01-09 Thread Yingchun Lai (Code Review)
Yingchun Lai has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20873 Change subject: Add CLion build directories to .gitignore .. Add CLion build directories to .gitignore Change-Id:

[kudu-CR] [compaction/flush] Cleanup of compaction and flush code paths

2024-01-09 Thread Ashwani Raina (Code Review)
Hello Mahesh Reddy, Alexey Serbin, Yingchun Lai, Attila Bukor, Yifan Zhang, Kudu Jenkins, Abhishek Chennaka, Ádám Bakai, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20720 to look at the new patch set (#8). Change subject: [compaction/flush] Cleanup of

[kudu-CR] [compaction/flush] Cleanup of compaction and flush code paths

2024-01-09 Thread Ashwani Raina (Code Review)
Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/20720 ) Change subject: [compaction/flush] Cleanup of compaction and flush code paths .. Patch Set 7: (3 comments) > (1 comment) > > > >

[kudu-CR] KUDU-3526 [java] Scanner should bind with a tserver in java client.

2024-01-09 Thread Song Jiacheng (Code Review)
Hello Marton Greber, Alexey Serbin, Yifan Zhang, Attila Bukor, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20715 to look at the new patch set (#27). Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client.

[kudu-CR] KUDU-3371 [fs] Use RocksDB to store LBM metadata

2024-01-09 Thread Yingchun Lai (Code Review)
Hello Alexey Serbin, Yuqi Du, Yifan Zhang, Kudu Jenkins, Abhishek Chennaka, KeDeng, Wang Xixu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18569 to look at the new patch set (#70). Change subject: KUDU-3371 [fs] Use RocksDB to store LBM metadata