[GitHub] [hbase] Apache9 commented on a diff in pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-13 Thread via GitHub
Apache9 commented on code in PR #5391: URL: https://github.com/apache/hbase/pull/5391#discussion_r1325295375 ## hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java: ## @@ -330,6 +322,73 @@ private boolean scheduleForRetry(IOException

[GitHub] [hbase] virajjasani commented on a diff in pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-13 Thread via GitHub
virajjasani commented on code in PR #5391: URL: https://github.com/apache/hbase/pull/5391#discussion_r1325387382 ## hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/NettyRpcConnection.java: ## @@ -347,7 +348,7 @@ public void operationComplete(ChannelFuture future) throws

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-14 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1718850992 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 30s |

[GitHub] [hbase] Apache-HBase commented on pull request #5401: HBASE-28061 HBaseTestingUtility failed to start MiniHbaseCluster in c…

2023-09-14 Thread via GitHub
Apache-HBase commented on PR #5401: URL: https://github.com/apache/hbase/pull/5401#issuecomment-1718864306 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 10s |

[GitHub] [hbase] Apache-HBase commented on pull request #5401: HBASE-28061 HBaseTestingUtility failed to start MiniHbaseCluster in c…

2023-09-14 Thread via GitHub
Apache-HBase commented on PR #5401: URL: https://github.com/apache/hbase/pull/5401#issuecomment-1718864359 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 24s |

[GitHub] [hbase] Apache-HBase commented on pull request #5401: HBASE-28061 HBaseTestingUtility failed to start MiniHbaseCluster in c…

2023-09-14 Thread via GitHub
Apache-HBase commented on PR #5401: URL: https://github.com/apache/hbase/pull/5401#issuecomment-1718895434 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] Apache9 commented on a diff in pull request #5401: HBASE-28061 HBaseTestingUtility failed to start MiniHbaseCluster in c…

2023-09-14 Thread via GitHub
Apache9 commented on code in PR #5401: URL: https://github.com/apache/hbase/pull/5401#discussion_r1325503532 ## hbase-server/src/main/java/org/apache/hadoop/hbase/fs/HFileSystem.java: ## @@ -17,6 +17,8 @@ */ package org.apache.hadoop.hbase.fs; +import static

[GitHub] [hbase] Apache9 opened a new pull request, #5401: HBASE-28061 HBaseTestingUtility failed to start MiniHbaseCluster in c…

2023-09-14 Thread via GitHub
Apache9 opened a new pull request, #5401: URL: https://github.com/apache/hbase/pull/5401 …ase of Hadoop3.3.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

[GitHub] [hbase] Apache-HBase commented on pull request #5401: HBASE-28061 HBaseTestingUtility failed to start MiniHbaseCluster in c…

2023-09-14 Thread via GitHub
Apache-HBase commented on PR #5401: URL: https://github.com/apache/hbase/pull/5401#issuecomment-1718898007 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 39s |

[GitHub] [hbase] virajjasani commented on a diff in pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-14 Thread via GitHub
virajjasani commented on code in PR #5391: URL: https://github.com/apache/hbase/pull/5391#discussion_r1325395461 ## hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/NettyRpcConnection.java: ## @@ -347,7 +348,7 @@ public void operationComplete(ChannelFuture future) throws

[GitHub] [hbase] zhangbutao commented on a diff in pull request #5401: HBASE-28061 HBaseTestingUtility failed to start MiniHbaseCluster in c…

2023-09-14 Thread via GitHub
zhangbutao commented on code in PR #5401: URL: https://github.com/apache/hbase/pull/5401#discussion_r1325439479 ## hbase-asyncfs/src/main/java/org/apache/hadoop/hbase/io/asyncfs/FanOutOneBlockAsyncDFSOutputHelper.java: ## @@ -383,7 +400,7 @@ private static List>

[GitHub] [hbase] Apache-HBase commented on pull request #5405: HBASE-28087 Add hadoop 3.3.6 in hadoopcheck

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5405: URL: https://github.com/apache/hbase/pull/5405#issuecomment-1721088253 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 40s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1721079638 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 29s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1721431777 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 36s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1721144416 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 34s |

