[DRILL-6275] drillbit direct_current memory usage is not populated/updated

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.13.0
• Fix Version/s: 1.14.0
• Component/s: Metadata
• Labels: ready-to-commit
• Assignee: Timothy Farkas
• Reporter: Chun Chang
• Reviewer: Kunal Khatua
• Created: 19/Mar/18 19:12
• Updated: 29/Mar/18 17:12
• Resolved: 29/Mar/18 17:12

Description

We used to keep track drill memory usage in sys.memory. And it was useful in detecting memory leaks. This feature seems broken. The direct_current memory usage is not populated or updated.

0: jdbc:drill:zk=10.10.30.166:5181> select * from sys.memory;
+---------------+------------+---------------+-------------+-----------------+---------------------+--------------+
| hostname | user_port | heap_current | heap_max | direct_current | jvm_direct_current | direct_max |
+---------------+------------+---------------+-------------+-----------------+---------------------+--------------+
| 10.10.30.168 | 31010 | 1162636800 | 2147483648 | 0 | 22096 | 10737418240 |
| 10.10.30.169 | 31010 | 1301175040 | 2147483648 | 0 | 22096 | 10737418240 |
| 10.10.30.166 | 31010 | 989448872 | 2147483648 | 0 | 22096 | 10737418240 |
| 10.10.30.167 | 31010 | 1767205312 | 2147483648 | 0 | 22096 | 10737418240 |
+---------------+------------+---------------+-------------+-----------------+---------------------+--------------+
4 rows selected (1.564 seconds)
0: jdbc:drill:zk=10.10.30.166:5181> select * from sys.version;
+------------------+-------------------------------------------+-------------------------------------------------------+----------------------------+---------------------+----------------------------+
| version | commit_id | commit_message | commit_time | build_email | build_time |
+------------------+-------------------------------------------+-------------------------------------------------------+----------------------------+---------------------+----------------------------+
| 1.13.0-SNAPSHOT | 534212456cc25a49272838cba91c223f63df7fd2 | Cleanup when closing, and cleanup spill after a kill | 07.03.2018 @ 16:18:27 PST | [e-mail] | 08.03.2018 @ 10:09:28 PST |
+------------------+-------------------------------------------+-------------------------------------------------------+----------------------------+---------------------+----------------------------+

Issue Links

• links to: GitHub Pull Request #1176