Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2024-06-12 Thread via GitHub


github-actions[bot] closed pull request #4854: HIVE-27854:Driver compile may 
causing memory leak
URL: https://github.com/apache/hive/pull/4854


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2024-06-05 Thread via GitHub


github-actions[bot] commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-2151160087

   This pull request has been automatically marked as stale because it has not 
had recent activity. It will be closed if no further activity occurs.
   Feel free to reach out on the d...@hive.apache.org list if the patch is in 
need of reviews.


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2024-02-05 Thread via GitHub


github-actions[bot] closed pull request #4854: HIVE-27854:Driver compile may 
causing memory leak
URL: https://github.com/apache/hive/pull/4854


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2024-02-05 Thread via GitHub


github-actions[bot] closed pull request #4856: HIVE-27854:Driver compile may 
causing memory leak
URL: https://github.com/apache/hive/pull/4856


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2024-01-29 Thread via GitHub


github-actions[bot] commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1915814624

   This pull request has been automatically marked as stale because it has not 
had recent activity. It will be closed if no further activity occurs.
   Feel free to reach out on the d...@hive.apache.org list if the patch is in 
need of reviews.


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2024-01-29 Thread via GitHub


github-actions[bot] commented on PR #4856:
URL: https://github.com/apache/hive/pull/4856#issuecomment-1915814591

   This pull request has been automatically marked as stale because it has not 
had recent activity. It will be closed if no further activity occurs.
   Feel free to reach out on the d...@hive.apache.org list if the patch is in 
need of reviews.


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


chenruotao commented on code in PR #4854:
URL: https://github.com/apache/hive/pull/4854#discussion_r1410700755