[GitHub] [hbase] Apache-HBase commented on pull request #5403: HBASE-28068 : Add hbase.normalizer.merge.max.region.count property …

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5403: URL: https://github.com/apache/hbase/pull/5403#issuecomment-1721406772 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1721500608 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 35s |

[GitHub] [hbase] Apache-HBase commented on pull request #5335: HBASE-27981 Add connection and request attributes to slow log

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5335: URL: https://github.com/apache/hbase/pull/5335#issuecomment-1721523389 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 14s |

[GitHub] [hbase] Apache-HBase commented on pull request #5403: HBASE-28068 : Add hbase.normalizer.merge.max.region.count property …

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5403: URL: https://github.com/apache/hbase/pull/5403#issuecomment-1721497863 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 53s |

[GitHub] [hbase] Apache-HBase commented on pull request #5405: HBASE-28087 Add hadoop 3.3.6 in hadoopcheck

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5405: URL: https://github.com/apache/hbase/pull/5405#issuecomment-1721084825 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 27s |

[GitHub] [hbase] Apache-HBase commented on pull request #5405: HBASE-28087 Add hadoop 3.3.6 in hadoopcheck

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5405: URL: https://github.com/apache/hbase/pull/5405#issuecomment-1721085051 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 33s |

[GitHub] [hbase] Apache-HBase commented on pull request #5335: HBASE-27981 Add connection and request attributes to slow log

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5335: URL: https://github.com/apache/hbase/pull/5335#issuecomment-1721229759 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 15s |

[GitHub] [hbase] Apache9 merged pull request #5401: HBASE-28061 HBaseTestingUtility failed to start MiniHbaseCluster in c…

2023-09-15 Thread via GitHub
Apache9 merged PR #5401: URL: https://github.com/apache/hbase/pull/5401 -- 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:

[GitHub] [hbase] Apache-HBase commented on pull request #5403: HBASE-28068 : Add hbase.normalizer.merge.max.region.count property …

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5403: URL: https://github.com/apache/hbase/pull/5403#issuecomment-1721121073 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 13s |

[GitHub] [hbase] Apache-HBase commented on pull request #5335: HBASE-27981 Add connection and request attributes to slow log

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5335: URL: https://github.com/apache/hbase/pull/5335#issuecomment-1721587954 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 47s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-14 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1719163063 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 24s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-14 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1719245986 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 29s |

[GitHub] [hbase] Apache-HBase commented on pull request #5406: HBASE-27853 Add client side table metrics for rpc calls and request latency.

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5406: URL: https://github.com/apache/hbase/pull/5406#issuecomment-1722156874 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 44s |

[GitHub] [hbase] Apache-HBase commented on pull request #5406: HBASE-27853 Add client side table metrics for rpc calls and request latency.

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5406: URL: https://github.com/apache/hbase/pull/5406#issuecomment-1722157388 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 44s |

[GitHub] [hbase] zhuyaogai opened a new pull request, #5406: HBASE-27853 Add client side table metrics for rpc calls and request latency.

2023-09-15 Thread via GitHub
zhuyaogai opened a new pull request, #5406: URL: https://github.com/apache/hbase/pull/5406 Detail: [HBASE-27853](https://issues.apache.org/jira/browse/HBASE-27853) and backport for branch-2. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [hbase] Apache-HBase commented on pull request #5406: HBASE-27853 Add client side table metrics for rpc calls and request latency.

2023-09-15 Thread via GitHub
Apache-HBase commented on PR #5406: URL: https://github.com/apache/hbase/pull/5406#issuecomment-1722123183 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 40s |

[GitHub] [hbase] d-c-manning commented on a diff in pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-15 Thread via GitHub
d-c-manning commented on code in PR #5391: URL: https://github.com/apache/hbase/pull/5391#discussion_r1327886285 ## hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java: ## @@ -330,6 +330,56 @@ private boolean

[GitHub] [hbase] frostruan closed pull request #4367: HBASE-26974 Introduce a LogRollProcedure

2023-09-15 Thread via GitHub
frostruan closed pull request #4367: HBASE-26974 Introduce a LogRollProcedure URL: https://github.com/apache/hbase/pull/4367 -- 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

