[jira] [Work logged] (HDFS-16254) Cleanup protobuf on exit of hdfs_allowSnapshot
[ https://issues.apache.org/jira/browse/HDFS-16254?focusedWorklogId=661042=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-661042 ] ASF GitHub Bot logged work on HDFS-16254: - Author: ASF GitHub Bot Created on: 06/Oct/21 16:24 Start Date: 06/Oct/21 16:24 Worklog Time Spent: 10m Work Description: goiri merged pull request #3518: URL: https://github.com/apache/hadoop/pull/3518 -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 661042) Time Spent: 1h 10m (was: 1h) > Cleanup protobuf on exit of hdfs_allowSnapshot > -- > > Key: HDFS-16254 > URL: https://issues.apache.org/jira/browse/HDFS-16254 > Project: Hadoop HDFS > Issue Type: Bug > Components: hdfs-client, libhdfs++, tools >Affects Versions: 3.4.0 >Reporter: Gautham Banasandra >Assignee: Gautham Banasandra >Priority: Major > Labels: pull-request-available > Fix For: 3.4.0 > > Time Spent: 1h 10m > Remaining Estimate: 0h > > Need to move the call google::protobuf::ShutdownProtobufLibrary() to main > method instead of > [AllowSnapshot::HandlePath|https://github.com/apache/hadoop/blob/35a8d48872a13438d4c4199b6ef5b902105e2eb2/hadoop-hdfs-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/tools/hdfs-allow-snapshot/hdfs-allow-snapshot.cc#L116-L117] > since we want the clean-up tasks to run only when the program exits. > The current implementation doesn't cause any issues since > AllowSnapshot::HandlePath is called only once. -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Work logged] (HDFS-16254) Cleanup protobuf on exit of hdfs_allowSnapshot
[ https://issues.apache.org/jira/browse/HDFS-16254?focusedWorklogId=660406=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-660406 ] ASF GitHub Bot logged work on HDFS-16254: - Author: ASF GitHub Bot Created on: 05/Oct/21 15:52 Start Date: 05/Oct/21 15:52 Worklog Time Spent: 10m Work Description: hadoop-yetus commented on pull request #3518: URL: https://github.com/apache/hadoop/pull/3518#issuecomment-934535525 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexec | 1m 28s | | Docker mode activated. | _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | | No case conflicting files found. | | +0 :ok: | codespell | 0m 0s | | codespell was not available. | | +1 :green_heart: | @author | 0m 0s | | The patch does not contain any @author tags. | | +1 :green_heart: | test4tests | 0m 0s | | The patch appears to include 1 new or modified test files. | _ trunk Compile Tests _ | | +1 :green_heart: | mvninstall | 27m 39s | | trunk passed | | +1 :green_heart: | compile | 3m 46s | | trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 | | +1 :green_heart: | compile | 3m 29s | | trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 | | +1 :green_heart: | mvnsite | 0m 28s | | trunk passed | | +1 :green_heart: | shadedclient | 60m 35s | | branch has no errors when building and testing our client artifacts. | _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 0m 18s | | the patch passed | | +1 :green_heart: | compile | 3m 19s | | the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 | | +1 :green_heart: | cc | 3m 19s | | the patch passed | | +1 :green_heart: | golang | 3m 19s | | the patch passed | | +1 :green_heart: | javac | 3m 19s | | the patch passed | | +1 :green_heart: | compile | 3m 27s | | the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 | | +1 :green_heart: | cc | 3m 27s | | the patch passed | | +1 :green_heart: | golang | 3m 27s | | the patch passed | | +1 :green_heart: | javac | 3m 27s | | the patch passed | | +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks issues. | | +1 :green_heart: | mvnsite | 0m 19s | | the patch passed | | +1 :green_heart: | shadedclient | 25m 3s | | patch has no errors when building and testing our client artifacts. | _ Other Tests _ | | +1 :green_heart: | unit | 88m 38s | | hadoop-hdfs-native-client in the patch passed. | | +1 :green_heart: | asflicense | 0m 34s | | The patch does not generate ASF License warnings. | | | | 186m 33s | | | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hadoop/pull/3518 | | Optional Tests | dupname asflicense compile cc mvnsite javac unit codespell golang | | uname | Linux caa7d242ca97 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/bin/hadoop.sh | | git revision | trunk / 95b537ee6a9ff3082c9ad9bc773f86fd4be04e50 | | Default Java | Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 | | Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 | | Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/testReport/ | | Max. process+thread count | 737 (vs. ulimit of 5500) | | modules | C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client | | Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/console | | versions | git=2.25.1 maven=3.6.3 | | Powered by | Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org | This message was automatically generated. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 660406) Time Spent: 1h (was: 50m) > Cleanup protobuf on exit of hdfs_allowSnapshot >
[jira] [Work logged] (HDFS-16254) Cleanup protobuf on exit of hdfs_allowSnapshot
[ https://issues.apache.org/jira/browse/HDFS-16254?focusedWorklogId=660293=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-660293 ] ASF GitHub Bot logged work on HDFS-16254: - Author: ASF GitHub Bot Created on: 05/Oct/21 12:45 Start Date: 05/Oct/21 12:45 Worklog Time Spent: 10m Work Description: hadoop-yetus commented on pull request #3518: URL: https://github.com/apache/hadoop/pull/3518#issuecomment-934377997 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexec | 15m 54s | | Docker mode activated. | _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | | No case conflicting files found. | | +0 :ok: | codespell | 0m 0s | | codespell was not available. | | +1 :green_heart: | @author | 0m 0s | | The patch does not contain any @author tags. | | +1 :green_heart: | test4tests | 0m 0s | | The patch appears to include 1 new or modified test files. | _ trunk Compile Tests _ | | +1 :green_heart: | mvninstall | 26m 46s | | trunk passed | | +1 :green_heart: | compile | 2m 37s | | trunk passed | | +1 :green_heart: | mvnsite | 0m 26s | | trunk passed | | +1 :green_heart: | shadedclient | 54m 31s | | branch has no errors when building and testing our client artifacts. | _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 0m 16s | | the patch passed | | +1 :green_heart: | compile | 2m 22s | | the patch passed | | +1 :green_heart: | cc | 2m 22s | | the patch passed | | +1 :green_heart: | golang | 2m 22s | | the patch passed | | +1 :green_heart: | javac | 2m 22s | | the patch passed | | +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks issues. | | +1 :green_heart: | mvnsite | 0m 18s | | the patch passed | | +1 :green_heart: | shadedclient | 24m 53s | | patch has no errors when building and testing our client artifacts. | _ Other Tests _ | | +1 :green_heart: | unit | 108m 57s | | hadoop-hdfs-native-client in the patch passed. | | +1 :green_heart: | asflicense | 0m 41s | | The patch does not generate ASF License warnings. | | | | 210m 21s | | | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hadoop/pull/3518 | | Optional Tests | dupname asflicense compile cc mvnsite javac unit codespell golang | | uname | Linux 9ffd18d828db 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/bin/hadoop.sh | | git revision | trunk / 95b537ee6a9ff3082c9ad9bc773f86fd4be04e50 | | Default Java | Debian-11.0.12+7-post-Debian-2deb10u1 | | Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/testReport/ | | Max. process+thread count | 731 (vs. ulimit of 5500) | | modules | C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client | | Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/console | | versions | git=2.20.1 maven=3.6.0 | | Powered by | Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org | This message was automatically generated. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 660293) Time Spent: 50m (was: 40m) > Cleanup protobuf on exit of hdfs_allowSnapshot > -- > > Key: HDFS-16254 > URL: https://issues.apache.org/jira/browse/HDFS-16254 > Project: Hadoop HDFS > Issue Type: Bug > Components: libhdfs++, tools >Affects Versions: 3.4.0 >Reporter: Gautham Banasandra >Assignee: Gautham Banasandra >Priority: Major > Labels: pull-request-available > Time Spent: 50m > Remaining Estimate: 0h > > Need to move the call google::protobuf::ShutdownProtobufLibrary() to main > method instead of >
[jira] [Work logged] (HDFS-16254) Cleanup protobuf on exit of hdfs_allowSnapshot
[ https://issues.apache.org/jira/browse/HDFS-16254?focusedWorklogId=660169=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-660169 ] ASF GitHub Bot logged work on HDFS-16254: - Author: ASF GitHub Bot Created on: 05/Oct/21 09:15 Start Date: 05/Oct/21 09:15 Worklog Time Spent: 10m Work Description: hadoop-yetus commented on pull request #3518: URL: https://github.com/apache/hadoop/pull/3518#issuecomment-934222381 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexec | 25m 37s | | Docker mode activated. | _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | | No case conflicting files found. | | +0 :ok: | codespell | 0m 0s | | codespell was not available. | | +1 :green_heart: | @author | 0m 0s | | The patch does not contain any @author tags. | | +1 :green_heart: | test4tests | 0m 0s | | The patch appears to include 1 new or modified test files. | _ trunk Compile Tests _ | | +1 :green_heart: | mvninstall | 21m 51s | | trunk passed | | +1 :green_heart: | compile | 2m 48s | | trunk passed | | +1 :green_heart: | mvnsite | 0m 43s | | trunk passed | | +1 :green_heart: | shadedclient | 43m 58s | | branch has no errors when building and testing our client artifacts. | _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 0m 22s | | the patch passed | | +1 :green_heart: | compile | 2m 37s | | the patch passed | | +1 :green_heart: | cc | 2m 37s | | the patch passed | | +1 :green_heart: | golang | 2m 37s | | the patch passed | | +1 :green_heart: | javac | 2m 37s | | the patch passed | | +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks issues. | | +1 :green_heart: | mvnsite | 0m 22s | | the patch passed | | +1 :green_heart: | shadedclient | 18m 7s | | patch has no errors when building and testing our client artifacts. | _ Other Tests _ | | +1 :green_heart: | unit | 106m 27s | | hadoop-hdfs-native-client in the patch passed. | | +1 :green_heart: | asflicense | 0m 46s | | The patch does not generate ASF License warnings. | | | | 200m 49s | | | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hadoop/pull/3518 | | Optional Tests | dupname asflicense compile cc mvnsite javac unit codespell golang | | uname | Linux d41f2df2da4e 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/bin/hadoop.sh | | git revision | trunk / 95b537ee6a9ff3082c9ad9bc773f86fd4be04e50 | | Default Java | Red Hat, Inc.-1.8.0_302-b08 | | Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/testReport/ | | Max. process+thread count | 544 (vs. ulimit of 5500) | | modules | C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client | | Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/console | | versions | git=2.27.0 maven=3.6.3 | | Powered by | Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org | This message was automatically generated. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 660169) Time Spent: 40m (was: 0.5h) > Cleanup protobuf on exit of hdfs_allowSnapshot > -- > > Key: HDFS-16254 > URL: https://issues.apache.org/jira/browse/HDFS-16254 > Project: Hadoop HDFS > Issue Type: Bug > Components: libhdfs++, tools >Affects Versions: 3.4.0 >Reporter: Gautham Banasandra >Assignee: Gautham Banasandra >Priority: Major > Labels: pull-request-available > Time Spent: 40m > Remaining Estimate: 0h > > Need to move the call google::protobuf::ShutdownProtobufLibrary() to main > method instead of >
[jira] [Work logged] (HDFS-16254) Cleanup protobuf on exit of hdfs_allowSnapshot
[ https://issues.apache.org/jira/browse/HDFS-16254?focusedWorklogId=660120=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-660120 ] ASF GitHub Bot logged work on HDFS-16254: - Author: ASF GitHub Bot Created on: 05/Oct/21 05:54 Start Date: 05/Oct/21 05:54 Worklog Time Spent: 10m Work Description: hadoop-yetus commented on pull request #3518: URL: https://github.com/apache/hadoop/pull/3518#issuecomment-934089364 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexec | 43m 2s | | Docker mode activated. | _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | | No case conflicting files found. | | +0 :ok: | codespell | 0m 0s | | codespell was not available. | | +1 :green_heart: | @author | 0m 0s | | The patch does not contain any @author tags. | | +1 :green_heart: | test4tests | 0m 0s | | The patch appears to include 1 new or modified test files. | _ trunk Compile Tests _ | | +1 :green_heart: | mvninstall | 35m 24s | | trunk passed | | +1 :green_heart: | compile | 2m 38s | | trunk passed | | +1 :green_heart: | mvnsite | 0m 29s | | trunk passed | | +1 :green_heart: | shadedclient | 56m 56s | | branch has no errors when building and testing our client artifacts. | _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 0m 17s | | the patch passed | | +1 :green_heart: | compile | 2m 27s | | the patch passed | | +1 :green_heart: | cc | 2m 27s | | the patch passed | | +1 :green_heart: | golang | 2m 27s | | the patch passed | | +1 :green_heart: | javac | 2m 27s | | the patch passed | | +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks issues. | | +1 :green_heart: | mvnsite | 0m 18s | | the patch passed | | +1 :green_heart: | shadedclient | 17m 45s | | patch has no errors when building and testing our client artifacts. | _ Other Tests _ | | +1 :green_heart: | unit | 95m 41s | | hadoop-hdfs-native-client in the patch passed. | | +1 :green_heart: | asflicense | 0m 36s | | The patch does not generate ASF License warnings. | | | | 219m 36s | | | | Subsystem | Report/Notes | |--:|:-| | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hadoop/pull/3518 | | Optional Tests | dupname asflicense compile cc mvnsite javac unit codespell golang | | uname | Linux d0e0d2ac28a4 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/bin/hadoop.sh | | git revision | trunk / 95b537ee6a9ff3082c9ad9bc773f86fd4be04e50 | | Default Java | Red Hat, Inc.-1.8.0_302-b08 | | Test Results | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/testReport/ | | Max. process+thread count | 625 (vs. ulimit of 5500) | | modules | C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client | | Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/2/console | | versions | git=2.9.5 maven=3.6.3 | | Powered by | Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org | This message was automatically generated. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 660120) Time Spent: 0.5h (was: 20m) > Cleanup protobuf on exit of hdfs_allowSnapshot > -- > > Key: HDFS-16254 > URL: https://issues.apache.org/jira/browse/HDFS-16254 > Project: Hadoop HDFS > Issue Type: Bug > Components: libhdfs++, tools >Affects Versions: 3.4.0 >Reporter: Gautham Banasandra >Assignee: Gautham Banasandra >Priority: Major > Labels: pull-request-available > Time Spent: 0.5h > Remaining Estimate: 0h > > Need to move the call google::protobuf::ShutdownProtobufLibrary() to main > method instead of >
[jira] [Work logged] (HDFS-16254) Cleanup protobuf on exit of hdfs_allowSnapshot
[ https://issues.apache.org/jira/browse/HDFS-16254?focusedWorklogId=659817=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-659817 ] ASF GitHub Bot logged work on HDFS-16254: - Author: ASF GitHub Bot Created on: 04/Oct/21 19:09 Start Date: 04/Oct/21 19:09 Worklog Time Spent: 10m Work Description: hadoop-yetus commented on pull request #3518: URL: https://github.com/apache/hadoop/pull/3518#issuecomment-933774121 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexec | 0m 0s | | Docker mode activated. | | -1 :x: | docker | 23m 37s | | Docker failed to build yetus/hadoop:13467f45240. | | Subsystem | Report/Notes | |--:|:-| | GITHUB PR | https://github.com/apache/hadoop/pull/3518 | | Console output | https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3518/1/console | | versions | git=2.17.1 | | Powered by | Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org | This message was automatically generated. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 659817) Time Spent: 20m (was: 10m) > Cleanup protobuf on exit of hdfs_allowSnapshot > -- > > Key: HDFS-16254 > URL: https://issues.apache.org/jira/browse/HDFS-16254 > Project: Hadoop HDFS > Issue Type: Bug > Components: libhdfs++, tools >Affects Versions: 3.4.0 >Reporter: Gautham Banasandra >Assignee: Gautham Banasandra >Priority: Major > Labels: pull-request-available > Time Spent: 20m > Remaining Estimate: 0h > > Need to move the call google::protobuf::ShutdownProtobufLibrary() to main > method instead of > [AllowSnapshot::HandlePath|https://github.com/apache/hadoop/blob/35a8d48872a13438d4c4199b6ef5b902105e2eb2/hadoop-hdfs-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/tools/hdfs-allow-snapshot/hdfs-allow-snapshot.cc#L116-L117] > since we want the clean-up tasks to run only when the program exits. > The current implementation doesn't cause any issues since > AllowSnapshot::HandlePath is called only once. -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Work logged] (HDFS-16254) Cleanup protobuf on exit of hdfs_allowSnapshot
[ https://issues.apache.org/jira/browse/HDFS-16254?focusedWorklogId=659787=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-659787 ] ASF GitHub Bot logged work on HDFS-16254: - Author: ASF GitHub Bot Created on: 04/Oct/21 18:43 Start Date: 04/Oct/21 18:43 Worklog Time Spent: 10m Work Description: GauthamBanasandra opened a new pull request #3518: URL: https://github.com/apache/hadoop/pull/3518 ### Description of PR * Moved the call google::protobuf::ShutdownProtobufLibrary() to main method instead of AllowSnapshot::HandlePath since we want the clean-up tasks to run only when the program exits. ### How was this patch tested? Unit tests ran fine. Also, verified by running the tool locally. ### For code changes: - [x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')? - [ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation? - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)? - [ ] If applicable, have you updated the `LICENSE`, `LICENSE-binary`, `NOTICE-binary` files? -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 659787) Remaining Estimate: 0h Time Spent: 10m > Cleanup protobuf on exit of hdfs_allowSnapshot > -- > > Key: HDFS-16254 > URL: https://issues.apache.org/jira/browse/HDFS-16254 > Project: Hadoop HDFS > Issue Type: Bug > Components: libhdfs++, tools >Affects Versions: 3.4.0 >Reporter: Gautham Banasandra >Assignee: Gautham Banasandra >Priority: Major > Time Spent: 10m > Remaining Estimate: 0h > > Need to move the call google::protobuf::ShutdownProtobufLibrary() to main > method instead of > [AllowSnapshot::HandlePath|https://github.com/apache/hadoop/blob/35a8d48872a13438d4c4199b6ef5b902105e2eb2/hadoop-hdfs-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/tools/hdfs-allow-snapshot/hdfs-allow-snapshot.cc#L116-L117] > since we want the clean-up tasks to run only when the program exits. > The current implementation doesn't cause any issues since > AllowSnapshot::HandlePath is called only once. -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org