Re: [PR] AMBARI-26020: upgrade Ambari Metrics to python3 [ambari-metrics]

2024-04-24 Thread via GitHub
virajjasani merged PR #126: URL: https://github.com/apache/ambari-metrics/pull/126 -- 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:

Re: [PR] AMBARI-26020: upgrade Ambari Metrics to python3 [ambari-metrics]

2024-04-23 Thread via GitHub
JiaLiangC commented on PR #126: URL: https://github.com/apache/ambari-metrics/pull/126#issuecomment-2073928704 @virajjasani Ambari python3 feature already merged, so could you help review this pr? -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] AMBARI-26020: upgrade Ambari Metrics to python3 [ambari-metrics]

2024-04-23 Thread via GitHub
JiaLiangC commented on code in PR #126: URL: https://github.com/apache/ambari-metrics/pull/126#discussion_r1577181525 ## ambari-metrics-host-monitoring/src/main/python/__init__.py: ## @@ -1,4 +1,4 @@ -#!/usr/bin/env python +#!/usr/bin/env python3 Review Comment: This is

Re: [PR] AMBARI-26020: upgrade Ambari Metrics to python3 [ambari-metrics]

2024-02-23 Thread via GitHub
timyuer commented on code in PR #126: URL: https://github.com/apache/ambari-metrics/pull/126#discussion_r1501329553 ## ambari-metrics-host-monitoring/src/main/python/__init__.py: ## @@ -1,4 +1,4 @@ -#!/usr/bin/env python +#!/usr/bin/env python3 Review Comment: It may be

Re: [PR] AMBARI-26020: upgrade Ambari Metrics to python3 [ambari-metrics]

2024-02-23 Thread via GitHub
timyuer commented on code in PR #126: URL: https://github.com/apache/ambari-metrics/pull/126#discussion_r1501329553 ## ambari-metrics-host-monitoring/src/main/python/__init__.py: ## @@ -1,4 +1,4 @@ -#!/usr/bin/env python +#!/usr/bin/env python3 Review Comment: It may be