[GitHub] [hbase] NihalJain commented on a diff in pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-16 Thread via GitHub
NihalJain commented on code in PR #5407: URL: https://github.com/apache/hbase/pull/5407#discussion_r1328004621 ## pom.xml: ## @@ -856,7 +856,7 @@ 2.1.43 1.0.57 2.12.2 -1.70 +1.76 Review Comment: should we add an enforcer rule to ban

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1722364415 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 24s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1722367839 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 37s |

[GitHub] [hbase] NihalJain commented on pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-16 Thread via GitHub
NihalJain commented on PR #5407: URL: https://github.com/apache/hbase/pull/5407#issuecomment-1722302815 First ran `mvn dependency:tree > original-tree.txt` without fix. Output of `grep 'bouncycastle' original-tree.txt` is as follows: ``` $ grep 'bouncycastle' original-tree.txt

[GitHub] [hbase] NihalJain commented on a diff in pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-16 Thread via GitHub
NihalJain commented on code in PR #5407: URL: https://github.com/apache/hbase/pull/5407#discussion_r1328005368 ## pom.xml: ## @@ -856,7 +856,7 @@ 2.1.43 1.0.57 2.12.2 -1.70 +1.76 Review Comment: Fixed, will undo this changes if others feel this is not

[GitHub] [hbase] Apache-HBase commented on pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5407: URL: https://github.com/apache/hbase/pull/5407#issuecomment-1722309013 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] NihalJain commented on a diff in pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-16 Thread via GitHub
NihalJain commented on code in PR #5407: URL: https://github.com/apache/hbase/pull/5407#discussion_r1328005910 ## pom.xml: ## @@ -856,7 +856,7 @@ 2.1.43 1.0.57 2.12.2 -1.70 +1.76 Review Comment: Verified the ban plugin is working correctly by deleting

[GitHub] [hbase] virajjasani commented on a diff in pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-16 Thread via GitHub
virajjasani commented on code in PR #5391: URL: https://github.com/apache/hbase/pull/5391#discussion_r1328010092 ## hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java: ## @@ -330,6 +330,56 @@ private boolean

[GitHub] [hbase] Apache-HBase commented on pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5407: URL: https://github.com/apache/hbase/pull/5407#issuecomment-1722369664 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 30s |

[GitHub] [hbase] Apache-HBase commented on pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5407: URL: https://github.com/apache/hbase/pull/5407#issuecomment-1722373649 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for krb auth failures

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1722328824 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 32s |

[GitHub] [hbase] Apache-HBase commented on pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5407: URL: https://github.com/apache/hbase/pull/5407#issuecomment-1722353947 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 32s |

[GitHub] [hbase] NihalJain merged pull request #5387: HBASE-28066 Drop duplicate test class TestShellRSGroups.java

2023-09-16 Thread via GitHub
NihalJain merged PR #5387: URL: https://github.com/apache/hbase/pull/5387 -- 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:

[GitHub] [hbase] Apache-HBase commented on pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-17 Thread via GitHub
Apache-HBase commented on PR #5407: URL: https://github.com/apache/hbase/pull/5407#issuecomment-1722423436 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 24s |

[GitHub] [hbase] Apache-HBase commented on pull request #5407: HBASE-28089 Upgrade BouncyCastle to fix CVE-2023-33201

2023-09-17 Thread via GitHub
Apache-HBase commented on PR #5407: URL: https://github.com/apache/hbase/pull/5407#issuecomment-1722426399 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 33s |

[GitHub] [hbase] Apache-HBase commented on pull request #5408: HBASE-26974 Introduce a LogRollProcedure

2023-09-16 Thread via GitHub
Apache-HBase commented on PR #5408: URL: https://github.com/apache/hbase/pull/5408#issuecomment-1722389900 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 29s |

[GitHub] [hbase] Apache-HBase commented on pull request #5408: HBASE-26974 Introduce a LogRollProcedure

2023-09-17 Thread via GitHub
Apache-HBase commented on PR #5408: URL: https://github.com/apache/hbase/pull/5408#issuecomment-1722426718 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 1s |

[GitHub] [hbase] frostruan opened a new pull request, #5408: HBASE-26974 Introduce a LogRollProcedure

