Reidd merged pull request #3237:
URL: https://github.com/apache/hbase/pull/3237
--
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.
For queries about this service, please
Apache-HBase commented on pull request #3202:
URL: https://github.com/apache/hbase/pull/3202#issuecomment-837872302
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 55
Apache-HBase commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837847210
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 4
Apache-HBase commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837831402
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 4m 0s
Apache-HBase commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837823204
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 29s
rahulLiving commented on pull request #3063:
URL: https://github.com/apache/hbase/pull/3063#issuecomment-837813571
Hi @ndimiduk I need to test out this change once, after addressing some
comments. I was out for sometime, hence couldn't do it and thus closed for time
being. I'll get it to c
Apache-HBase commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837776421
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 30s
Apache-HBase commented on pull request #3202:
URL: https://github.com/apache/hbase/pull/3202#issuecomment-837775180
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 6m 36s
Apache-HBase commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837773079
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 28s
Apache-HBase commented on pull request #3202:
URL: https://github.com/apache/hbase/pull/3202#issuecomment-837766619
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 8s
Apache-HBase commented on pull request #3202:
URL: https://github.com/apache/hbase/pull/3202#issuecomment-837765986
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 30s
[
https://issues.apache.org/jira/browse/HBASE-25858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342273#comment-17342273
]
Reid Chan commented on HBASE-25858:
---
Try merging, FYI [~apurtell]
> [branch-1] make h
apurtell commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837688241
Rolled up the first round of review feedback into d4d707d
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629810127
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -241,10 +246,27 @@ public void write(Cell cell)
Apache-HBase commented on pull request #3246:
URL: https://github.com/apache/hbase/pull/3246#issuecomment-837684227
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 34
Apache-HBase commented on pull request #3246:
URL: https://github.com/apache/hbase/pull/3246#issuecomment-837677173
--
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.
For quer
apurtell edited a comment on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837666043
I have some changes coming soon. Accepted @bharathv 's argument that
unconditionally compressing values if value compression is enabled is fine even
if some value cases ma
apurtell commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837666043
I have some changes coming soon. Accepted @bharathv 's argument that
unconditionally compressing values if value compression is enabled is fine even
if some value cases may not c
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629802049
##
File path: hbase-common/src/main/java/org/apache/hadoop/hbase/KeyValue.java
##
@@ -230,6 +230,11 @@ public static long getKeyDataStructureSize(int rleng
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629801863
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -241,10 +246,27 @@ public void write(Cell cell)
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629801332
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -302,14 +343,28 @@ protected Cell parseCell() th
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629801332
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -302,14 +343,28 @@ protected Cell parseCell() th
Apache-HBase commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837655575
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 2
Apache-HBase commented on pull request #3246:
URL: https://github.com/apache/hbase/pull/3246#issuecomment-837638994
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 26
Apache-HBase commented on pull request #3246:
URL: https://github.com/apache/hbase/pull/3246#issuecomment-837636720
--
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.
For quer
rda3mon commented on a change in pull request #3246:
URL: https://github.com/apache/hbase/pull/3246#discussion_r629784728
##
File path:
hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupManager.java
##
@@ -295,6 +295,11 @@ public void setBackupInfo(BackupInf
bharathv commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629779970
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629779580
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629779580
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629779580
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
Apache9 commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837580362
> > So we will only compress value?
>
> This is an enhancement to existing WAL compression. As you know the
existing WAL compression already compresses other aspects of WAL
Apache-HBase commented on pull request #3251:
URL: https://github.com/apache/hbase/pull/3251#issuecomment-837580124
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 8s
Apache9 commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629777261
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limite
bharathv commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629772178
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -302,14 +343,28 @@ protected Cell parseCell() th
Apache-HBase commented on pull request #3251:
URL: https://github.com/apache/hbase/pull/3251#issuecomment-837558899
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 2m 39
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629763436
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
[
https://issues.apache.org/jira/browse/HBASE-25032?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342205#comment-17342205
]
Bharath Vissapragada commented on HBASE-25032:
--
Oops, this patch was commit
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629763436
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629763436
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629763436
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629763212
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -302,14 +343,28 @@ protected Cell parseCell() th
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629759791
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -256,6 +278,26 @@ public void write(Cell cell) t
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629761476
##
File path: hbase-protocol-shaded/src/main/protobuf/server/region/WAL.proto
##
@@ -32,6 +32,7 @@ message WALHeader {
optional bool has_tag_compression
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629760897
##
File path: hbase-common/src/main/java/org/apache/hadoop/hbase/KeyValue.java
##
@@ -230,6 +230,11 @@ public static long getKeyDataStructureSize(int rleng
bharathv commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629760196
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java
##
@@ -34,21 +36,49 @@
@InterfaceAudience.Limit
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629759981
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -256,6 +278,26 @@ public void write(Cell cell) t
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629759791
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -256,6 +278,26 @@ public void write(Cell cell) t
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629757734
##
File path: hbase-protocol-shaded/src/main/protobuf/server/region/WAL.proto
##
@@ -32,6 +32,7 @@ message WALHeader {
optional bool has_tag_compression
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629757734
##
File path: hbase-protocol-shaded/src/main/protobuf/server/region/WAL.proto
##
@@ -32,6 +32,7 @@ message WALHeader {
optional bool has_tag_compression
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629757734
##
File path: hbase-protocol-shaded/src/main/protobuf/server/region/WAL.proto
##
@@ -32,6 +32,7 @@ message WALHeader {
optional bool has_tag_compression
apurtell commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629757222
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -241,10 +246,27 @@ public void write(Cell cell)
apurtell edited a comment on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837518016
> Is it reasonable to always use the same compressor all the time for WALs?
It strikes me that operators make an explicit choice about the compression used
in their tables
apurtell commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837518016
> Is it reasonable to always use the same compressor all the time for WALs?
It strikes me that operators make an explicit choice about the compression used
in their tables, so ma
Apache-HBase commented on pull request #3242:
URL: https://github.com/apache/hbase/pull/3242#issuecomment-837517120
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 45
[
https://issues.apache.org/jira/browse/HBASE-25855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342201#comment-17342201
]
Sean Busbey commented on HBASE-25855:
-
ah I see. 3.5.0 was the htrace replacement mo
apurtell edited a comment on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837512098
> There is a potential performance improvement here. We could create a
Deflater/Inflater pair per column family.
This idea didn't pan out. It makes compression in m
apurtell commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837512098
bq. There is a potential performance improvement here. We could create a
Deflater/Inflater pair per column family.
This idea didn't pan out. It makes compression in my tes
[
https://issues.apache.org/jira/browse/HBASE-25855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sean Busbey reassigned HBASE-25855:
---
Assignee: Nick Dimiduk (was: Sean Busbey)
> [hbase-thirdparty] Fix typo in jersey relocati
[
https://issues.apache.org/jira/browse/HBASE-25855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sean Busbey reassigned HBASE-25855:
---
Assignee: Sean Busbey (was: Nick Dimiduk)
> [hbase-thirdparty] Fix typo in jersey relocati
busbey commented on a change in pull request #51:
URL: https://github.com/apache/hbase-thirdparty/pull/51#discussion_r629751088
##
File path: hbase-shaded-jackson-jaxrs-json-provider/pom.xml
##
@@ -0,0 +1,110 @@
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://www.w3.o
ndimiduk commented on a change in pull request #51:
URL: https://github.com/apache/hbase-thirdparty/pull/51#discussion_r629746479
##
File path: hbase-shaded-jackson-jaxrs-json-provider/pom.xml
##
@@ -0,0 +1,110 @@
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://www.w3
ndimiduk commented on pull request #3063:
URL: https://github.com/apache/hbase/pull/3063#issuecomment-837478253
Hi @rahulLiving , I noticed you closed this PR. Are you planning to take
this one in a different direction?
--
This is an automated message from the Apache Git Service.
To resp
Apache-HBase commented on pull request #3250:
URL: https://github.com/apache/hbase/pull/3250#issuecomment-837463452
--
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.
For quer
saintstack commented on a change in pull request #3249:
URL: https://github.com/apache/hbase/pull/3249#discussion_r629734904
##
File path: hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java
##
@@ -1601,6 +1601,89 @@
*/
public static final int BATCH_ROWS_T
Apache-HBase commented on pull request #3242:
URL: https://github.com/apache/hbase/pull/3242#issuecomment-837454813
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 35
saintstack commented on a change in pull request #3246:
URL: https://github.com/apache/hbase/pull/3246#discussion_r629730178
##
File path:
hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupManager.java
##
@@ -295,6 +295,11 @@ public void setBackupInfo(Backup
saintstack commented on a change in pull request #3248:
URL: https://github.com/apache/hbase/pull/3248#discussion_r629719036
##
File path:
hbase-balancer/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java
##
@@ -59,10 +59,11 @@
*/
@Deprecated
String HBASE
ndimiduk commented on pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#issuecomment-837437980
Oh, one other comment I forgot to mention before hitting submit. Is it
reasonable to always use the same compressor all the time for WALs? It strikes
me that operators make an ex
ndimiduk commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629712475
##
File path: hbase-common/src/main/java/org/apache/hadoop/hbase/KeyValue.java
##
@@ -230,6 +230,11 @@ public static long getKeyDataStructureSize(int rleng
shahrs87 commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629711573
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -256,6 +278,26 @@ public void write(Cell cell) t
[
https://issues.apache.org/jira/browse/HBASE-25032?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342162#comment-17342162
]
Huaxiang Sun commented on HBASE-25032:
--
Yeah, no new release is needed for 2.3.*.
shahrs87 commented on a change in pull request #3244:
URL: https://github.com/apache/hbase/pull/3244#discussion_r629708006
##
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALCellCodec.java
##
@@ -256,6 +278,26 @@ public void write(Cell cell) t
Apache-HBase commented on pull request #51:
URL: https://github.com/apache/hbase-thirdparty/pull/51#issuecomment-837377636
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec |
[
https://issues.apache.org/jira/browse/HBASE-25855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Nick Dimiduk updated HBASE-25855:
-
Resolution: Fixed
Status: Resolved (was: Patch Available)
> [hbase-thirdparty] Fix typo
Apache-HBase commented on pull request #3251:
URL: https://github.com/apache/hbase/pull/3251#issuecomment-837357330
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 36
[
https://issues.apache.org/jira/browse/HBASE-25855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Nick Dimiduk updated HBASE-25855:
-
Summary: [hbase-thirdparty] Fix typo in jersey relocation path (was: Fix
typo in jersey relocat
[
https://issues.apache.org/jira/browse/HBASE-25855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Nick Dimiduk updated HBASE-25855:
-
Fix Version/s: (was: thirdparty-3.5.1)
thirdparty-4.0.0
> Fix typo in jer
[
https://issues.apache.org/jira/browse/HBASE-25855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342148#comment-17342148
]
Nick Dimiduk commented on HBASE-25855:
--
It appears to me that hbase-core has not mo
Apache-HBase commented on pull request #3250:
URL: https://github.com/apache/hbase/pull/3250#issuecomment-837331291
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 31s
Apache-HBase commented on pull request #3242:
URL: https://github.com/apache/hbase/pull/3242#issuecomment-837320433
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 0m 31
[
https://issues.apache.org/jira/browse/HBASE-25032?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342130#comment-17342130
]
Andrew Kyle Purtell commented on HBASE-25032:
-
Good, so no release is affect
[
https://issues.apache.org/jira/browse/HBASE-25774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342127#comment-17342127
]
Michael Stack commented on HBASE-25774:
---
Thanks for figuring the race [~zhangduo]
[
https://issues.apache.org/jira/browse/HBASE-25876?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Stack updated HBASE-25876:
--
Priority: Trivial (was: Major)
> Add retry if we fail to read all bytes of the protobuf magic
[
https://issues.apache.org/jira/browse/HBASE-25876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342125#comment-17342125
]
Michael Stack commented on HBASE-25876:
---
Just two places (thought there were m
saintstack opened a new pull request #3251:
URL: https://github.com/apache/hbase/pull/3251
…gic marker
--
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.
For queries about th
[
https://issues.apache.org/jira/browse/HBASE-25875?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Pankaj Kumar updated HBASE-25875:
-
Fix Version/s: 3.0.0-alpha-1
Status: Patch Available (was: In Progress)
> RegionServ
ndimiduk commented on a change in pull request #3243:
URL: https://github.com/apache/hbase/pull/3243#discussion_r629644198
##
File path: hbase-rest/pom.xml
##
@@ -160,7 +151,6 @@
org.apache.hbase
hbase-protocol-shaded
- jar
Review comment:
Yes,
ndimiduk commented on a change in pull request #3243:
URL: https://github.com/apache/hbase/pull/3243#discussion_r629645829
##
File path:
hbase-rest/src/test/java/org/apache/hadoop/hbase/rest/RowResourceBase.java
##
@@ -18,22 +18,16 @@
package org.apache.hadoop.hbase.rest;
busbey commented on a change in pull request #51:
URL: https://github.com/apache/hbase-thirdparty/pull/51#discussion_r629645813
##
File path: hbase-shaded-jackson-jaxrs-json-provider/pom.xml
##
@@ -0,0 +1,110 @@
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://www.w3.o
ndimiduk commented on a change in pull request #3243:
URL: https://github.com/apache/hbase/pull/3243#discussion_r629645403
##
File path:
hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/model/ScannerModel.java
##
@@ -18,22 +17,19 @@
*/
package org.apache.hadoop.hbase.r
Apache-HBase commented on pull request #3242:
URL: https://github.com/apache/hbase/pull/3242#issuecomment-837244127
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|::|--:|:|:|
| +0 :ok: | reexec | 1m 47
pankaj72981 opened a new pull request #3250:
URL: https://github.com/apache/hbase/pull/3250
--
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.
For queries about this service
[
https://issues.apache.org/jira/browse/HBASE-25875?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Work on HBASE-25875 started by Pankaj Kumar.
> RegionServer failed to start due to IllegalThreadStateException in
> Authent
busbey commented on pull request #51:
URL: https://github.com/apache/hbase-thirdparty/pull/51#issuecomment-837242913
> > Since the relocated jaxrs provider is intimately tied to our use of
jersey, can we fold this into our packaged relocation of jersey in the
`hbase-shaded-jersey` module?
ndimiduk commented on pull request #51:
URL: https://github.com/apache/hbase-thirdparty/pull/51#issuecomment-837239785
> Since the relocated jaxrs provider is intimately tied to our use of
jersey, can we fold this into our packaged relocation of jersey in the
`hbase-shaded-jersey` module?
Michael Stack created HBASE-25876:
-
Summary: Add retry if we fail to read all bytes of the protobuf
magic marker
Key: HBASE-25876
URL: https://issues.apache.org/jira/browse/HBASE-25876
Project: HBase
ndimiduk commented on a change in pull request #51:
URL: https://github.com/apache/hbase-thirdparty/pull/51#discussion_r629642048
##
File path: hbase-shaded-jackson-jaxrs-json-provider/pom.xml
##
@@ -0,0 +1,110 @@
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://www.w3
[
https://issues.apache.org/jira/browse/HBASE-25875?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342113#comment-17342113
]
Pankaj Kumar commented on HBASE-25875:
--
NettyRpcServer should start the secret mana
[
https://issues.apache.org/jira/browse/HBASE-25859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Stack resolved HBASE-25859.
---
Fix Version/s: 2.4.4
2.5.0
3.0.0-alpha-1
Hadoop Fl
[
https://issues.apache.org/jira/browse/HBASE-25875?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17342111#comment-17342111
]
Pankaj Kumar commented on HBASE-25875:
--
This is a race condition problem, where Lea
1 - 100 of 168 matches
Mail list logo