##
ql/src/java/org/apache/hadoop/hive/ql/DriverState.java:
##
@@ -89,7 +89,11 @@ public void compiling() {
   public void compilingWithLocking() {
 lock();
 try {
-  driverState = State.COMPILING;
+  if (isDestroyed() || isClosed()) {
+abort();

Review Comment:
   yes, but the question is that the driver may be set null when the query is 
in the middle of compilation, and if the compiling thread just runs before 
lock(), the driverState will be reset to State.COMPILING from State.CLOSED OR 
State.DESTROYED, and keep running.



-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


deniskuzZ commented on code in PR #4854:
URL: https://github.com/apache/hive/pull/4854#discussion_r1410687988


##
ql/src/java/org/apache/hadoop/hive/ql/DriverState.java:
##
@@ -89,7 +89,11 @@ public void compiling() {
   public void compilingWithLocking() {
 lock();
 try {
-  driverState = State.COMPILING;
+  if (isDestroyed() || isClosed()) {
+abort();

Review Comment:
   `abort` is only used if the query is in the middle of compilation or 
execution 
   
   if (driverState.isCompiling() || driverState.isExecuting()) {
   driverState.abort();
   }
   



-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


chenruotao commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1833779007

   > in SQLOperation#cleanup we call driver.close before settting it to NULL
   > 
   > ```
   > if (driver != null) {
   >   driver.close();
   >   driver.destroy();
   > }
   > driver = null;
   > ```
   
   In my test, set driver = null can not stop the compiling thread, only set 
aborted = true can interrupt the thread, the thread still run after set null, 
and it is the root cause of  memory leak in Hive 3


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


deniskuzZ commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1833755597

   in SQLOperation#cleanup we call driver.close before settting it to NULL
   
   if (driver != null) {
 driver.close();
 driver.destroy();
   }
   driver = null;
   


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


chenruotao commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1833738961

   And the PR on Hive 3 can fix the Bug


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


chenruotao commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1833738080

   > @chenruotao, code linked in a JIRA is not relevant for Hive 4. Could you 
reproduce the same issue in master branch?
   
   I can not reproduce on master branch, it just happend on hive 3, but I still 
think it should not go on compiling after it be closed or destroyed,so i also 
put a PR on master


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


deniskuzZ commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1833719906

   @chenruotao, code linked in a JIRA is not relevant for Hive 4. Could you 
reproduce the same issue in master branch?


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


chenruotao commented on PR #4856:
URL: https://github.com/apache/hive/pull/4856#issuecomment-1833716389

   > @chenruotao, can we close this PR as it has a duplicate?
   
   OK, if it doesn't need to be fixed in version 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 unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-30 Thread via GitHub


deniskuzZ commented on PR #4856:
URL: https://github.com/apache/hive/pull/4856#issuecomment-1833705149

   @chenruotao, can we close this PR as it has a duplicate?


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-08 Thread via GitHub


chenruotao commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1801819744

   @aturoczy Hi i do test on master, this commit code can stop driver go on 
compiling after be closed or destroyed, but master would not cause memory leak 
even no this commit code, anyway, i still think the driver should not go on 
compiling after that, maybe cause other problems, please take a look.


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-08 Thread via GitHub


aturoczy commented on PR #4856:
URL: https://github.com/apache/hive/pull/4856#issuecomment-1801543417

   Sure! Please ping me on that PR 


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-08 Thread via GitHub


chenruotao commented on PR #4856:
URL: https://github.com/apache/hive/pull/4856#issuecomment-1801540137

   > > > I would merge this PR with #4854 Both PR are belong to the same issue. 
Does not need 2 PR for this.
   > > > Btw is this resolved on master?
   > > 
   > > 
   > > sorry for 2 PR, because the related code in master has a difference from 
brance 3.1.2, so I submit 2 PR, one for brance 3.1, one for master. And I think 
there is the same problem on master, but it can not return directly, so i set 
abort.
   > 
   > Please check on master. It would be better to have this PR on master, and 
we after that backport to 3.1. It could be a valuable change for all version
   
   ok, I will do update on https://github.com/apache/hive/pull/4854 later, that 
is on master, please take a review


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-08 Thread via GitHub


aturoczy commented on PR #4856:
URL: https://github.com/apache/hive/pull/4856#issuecomment-1801516099

   > > I would merge this PR with #4854 Both PR are belong to the same issue. 
Does not need 2 PR for this.
   > > Btw is this resolved on master?
   > 
   > sorry for 2 PR, because the related code in master has a difference from 
brance 3.1.2, so I submit 2 PR, one for brance 3.1, one for master. And I think 
there is the same problem on master, but it can not return directly, so i set 
abort.
   
   Please check on master. It would be better to have this PR on master, and we 
after that backport to 3.1. It could be a valuable change for all version


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-08 Thread via GitHub


chenruotao commented on PR #4856:
URL: https://github.com/apache/hive/pull/4856#issuecomment-1801509860

   > I would merge this PR with #4854 Both PR are belong to the same issue. 
Does not need 2 PR for this.
   > 
   > Btw is this resolved on master?
   
   sorry for 2 PR, because the related code in master  has a difference from 
brance 3.1.2, so I submit 2 PR, one for brance 3.1, one for master.  And I 
think there is the same problem on master, but it can not return directly, i 
set abort.


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-08 Thread via GitHub


aturoczy commented on code in PR #4856:
URL: https://github.com/apache/hive/pull/4856#discussion_r1386324348


##
ql/src/java/org/apache/hadoop/hive/ql/Driver.java:
##
@@ -508,6 +508,10 @@ private void compile(String command, boolean resetTaskIds, 
boolean deferClose) t
 perfLogger.PerfLogBegin(CLASS_NAME, PerfLogger.COMPILE);
 lDrvState.stateLock.lock();
 try {
+  if (lDrvState.driverState == DriverState.DESTROYED || 
lDrvState.driverState == DriverState.CLOSED) {
+LOG.warn("FAILED: Query command could not be compiled because driver 
has been cancelled, closed or destroyed.");

Review Comment:
   ok



-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-08 Thread via GitHub


chenruotao commented on code in PR #4856:
URL: https://github.com/apache/hive/pull/4856#discussion_r1386320870


##
ql/src/java/org/apache/hadoop/hive/ql/Driver.java:
##
@@ -508,6 +508,10 @@ private void compile(String command, boolean resetTaskIds, 
boolean deferClose) t
 perfLogger.PerfLogBegin(CLASS_NAME, PerfLogger.COMPILE);
 lDrvState.stateLock.lock();
 try {
+  if (lDrvState.driverState == DriverState.DESTROYED || 
lDrvState.driverState == DriverState.CLOSED) {
+LOG.warn("FAILED: Query command could not be compiled because driver 
has been cancelled, closed or destroyed.");

Review Comment:
   But there is no driverState named canceled, i think the closed of 
driverStates include canceled



-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-08 Thread via GitHub


aturoczy commented on code in PR #4856:
URL: https://github.com/apache/hive/pull/4856#discussion_r1386291771


##
ql/src/java/org/apache/hadoop/hive/ql/Driver.java:
##
@@ -508,6 +508,10 @@ private void compile(String command, boolean resetTaskIds, 
boolean deferClose) t
 perfLogger.PerfLogBegin(CLASS_NAME, PerfLogger.COMPILE);
 lDrvState.stateLock.lock();
 try {
+  if (lDrvState.driverState == DriverState.DESTROYED || 
lDrvState.driverState == DriverState.CLOSED) {
+LOG.warn("FAILED: Query command could not be compiled because driver 
has been cancelled, closed or destroyed.");

Review Comment:
   Maybe better:
   _Query compilation failed as the driver was canceled, closed, or destroyed_ 



-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-07 Thread via GitHub


sonarcloud[bot] commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1798019402

   Kudos, SonarCloud Quality Gate passed!  [![Quality Gate 
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
 'Quality Gate 
passed')](https://sonarcloud.io/dashboard?id=apache_hive=4854)
   
   
[![Bug](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/bug-16px.png
 
'Bug')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
  
   
[![Vulnerability](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/vulnerability-16px.png
 
'Vulnerability')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
  
   [![Security 
Hotspot](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/security_hotspot-16px.png
 'Security 
Hotspot')](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
  
   [![Code 
Smell](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/code_smell-16px.png
 'Code 
Smell')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
 [0 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
   
   [![No Coverage 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/CoverageChart/NoCoverageInfo-16px.png
 'No Coverage 
information')](https://sonarcloud.io/component_measures?id=apache_hive=4854=coverage=list)
 No Coverage information  
   [![No Duplication 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/Duplications/NoDuplicationInfo-16px.png
 'No Duplication 
information')](https://sonarcloud.io/component_measures?id=apache_hive=4854=duplicated_lines_density=list)
 No Duplication information
   
   
![warning](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/message_warning-16px.png
 'warning') The version of Java (11.0.8) you have used to run this analysis is 
deprecated and we will stop accepting it soon. Please update to at least Java 
17.
   Read more [here](https://docs.sonarcloud.io/appendices/scanner-environment/)
   
   
   


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-06 Thread via GitHub


sonarcloud[bot] commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1797814538

   Kudos, SonarCloud Quality Gate passed!  [![Quality Gate 
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
 'Quality Gate 
passed')](https://sonarcloud.io/dashboard?id=apache_hive=4854)
   
   
[![Bug](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/bug-16px.png
 
'Bug')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
  
   
[![Vulnerability](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/vulnerability-16px.png
 
'Vulnerability')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
  
   [![Security 
Hotspot](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/security_hotspot-16px.png
 'Security 
Hotspot')](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
  
   [![Code 
Smell](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/code_smell-16px.png
 'Code 
Smell')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
 [2 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
   
   [![No Coverage 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/CoverageChart/NoCoverageInfo-16px.png
 'No Coverage 
information')](https://sonarcloud.io/component_measures?id=apache_hive=4854=coverage=list)
 No Coverage information  
   [![No Duplication 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/Duplications/NoDuplicationInfo-16px.png
 'No Duplication 
information')](https://sonarcloud.io/component_measures?id=apache_hive=4854=duplicated_lines_density=list)
 No Duplication information
   
   
![warning](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/message_warning-16px.png
 'warning') The version of Java (11.0.8) you have used to run this analysis is 
deprecated and we will stop accepting it soon. Please update to at least Java 
17.
   Read more [here](https://docs.sonarcloud.io/appendices/scanner-environment/)
   
   
   


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-06 Thread via GitHub


chenruotao commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1797331451

   @miklosgergely hi, could you tell me why my code test fail ? please


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org



Re: [PR] HIVE-27854:Driver compile may causing memory leak [hive]

2023-11-06 Thread via GitHub


sonarcloud[bot] commented on PR #4854:
URL: https://github.com/apache/hive/pull/4854#issuecomment-1794722645

   Kudos, SonarCloud Quality Gate passed!  [![Quality Gate 
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
 'Quality Gate 
passed')](https://sonarcloud.io/dashboard?id=apache_hive=4854)
   
   
[![Bug](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/bug-16px.png
 
'Bug')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=BUG)
  
   
[![Vulnerability](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/vulnerability-16px.png
 
'Vulnerability')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=VULNERABILITY)
  
   [![Security 
Hotspot](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/security_hotspot-16px.png
 'Security 
Hotspot')](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_hive=4854=false=SECURITY_HOTSPOT)
  
   [![Code 
Smell](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/code_smell-16px.png
 'Code 
Smell')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
 [0 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_hive=4854=false=CODE_SMELL)
   
   [![No Coverage 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/CoverageChart/NoCoverageInfo-16px.png
 'No Coverage 
information')](https://sonarcloud.io/component_measures?id=apache_hive=4854=coverage=list)
 No Coverage information  
   [![No Duplication 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/Duplications/NoDuplicationInfo-16px.png
 'No Duplication 
information')](https://sonarcloud.io/component_measures?id=apache_hive=4854=duplicated_lines_density=list)
 No Duplication information
   
   
![warning](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/message_warning-16px.png
 'warning') The version of Java (11.0.8) you have used to run this analysis is 
deprecated and we will stop accepting it soon. Please update to at least Java 
17.
   Read more [here](https://docs.sonarcloud.io/appendices/scanner-environment/)
   
   
   


-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org