2023-09-16 Thread via GitHub
frostruan opened a new pull request, #5408: URL: https://github.com/apache/hbase/pull/5408 **This PR tries to reimplement the log-roll procedure with proc-v2.** **Modifies the following things** **client side:** instead of calling `admin.execProcedure()`, now we call

[GitHub] [hbase] Apache-HBase commented on pull request #5408: HBASE-26974 Introduce a LogRollProcedure

2023-09-17 Thread via GitHub
Apache-HBase commented on PR #5408: URL: https://github.com/apache/hbase/pull/5408#issuecomment-1722442599 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 43s |

[GitHub] [hbase] Apache-HBase commented on pull request #5408: HBASE-26974 Introduce a LogRollProcedure

2023-09-17 Thread via GitHub
Apache-HBase commented on PR #5408: URL: https://github.com/apache/hbase/pull/5408#issuecomment-1722529397 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 34s |

[GitHub] [hbase] Apache-HBase commented on pull request #5408: HBASE-26974 Introduce a LogRollProcedure

2023-09-17 Thread via GitHub
Apache-HBase commented on PR #5408: URL: https://github.com/apache/hbase/pull/5408#issuecomment-1722577245 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 42s |

[GitHub] [hbase] Apache-HBase commented on pull request #5408: HBASE-26974 Introduce a LogRollProcedure

2023-09-17 Thread via GitHub
Apache-HBase commented on PR #5408: URL: https://github.com/apache/hbase/pull/5408#issuecomment-1722575490 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 29s |

[GitHub] [hbase] Apache-HBase commented on pull request #5369: HBASE-28042 Snapshot corruptions due to non-atomic rename within same filesystem

2023-08-24 Thread via GitHub
Apache-HBase commented on PR #5369: URL: https://github.com/apache/hbase/pull/5369#issuecomment-1691436752 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 34s |

[GitHub] [hbase] Apache-HBase commented on pull request #5369: HBASE-28042 Snapshot corruptions due to non-atomic rename within same filesystem

2023-08-24 Thread via GitHub
Apache-HBase commented on PR #5369: URL: https://github.com/apache/hbase/pull/5369#issuecomment-1691396786 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] ragarkar commented on a diff in pull request #5342: HBASE-27998 Enhance region metrics to include prefetch ratio for each…

2023-08-24 Thread via GitHub
ragarkar commented on code in PR #5342: URL: https://github.com/apache/hbase/pull/5342#discussion_r1304112607 ## hbase-client/src/main/java/org/apache/hadoop/hbase/ServerMetrics.java: ## @@ -106,4 +106,6 @@ default String getVersion() { @Nullable List getTasks(); + /**

[GitHub] [hbase] anmolnar opened a new pull request, #5370: HBASE-28038 Add TLS settings to ZooKeeper client

2023-08-24 Thread via GitHub
anmolnar opened a new pull request, #5370: URL: https://github.com/apache/hbase/pull/5370 Looks like HBase has 2 different prefixes for ZooKeeper properties: `hbase.zookeeper.property.` `hbase.client.zookeeper.property.` and we also have these: `hbase.zookeeper.quorum`

[GitHub] [hbase] anmolnar commented on a diff in pull request #5363: HBASE-28008 Add support for netty tcnative

2023-08-24 Thread via GitHub
anmolnar commented on code in PR #5363: URL: https://github.com/apache/hbase/pull/5363#discussion_r1304464029 ## hbase-common/src/main/java/org/apache/hadoop/hbase/io/crypto/tls/X509Util.java: ## @@ -83,9 +88,10 @@ public final class X509Util { public static final String

[GitHub] [hbase] Apache-HBase commented on pull request #5370: HBASE-28038 Add TLS settings to ZooKeeper client

2023-08-24 Thread via GitHub
Apache-HBase commented on PR #5370: URL: https://github.com/apache/hbase/pull/5370#issuecomment-1691887726 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 27s |

[GitHub] [hbase] Apache-HBase commented on pull request #5370: HBASE-28038 Add TLS settings to ZooKeeper client

2023-08-24 Thread via GitHub
Apache-HBase commented on PR #5370: URL: https://github.com/apache/hbase/pull/5370#issuecomment-1691889829 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 2m 43s |

