Vuk Ercegovac has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/9497


Change subject: IMPALA-6599: fixes return for NativeLibCacheSetNeedsRefresh
......................................................................

IMPALA-6599: fixes return for NativeLibCacheSetNeedsRefresh

Current fe_support.cc:[..]_FeSupport_NativeLibCacheSetNeedsRefresh
always returns false. In the frontend, this is logged, which causes
unneeded, incorrect, and confusing spam.

This method returns false if unable to manage the string input
argument (path). It then invokes the lib_cache's SetNeedsRefresh
which always succeeds (either the path does not exist or, if it exists,
needs refresh is set). This change modifies the return value after
this call to be true instead of false.

Testing:
- verified the spam without the change by looking at the logs from
  query_test/test_udfs.py (~4000 log messages)
- verified that none of these log messages show up with the change
  applied.

Change-Id: I11f34a63a25f5ab6acabcc2f52b7e8f22d8a4da3
---
M be/src/service/fe-support.cc
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/97/9497/1
--
To view, visit http://gerrit.cloudera.org:8080/9497
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I11f34a63a25f5ab6acabcc2f52b7e8f22d8a4da3
Gerrit-Change-Number: 9497
Gerrit-PatchSet: 1
Gerrit-Owner: Vuk Ercegovac <vercego...@cloudera.com>

Reply via email to