Xiaoyu Yao created HDDS-3515:
Summary: Ensure consistent OM token service field in HA environment
Key: HDDS-3515
URL: https://issues.apache.org/jira/browse/HDDS-3515
Project: Hadoop Distributed Data Store
bharatviswa504 edited a comment on pull request #857:
URL: https://github.com/apache/hadoop-ozone/pull/857#issuecomment-621985964
Sorry for the late review. Since it is marked as draft until 2 days back,
not looked into it.
Few comments during initial review of the PR.
1.
[
https://issues.apache.org/jira/browse/HDDS-3504?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17096722#comment-17096722
]
Stephen O'Donnell commented on HDDS-3504:
-
Looking at a few of the failures, it seems its the
maobaolong commented on pull request #875:
URL: https://github.com/apache/hadoop-ozone/pull/875#issuecomment-621970960
@adoroszlai Thank you for your review and helping me to correct more fault.
I have accept all of your suggestion and commit into this PR, PTAL.
[
https://issues.apache.org/jira/browse/HDDS-3504?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17096732#comment-17096732
]
Stephen O'Donnell commented on HDDS-3504:
-
I wonder if these space problems are appearing in this
bharatviswa504 commented on pull request #857:
URL: https://github.com/apache/hadoop-ozone/pull/857#issuecomment-621985964
Sorry for the late review. Since it is marked for draft until 2 days back,
not looked into it.
1. Previously listBuckets owned by an authenticated user. Now we
bharatviswa504 edited a comment on pull request #857:
URL: https://github.com/apache/hadoop-ozone/pull/857#issuecomment-621985964
Sorry for the late review. Since it is marked for draft until 2 days back,
not looked into it.
Few comments during initial review of the PR.
1.
hanishakoneru commented on pull request #770:
URL: https://github.com/apache/hadoop-ozone/pull/770#issuecomment-621996062
LGTM. +1 pending CI.
This is an automated message from the Apache Git Service.
To respond to the
[
https://issues.apache.org/jira/browse/HDDS-3515?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HDDS-3515:
-
Labels: pull-request-available (was: )
> Ensure consistent OM token service field in HA
xiaoyuyao opened a new pull request #889:
URL: https://github.com/apache/hadoop-ozone/pull/889
## What changes were proposed in this pull request?
Consistent DT service name for HA enabled OM setup.
## What is the link to the Apache JIRA
[
https://issues.apache.org/jira/browse/HDDS-3504?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Stephen O'Donnell updated HDDS-3504:
Attachment:
docker-ozone-topology-ozone-topology-basic-scm-2020-04-27-830.log.gz
>
[
https://issues.apache.org/jira/browse/HDDS-3504?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Stephen O'Donnell updated HDDS-3504:
Attachment:
docker-ozone-topology-ozone-topology-basic-scm-2020-04-20-792.log.gz
>
[
https://issues.apache.org/jira/browse/HDDS-3504?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Stephen O'Donnell reassigned HDDS-3504:
---
Assignee: Stephen O'Donnell
> Topology related acceptance test is intermittent
>
[
https://issues.apache.org/jira/browse/HDDS-3411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Vivek Ratnavel Subramanian resolved HDDS-3411.
--
Resolution: Fixed
> Switch Recon SQL DB to Derby.
>
fapifta edited a comment on pull request #881:
URL: https://github.com/apache/hadoop-ozone/pull/881#issuecomment-622181460
Hi @sodonnel, thank you for this work. As I see the code itself seems to be
fairly correct as far as I can tell based on just reading through the changes I
haven't
cxorm commented on pull request #770:
URL: https://github.com/apache/hadoop-ozone/pull/770#issuecomment-622094368
Rebase latest master-branch(#839) and trigger github-actions.
This is an automated message from the Apache Git
hanishakoneru commented on a change in pull request #856:
URL: https://github.com/apache/hadoop-ozone/pull/856#discussion_r418304296
##
File path:
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/OzoneManagerDoubleBuffer.java
##
@@ -173,17 +196,19 @@
[
https://issues.apache.org/jira/browse/HDDS-3475?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HDDS-3475:
-
Labels: pull-request-available (was: )
> Use transactionInfo table to persist transaction
[
https://issues.apache.org/jira/browse/HDDS-3515?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bharat Viswanadham resolved HDDS-3515.
--
Fix Version/s: 0.6.0
Resolution: Fixed
> Ensure consistent OM token service
bharatviswa504 commented on pull request #889:
URL: https://github.com/apache/hadoop-ozone/pull/889#issuecomment-622130171
Thank You @xiaoyuyao for the contribution and @vivekratnavel for the review.
This is an automated
fapifta commented on pull request #881:
URL: https://github.com/apache/hadoop-ozone/pull/881#issuecomment-622181460
Hi @sodonnel, thank you for this work. As I see the code itself seems to be
fairly correct as far as I can tell based on just reading through the changes I
haven't seen a
xiaoyuyao commented on pull request #724:
URL: https://github.com/apache/hadoop-ozone/pull/724#issuecomment-622100398
TestBlockOutputStreamWithFailures seems to take long time to finish (10m)
with pending flaky issues as query below. Should we consider disable it?
[
https://issues.apache.org/jira/browse/HDDS-3082?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Stephen O'Donnell updated HDDS-3082:
Summary: Create Recon task to detect mis-replicated containers (was: Tool
to check
[
https://issues.apache.org/jira/browse/HDDS-3082?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Stephen O'Donnell reassigned HDDS-3082:
---
Assignee: Stephen O'Donnell
> Create Recon task to detect mis-replicated containers
vivekratnavel commented on pull request #839:
URL: https://github.com/apache/hadoop-ozone/pull/839#issuecomment-622073867
@avijayanhwx Thanks for working on this!
@swagle @elek Thanks for the reviews!
I have merged the change.
avijayanhwx commented on a change in pull request #881:
URL: https://github.com/apache/hadoop-ozone/pull/881#discussion_r418295719
##
File path:
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/SCMCommonPlacementPolicy.java
##
@@ -198,4 +200,65 @@ public
fapifta edited a comment on pull request #881:
URL: https://github.com/apache/hadoop-ozone/pull/881#issuecomment-622181460
Hi @sodonnel, thank you for this work. As I see the code itself seems to be
fairly correct as far as I can tell based on just reading through the changes I
haven't
fapifta edited a comment on pull request #881:
URL: https://github.com/apache/hadoop-ozone/pull/881#issuecomment-622181460
Hi @sodonnel, thank you for this work. As I see the code itself seems to be
fairly correct as far as I can tell based on just reading through the changes I
haven't
[
https://issues.apache.org/jira/browse/HDDS-3389?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Siyao Meng updated HDDS-3389:
-
Status: Patch Available (was: Open)
> Add response to SetVolumePropertyResponse proto
>
vivekratnavel edited a comment on pull request #706:
URL: https://github.com/apache/hadoop-ozone/pull/706#issuecomment-614417781
@nilotpalnandi Thanks for working on this! Can you please resolve checkstyle
issues and resolve merge conflicts?
cxorm commented on pull request #399:
URL: https://github.com/apache/hadoop-ozone/pull/399#issuecomment-622122074
Rebase latest master-branch (#839) to resolve conflict.
This is an automated message from the Apache Git
[
https://issues.apache.org/jira/browse/HDDS-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HDDS-3510:
-
Labels: pull-request-available (was: )
> Upgrade Ratis to 0.6.0-2816ea6-SNAPSHOT
>
adoroszlai commented on pull request #888:
URL: https://github.com/apache/hadoop-ozone/pull/888#issuecomment-622257964
> Should we also update the License Notice -
https://github.com/apache/hadoop-ozone/blob/master/hadoop-ozone/dist/src/main/license/bin/NOTICE.txt#L269
?
Thanks for
adoroszlai commented on pull request #523:
URL: https://github.com/apache/hadoop-ozone/pull/523#issuecomment-622253214
@xiaoyuyao could you please review?
This is an automated message from the Apache Git Service.
To respond
[
https://issues.apache.org/jira/browse/HDDS-2796?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HDDS-2796:
-
Labels: pull-request-available (was: )
> beyond/_index.md translation
>
bharatviswa504 commented on pull request #691:
URL: https://github.com/apache/hadoop-ozone/pull/691#issuecomment-622246921
> > Overall LGTM. One minor comment and a general question.
> > I have one question in general, will this add some overhead and is there
a way to disable it?
> >
[
https://issues.apache.org/jira/browse/HDDS-3495?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Attila Doroszlai resolved HDDS-3495.
Fix Version/s: 0.6.0
Resolution: Done
> Clarify the document of OzoneFs
>
[
https://issues.apache.org/jira/browse/HDDS-3495?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Attila Doroszlai updated HDDS-3495:
---
Labels: (was: pull-request-available)
> Clarify the document of OzoneFs
>
adoroszlai opened a new pull request #890:
URL: https://github.com/apache/hadoop-ozone/pull/890
## What changes were proposed in this pull request?
Upgrade Ratis to `0.6.0-2816ea6-SNAPSHOT` to get the fix for
[RATIS-840](https://issues.apache.org/jira/browse/RATIS-840).
adoroszlai opened a new pull request #888:
URL: https://github.com/apache/hadoop-ozone/pull/888
## What changes were proposed in this pull request?
Few parts of Ozone still use `commons-lang`, while most are already on
`commons-lang3`. This change proposes to update those remaining
[
https://issues.apache.org/jira/browse/HDDS-3502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HDDS-3502:
-
Labels: pull-request-available (was: )
> Remove dependence on commons-lang
>
[
https://issues.apache.org/jira/browse/HDDS-3502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Attila Doroszlai updated HDDS-3502:
---
Status: Patch Available (was: In Progress)
> Remove dependence on commons-lang
>
bharatviswa504 edited a comment on pull request #855:
URL: https://github.com/apache/hadoop-ozone/pull/855#issuecomment-621947648
> > Thanks the answer @bharatviswa504 . Still working on my understanding
but one small comments in the mean time:
> > If the table is TransactionInfo ->
bharatviswa504 edited a comment on pull request #855:
URL: https://github.com/apache/hadoop-ozone/pull/855#issuecomment-621947648
> > Thanks the answer @bharatviswa504 . Still working on my understanding
but one small comments in the mean time:
> > If the table is TransactionInfo ->
[
https://issues.apache.org/jira/browse/HDDS-3512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17096526#comment-17096526
]
Steve Loughran commented on HDDS-3512:
--
no idea
> s3g multi-upload saved content incorrect when
cxorm commented on pull request #866:
URL: https://github.com/apache/hadoop-ozone/pull/866#issuecomment-621885884
Thanks @iamabug for the translation.
Overall LGTM. Just a minor comment.
This is an automated message
cxorm commented on a change in pull request #870:
URL: https://github.com/apache/hadoop-ozone/pull/870#discussion_r418026833
##
File path: hadoop-hdds/docs/content/security/SecureOzone.zh.md
##
@@ -0,0 +1,158 @@
+---
+title: "安全化 Ozone"
+date: "2019-April-03"
+summary: 简要介绍
cxorm commented on a change in pull request #866:
URL: https://github.com/apache/hadoop-ozone/pull/866#discussion_r418046012
##
File path: hadoop-hdds/docs/content/security/_index.zh.md
##
@@ -0,0 +1,33 @@
+---
+title: 安全
+name: Security
+identifier: SecureOzone
+menu: main
bharatviswa504 commented on pull request #855:
URL: https://github.com/apache/hadoop-ozone/pull/855#issuecomment-621939995
> Thanks the answer @bharatviswa504 . Still working on my understanding but
one small comments in the mean time:
>
> If the table is TransactionInfo ->
cxorm commented on pull request #870:
URL: https://github.com/apache/hadoop-ozone/pull/870#issuecomment-621870939
Thanks @iamabug for the translation.
Overall LGTM, a minor title could be consistent with the latest release.
bharatviswa504 edited a comment on pull request #855:
URL: https://github.com/apache/hadoop-ozone/pull/855#issuecomment-621939995
> Thanks the answer @bharatviswa504 . Still working on my understanding but
one small comments in the mean time:
>
> If the table is TransactionInfo ->
bharatviswa504 commented on pull request #855:
URL: https://github.com/apache/hadoop-ozone/pull/855#issuecomment-621947648
> > Thanks the answer @bharatviswa504 . Still working on my understanding
but one small comments in the mean time:
> > If the table is TransactionInfo ->
cxorm commented on a change in pull request #770:
URL: https://github.com/apache/hadoop-ozone/pull/770#discussion_r417775797
##
File path:
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/request/security/TestOMGetDelegationTokenRequest.java
##
@@ -0,0
[
https://issues.apache.org/jira/browse/HDDS-2406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Attila Doroszlai resolved HDDS-2406.
Fix Version/s: 0.6.0
Resolution: Fixed
Thanks [~flirmnave] for the contribution.
>
[
https://issues.apache.org/jira/browse/HDDS-2406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Attila Doroszlai updated HDDS-2406:
---
Labels: (was: pull-request-available)
> ozone shell key get throws IllegalArgumentException
[
https://issues.apache.org/jira/browse/HDDS-3514?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
runzhiwang updated HDDS-3514:
-
Description: This depends on
[RATIS-845|https://issues.apache.org/jira/browse/RATIS-845], find the
runzhiwang opened a new pull request #885:
URL: https://github.com/apache/hadoop-ozone/pull/885
## What changes were proposed in this pull request?
This PR depends on
[RATIS-845](https://github.com/apache/incubator-ratis/pull/76), we should wait
RATIS-845 to merge.
As the
[
https://issues.apache.org/jira/browse/HDDS-3514?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HDDS-3514:
-
Labels: pull-request-available (was: )
> Memory leak of RaftServerImpl
>
runzhiwang created HDDS-3514:
Summary: Memory leak of RaftServerImpl
Key: HDDS-3514
URL: https://issues.apache.org/jira/browse/HDDS-3514
Project: Hadoop Distributed Data Store
Issue Type: Bug
[
https://issues.apache.org/jira/browse/HDDS-3513?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Simon Su reassigned HDDS-3513:
--
Assignee: Simon Su
> OzoneConfiguration missing ozone-site.xml by default.
>
[
https://issues.apache.org/jira/browse/HDDS-3469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shashikant Banerjee updated HDDS-3469:
--
Fix Version/s: 0.6.0
Resolution: Fixed
Status: Resolved (was: Patch
adoroszlai commented on pull request #878:
URL: https://github.com/apache/hadoop-ozone/pull/878#issuecomment-621715807
Thanks @bshashikant for reviewing and merging this.
This is an automated message from the Apache Git
[
https://issues.apache.org/jira/browse/HDDS-3482?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Marton Elek resolved HDDS-3482.
---
Fix Version/s: 0.6.0
Resolution: Fixed
> Impl getUriDefaultPort to BasicOzFsOzFs, return -1
elek commented on pull request #862:
URL: https://github.com/apache/hadoop-ozone/pull/862#issuecomment-621732469
Merged with d849904c5
This is an automated message from the Apache Git Service.
To respond to the message,
[
https://issues.apache.org/jira/browse/HDDS-3514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17096341#comment-17096341
]
Shashikant Banerjee commented on HDDS-3514:
---
[~elek]/[~avijayan], can please have a look?
>
github-actions[bot] commented on pull request #612:
URL: https://github.com/apache/hadoop-ozone/pull/612#issuecomment-621712297
Thank you very much for the patch. I am closing this PR __temporarily__ as
there was no
activity recently and it is waiting for response from its author.
elek commented on pull request #612:
URL: https://github.com/apache/hadoop-ozone/pull/612#issuecomment-621712174
/close
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
elek removed a comment on pull request #862:
URL: https://github.com/apache/hadoop-ozone/pull/862#issuecomment-621716069
Thanks to explain it. Sorry, I missed the description in the Jira, (side
note: I like your detailed problem descriptions).
Based on this description we don't need
[
https://issues.apache.org/jira/browse/HDDS-3482?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Marton Elek updated HDDS-3482:
--
Summary: Impl getUriDefaultPort to BasicOzFsOzFs, return -1 forever to pass
the checkPath (was:
elek commented on pull request #740:
URL: https://github.com/apache/hadoop-ozone/pull/740#issuecomment-621722576
> @elek I close it temporarily, because I‘m not sure whether hostname will
be used or not in future.
We can't be prepared for all the possible challenges in the future.
runzhiwang commented on pull request #740:
URL: https://github.com/apache/hadoop-ozone/pull/740#issuecomment-621725540
@elek It's ok to remove it. I get hostname from client because now hostname
only for log, unreliable does not matter. If you have no time, I am also
available to remove
elek commented on pull request #724:
URL: https://github.com/apache/hadoop-ozone/pull/724#issuecomment-621724954
Still +1
As I wrote earlier, I am fine to remove the (half-made) fork temporary to
have a working state and do proper (working) fork later.
> Still need some
elek commented on pull request #855:
URL: https://github.com/apache/hadoop-ozone/pull/855#issuecomment-621726993
Thanks the answer @bharatviswa504 . Still working on my understanding but
one small comments in the mean time:
If the table is TransactionInfo ->
runzhiwang edited a comment on pull request #740:
URL: https://github.com/apache/hadoop-ozone/pull/740#issuecomment-621725540
@elek It's ok to remove it. If you have no time, I am also available to
remove the hostname.
This
elek commented on pull request #578:
URL: https://github.com/apache/hadoop-ozone/pull/578#issuecomment-621733092
Thanks the review @arp7 @bshashikant @adoroszlai
Happy to achieve consensus with the help of a long conversation.
Will merge it soon.
[
https://issues.apache.org/jira/browse/HDDS-3053?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Marton Elek resolved HDDS-3053.
---
Fix Version/s: 0.6.0
Resolution: Fixed
> Decrease the number of the chunk writer threads
>
[
https://issues.apache.org/jira/browse/HDDS-3473?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HDDS-3473:
-
Labels: pull-request-available (was: )
> ozone chunkinfo CLI should display block file path info
sadanand48 opened a new pull request #886:
URL: https://github.com/apache/hadoop-ozone/pull/886
## What changes were proposed in this pull request?
The debug tool CLI was not considering both the file per block and file per
chunk strategies while displaying the file path .This change
mukul1987 commented on a change in pull request #886:
URL: https://github.com/apache/hadoop-ozone/pull/886#discussion_r417827508
##
File path:
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/ChunkKeyHandler.java
##
@@ -128,17 +131,23 @@ protected void
mukul1987 commented on pull request #828:
URL: https://github.com/apache/hadoop-ozone/pull/828#issuecomment-621681600
Also lets add an acceptance test to check that Mountd server is coming up
This is an automated message
mukul1987 commented on pull request #828:
URL: https://github.com/apache/hadoop-ozone/pull/828#issuecomment-621681101
Hi @prashantpogde , I can see close to 144 commits. Can you please rebase
the branch against master and then post the change.
Also there are still checkstyle and
aryangupta1998 opened a new pull request #887:
URL: https://github.com/apache/hadoop-ozone/pull/887
## What changes were proposed in this pull request?
'cp' (copy) command doesn’t work with hdfs-fuse file system. It works by
passing 'cp' command two times for the same file. It
[
https://issues.apache.org/jira/browse/HDDS-3492?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HDDS-3492:
-
Labels: pull-request-available (was: )
> With hdfs-fuse file implementation 'cp' command is not
elek commented on pull request #740:
URL: https://github.com/apache/hadoop-ozone/pull/740#issuecomment-621749349
> Might be some external providers like ranger might need this, not sure
ranger is currently using it or not.
I checked the ranger plugin and I couldn't see any usage:
adoroszlai commented on pull request #857:
URL: https://github.com/apache/hadoop-ozone/pull/857#issuecomment-621758109
Thanks @elek for reviewing and committing it.
This is an automated message from the Apache Git Service.
[
https://issues.apache.org/jira/browse/HDDS-3385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Marton Elek updated HDDS-3385:
--
Fix Version/s: 0.6.0
Resolution: Fixed
Status: Resolved (was: Patch Available)
>
86 matches
Mail list logo