[GitHub] [hbase] Apache-HBase commented on pull request #5370: HBASE-28038 Add TLS settings to ZooKeeper client

2023-08-24 Thread via GitHub
Apache-HBase commented on PR #5370: URL: https://github.com/apache/hbase/pull/5370#issuecomment-1691896272 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] wchevreuil commented on a diff in pull request #5369: HBASE-28042 Snapshot corruptions due to non-atomic rename within same filesystem

2023-08-24 Thread via GitHub
wchevreuil commented on code in PR #5369: URL: https://github.com/apache/hbase/pull/5369#discussion_r1304301738 ## hbase-server/src/test/java/org/apache/hadoop/hbase/security/access/TestSnapshotScannerHDFSAclController.java: ## @@ -158,6 +158,7 @@ public void testGrantGlobal1()

[GitHub] [hbase] Apache-HBase commented on pull request #5342: HBASE-27998 Enhance region metrics to include prefetch ratio for each…

2023-08-24 Thread via GitHub
Apache-HBase commented on PR #5342: URL: https://github.com/apache/hbase/pull/5342#issuecomment-1691771732 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] Apache-HBase commented on pull request #5342: HBASE-27998 Enhance region metrics to include prefetch ratio for each…

2023-08-24 Thread via GitHub
Apache-HBase commented on PR #5342: URL: https://github.com/apache/hbase/pull/5342#issuecomment-1691780190 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 31s |

[GitHub] [hbase] Apache-HBase commented on pull request #5381: HBASE-28058 Adjust the order of acquiring the takeSnapshot locks to r…esolve the deadlock issue.

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5381: URL: https://github.com/apache/hbase/pull/5381#issuecomment-1701426023 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] Apache-HBase commented on pull request #5382: Use correct units in RegionLoad#getStoreUncompressedSizeMB()

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5382: URL: https://github.com/apache/hbase/pull/5382#issuecomment-1701955373 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 45s |

[GitHub] [hbase] Apache-HBase commented on pull request #5373: HBASE-28043 Reduce seeks from beginning of block in StoreFileScanner.seekToPreviousRow

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5373: URL: https://github.com/apache/hbase/pull/5373#issuecomment-1701830893 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 24s |

[GitHub] [hbase] Apache-HBase commented on pull request #5381: HBASE-28058 Adjust the order of acquiring the takeSnapshot locks to r…esolve the deadlock issue.

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5381: URL: https://github.com/apache/hbase/pull/5381#issuecomment-1701730953 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] charlesconnell opened a new pull request, #5382: Use correct units in RegionLoad#getStoreUncompressedSizeMB()

2023-08-31 Thread via GitHub
charlesconnell opened a new pull request, #5382: URL: https://github.com/apache/hbase/pull/5382 (no comment) -- 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

[GitHub] [hbase] Apache-HBase commented on pull request #5382: Use correct units in RegionLoad#getStoreUncompressedSizeMB()

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5382: URL: https://github.com/apache/hbase/pull/5382#issuecomment-1701957983 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 48s |

[GitHub] [hbase] Apache-HBase commented on pull request #5382: Use correct units in RegionLoad#getStoreUncompressedSizeMB()

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5382: URL: https://github.com/apache/hbase/pull/5382#issuecomment-1701954229 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 43s |

[GitHub] [hbase] Apache-HBase commented on pull request #5373: HBASE-28043 Reduce seeks from beginning of block in StoreFileScanner.seekToPreviousRow

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5373: URL: https://github.com/apache/hbase/pull/5373#issuecomment-1701985727 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 26s |

[GitHub] [hbase] frostruan commented on pull request #5381: HBASE-28058 Adjust the order of acquiring the takeSnapshot locks to r…esolve the deadlock issue.

2023-08-31 Thread via GitHub
frostruan commented on PR #5381: URL: https://github.com/apache/hbase/pull/5381#issuecomment-1702082826 this is indeed a problem. nice finding ! Thanks for the PR. +1 Also, do you think if we can just make `submitSnapshotProcedure` a `synchronized` method? -- This is an

[GitHub] [hbase] Apache-HBase commented on pull request #5373: HBASE-28043 Reduce seeks from beginning of block in StoreFileScanner.seekToPreviousRow

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5373: URL: https://github.com/apache/hbase/pull/5373#issuecomment-1702016533 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 37s |

[GitHub] [hbase] Apache-HBase commented on pull request #5381: HBASE-28058 Adjust the order of acquiring the takeSnapshot locks to r…esolve the deadlock issue.

2023-08-31 Thread via GitHub
Apache-HBase commented on PR #5381: URL: https://github.com/apache/hbase/pull/5381#issuecomment-1701736536 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 25s |

[GitHub] [hbase] Apache-HBase commented on pull request #5371: HBASE-28044 Reduce frequency of saving backing map in persistence cache

2023-08-30 Thread via GitHub
Apache-HBase commented on PR #5371: URL: https://github.com/apache/hbase/pull/5371#issuecomment-1699257891 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 27s |

[GitHub] [hbase] guluo2016 opened a new pull request, #5377: HBASE-28051 The annotation about RegionProcedureStore.delete is not right

2023-08-30 Thread via GitHub
guluo2016 opened a new pull request, #5377: URL: https://github.com/apache/hbase/pull/5377 Details see: HBASE-28051 -- 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

[GitHub] [hbase] Apache-HBase commented on pull request #5378: HBASE-28052 Removing the useless parameters from ProcedureExecutor.loadProcedures

2023-08-30 Thread via GitHub
Apache-HBase commented on PR #5378: URL: https://github.com/apache/hbase/pull/5378#issuecomment-1699399424 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 14s |

[GitHub] [hbase] Apache-HBase commented on pull request #5376: HBASE-27999 Implement cache prefetch aware load balancer

2023-08-30 Thread via GitHub
Apache-HBase commented on PR #5376: URL: https://github.com/apache/hbase/pull/5376#issuecomment-1699449773 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 13s |

[GitHub] [hbase] ragarkar commented on a diff in pull request #5376: HBASE-27999 Implement cache prefetch aware load balancer

2023-08-30 Thread via GitHub
ragarkar commented on code in PR #5376: URL: https://github.com/apache/hbase/pull/5376#discussion_r1310267557 ## hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/bucket/BucketCache.java: ## @@ -1477,6 +1477,7 @@ private void disableCache() { // If persistent

[GitHub] [hbase] wchevreuil commented on a diff in pull request #5370: HBASE-28038 Add TLS settings to ZooKeeper client

2023-08-30 Thread via GitHub
wchevreuil commented on code in PR #5370: URL: https://github.com/apache/hbase/pull/5370#discussion_r1310281661 ## hbase-common/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKConfig.java: ## @@ -330,4 +341,27 @@ public static String

[GitHub] [hbase] Apache-HBase commented on pull request #5378: HBASE-28052 Removing the useless parameters from ProcedureExecutor.loadProcedures

2023-08-30 Thread via GitHub
Apache-HBase commented on PR #5378: URL: https://github.com/apache/hbase/pull/5378#issuecomment-1699391089 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 31s |

[GitHub] [hbase] Apache-HBase commented on pull request #5378: HBASE-28052 Removing the useless parameters from ProcedureExecutor.loadProcedures

2023-08-30 Thread via GitHub
Apache-HBase commented on PR #5378: URL: https://github.com/apache/hbase/pull/5378#issuecomment-1699410500 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 37s |

[GitHub] [hbase] ankitsinghal commented on a diff in pull request #5371: HBASE-28044 Reduce frequency of saving backing map in persistence cache

2023-08-30 Thread via GitHub
ankitsinghal commented on code in PR #5371: URL: https://github.com/apache/hbase/pull/5371#discussion_r1310571851 ## hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/bucket/BucketCache.java: ## @@ -1700,17 +1853,17 @@ public BucketEntry writeToCache(final IOEngine

[GitHub] [hbase] Apache-HBase commented on pull request #5392: HBASE-28055 Addendum. Fixing the compilation failure

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5392: URL: https://github.com/apache/hbase/pull/5392#issuecomment-1715777394 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 40s |

[GitHub] [hbase] Apache-HBase commented on pull request #5394: HBASE-28055 Addendum. Fixing the compilation failure

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5394: URL: https://github.com/apache/hbase/pull/5394#issuecomment-1715884281 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 34s |

[GitHub] [hbase] Apache-HBase commented on pull request #5394: HBASE-28055 Addendum. Fixing the compilation failure

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5394: URL: https://github.com/apache/hbase/pull/5394#issuecomment-1715924864 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 5s |

[GitHub] [hbase] Apache-HBase commented on pull request #5384: HBASE-28065 Corrupt HFile data is mishandled in several cases (branch-2.5)

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5384: URL: https://github.com/apache/hbase/pull/5384#issuecomment-1715970314 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 39s |

[GitHub] [hbase] Apache-HBase commented on pull request #5391: HBASE-28050 RSProcedureDispatcher to fail-fast for SaslException

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5391: URL: https://github.com/apache/hbase/pull/5391#issuecomment-1716165368 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 36s |

[GitHub] [hbase] ndimiduk commented on a diff in pull request #5384: WIP HBASE-28065 Corrupt HFile data is mishandled in several cases

2023-09-12 Thread via GitHub
ndimiduk commented on code in PR #5384: URL: https://github.com/apache/hbase/pull/5384#discussion_r1323068390 ## hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/HFileBlock.java: ## @@ -1658,16 +1676,19 @@ private ByteBuff allocate(int size, boolean intoHeap) {

[GitHub] [hbase] bbeaudreault commented on a diff in pull request #5384: WIP HBASE-28065 Corrupt HFile data is mishandled in several cases

2023-09-12 Thread via GitHub
bbeaudreault commented on code in PR #5384: URL: https://github.com/apache/hbase/pull/5384#discussion_r1323167385 ## hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/HFileBlock.java: ## @@ -1700,6 +1721,21 @@ protected HFileBlock

[GitHub] [hbase] hiping-tech commented on a diff in pull request #5381: HBASE-28058 Adjust the order of acquiring the takeSnapshot locks to r…esolve the deadlock issue.

2023-09-12 Thread via GitHub
hiping-tech commented on code in PR #5381: URL: https://github.com/apache/hbase/pull/5381#discussion_r1323255457 ## hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/SnapshotManager.java: ## @@ -670,11 +670,13 @@ public void takeSnapshot(SnapshotDescription

[GitHub] [hbase] bbeaudreault commented on pull request #5384: HBASE-28065 Corrupt HFile data is mishandled in several cases (branch-2.5)

2023-09-12 Thread via GitHub
bbeaudreault commented on PR #5384: URL: https://github.com/apache/hbase/pull/5384#issuecomment-1716136374 Oddly, I checked out the code locally and ran the UTs and they all pass. I will try kicking off a new build once this one finishes. -- This is an automated message from the Apache

[GitHub] [hbase] Apache-HBase commented on pull request #5397: HBASE-28076 NPE on initialization error in RecoveredReplicationSource…

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5397: URL: https://github.com/apache/hbase/pull/5397#issuecomment-1716148748 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 13s |

[GitHub] [hbase] Apache-HBase commented on pull request #5384: HBASE-28065 Corrupt HFile data is mishandled in several cases (branch-2.5)

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5384: URL: https://github.com/apache/hbase/pull/5384#issuecomment-1715975167 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 36s |

[GitHub] [hbase] Apache-HBase commented on pull request #5228: HBASE-27853 Add client side table metrics for rpc calls and request latency.

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5228: URL: https://github.com/apache/hbase/pull/5228#issuecomment-1716187665 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 29s |

[GitHub] [hbase] Apache-HBase commented on pull request #5397: HBASE-28076 NPE on initialization error in RecoveredReplicationSource…

2023-09-12 Thread via GitHub
Apache-HBase commented on PR #5397: URL: https://github.com/apache/hbase/pull/5397#issuecomment-1716202251 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 43s |

[GitHub] [hbase] bbeaudreault commented on pull request #5228: HBASE-27853 Add client side table metrics for rpc calls and request latency.

2023-09-12 Thread via GitHub
bbeaudreault commented on PR #5228: URL: https://github.com/apache/hbase/pull/5228#issuecomment-1716206315 Test failure looks unrelated. LGTM -- 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

<    1   2   3   4   5   6   7   8   9   10   >