This is an automated email from the ASF dual-hosted git repository.
jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new d40cba4d50 10218: Added log statement for jvm options. (#10446)
d40cba4d50 is described below
commit d40cba4d50ad656f7495d029787aa6ca413d54bc
Author: Abhishek Sharma <abhishek.sha...@spothero.com>
AuthorDate: Tue Mar 28 03:51:04 2023 -0400
10218: Added log statement for jvm options. (#10446)
---
.../java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java | 2 ++
.../main/java/org/apache/pinot/controller/BaseControllerStarter.java | 2 ++
.../src/main/java/org/apache/pinot/minion/BaseMinionStarter.java | 2 ++
.../java/org/apache/pinot/server/starter/helix/BaseServerStarter.java | 2 ++
4 files changed, 8 insertions(+)
diff --git
a/pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java
b/pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java
index 0f19a7e4ed..e9b6ec3c9a 100644
---
a/pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java
+++
b/pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java
@@ -56,6 +56,7 @@ import org.apache.pinot.common.metadata.ZKMetadataProvider;
import org.apache.pinot.common.metrics.BrokerGauge;
import org.apache.pinot.common.metrics.BrokerMeter;
import org.apache.pinot.common.metrics.BrokerMetrics;
+import org.apache.pinot.common.utils.PinotAppConfigs;
import org.apache.pinot.common.utils.ServiceStartableUtils;
import org.apache.pinot.common.utils.ServiceStatus;
import org.apache.pinot.common.utils.TlsUtils;
@@ -223,6 +224,7 @@ public abstract class BaseBrokerStarter implements
ServiceStartable {
public void start()
throws Exception {
LOGGER.info("Starting Pinot broker (Version: {})", PinotVersion.VERSION);
+ LOGGER.info("Broker configs: {}", new
PinotAppConfigs(getConfig()).toJSONString());
_isStarting = true;
Utils.logVersions();
diff --git
a/pinot-controller/src/main/java/org/apache/pinot/controller/BaseControllerStarter.java
b/pinot-controller/src/main/java/org/apache/pinot/controller/BaseControllerStarter.java
index 6be44b068c..381097a1b6 100644
---
a/pinot-controller/src/main/java/org/apache/pinot/controller/BaseControllerStarter.java
+++
b/pinot-controller/src/main/java/org/apache/pinot/controller/BaseControllerStarter.java
@@ -60,6 +60,7 @@ import org.apache.pinot.common.metrics.ValidationMetrics;
import org.apache.pinot.common.minion.InMemoryTaskManagerStatusCache;
import org.apache.pinot.common.minion.TaskGeneratorMostRecentRunInfo;
import org.apache.pinot.common.minion.TaskManagerStatusCache;
+import org.apache.pinot.common.utils.PinotAppConfigs;
import org.apache.pinot.common.utils.ServiceStartableUtils;
import org.apache.pinot.common.utils.ServiceStatus;
import org.apache.pinot.common.utils.TlsUtils;
@@ -319,6 +320,7 @@ public abstract class BaseControllerStarter implements
ServiceStartable {
@Override
public void start() {
LOGGER.info("Starting Pinot controller in mode: {}. (Version: {})",
_controllerMode.name(), PinotVersion.VERSION);
+ LOGGER.info("Controller configs: {}", new
PinotAppConfigs(getConfig()).toJSONString());
Utils.logVersions();
// Set up controller metrics
diff --git
a/pinot-minion/src/main/java/org/apache/pinot/minion/BaseMinionStarter.java
b/pinot-minion/src/main/java/org/apache/pinot/minion/BaseMinionStarter.java
index 2448af71c9..845ec95e66 100644
--- a/pinot-minion/src/main/java/org/apache/pinot/minion/BaseMinionStarter.java
+++ b/pinot-minion/src/main/java/org/apache/pinot/minion/BaseMinionStarter.java
@@ -42,6 +42,7 @@ import org.apache.pinot.common.metrics.MinionGauge;
import org.apache.pinot.common.metrics.MinionMeter;
import org.apache.pinot.common.metrics.MinionMetrics;
import org.apache.pinot.common.utils.ClientSSLContextGenerator;
+import org.apache.pinot.common.utils.PinotAppConfigs;
import org.apache.pinot.common.utils.ServiceStartableUtils;
import org.apache.pinot.common.utils.ServiceStatus;
import org.apache.pinot.common.utils.TlsUtils;
@@ -167,6 +168,7 @@ public abstract class BaseMinionStarter implements
ServiceStartable {
public void start()
throws Exception {
LOGGER.info("Starting Pinot minion: {} (Version: {})", _instanceId,
PinotVersion.VERSION);
+ LOGGER.info("Minion configs: {}", new
PinotAppConfigs(getConfig()).toJSONString());
Utils.logVersions();
MinionContext minionContext = MinionContext.getInstance();
diff --git
a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java
b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java
index 4a09290822..e9fdab9511 100644
---
a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java
+++
b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java
@@ -54,6 +54,7 @@ import org.apache.pinot.common.metadata.ZKMetadataProvider;
import org.apache.pinot.common.metrics.ServerMeter;
import org.apache.pinot.common.metrics.ServerMetrics;
import org.apache.pinot.common.restlet.resources.SystemResourceInfo;
+import org.apache.pinot.common.utils.PinotAppConfigs;
import org.apache.pinot.common.utils.ServiceStartableUtils;
import org.apache.pinot.common.utils.ServiceStatus;
import org.apache.pinot.common.utils.ServiceStatus.Status;
@@ -502,6 +503,7 @@ public abstract class BaseServerStarter implements
ServiceStartable {
public void start()
throws Exception {
LOGGER.info("Starting Pinot server (Version: {})", PinotVersion.VERSION);
+ LOGGER.info("Server configs: {}", new
PinotAppConfigs(getConfig()).toJSONString());
long startTimeMs = System.currentTimeMillis();
// install default SSL context if necessary (even if not force-enabled
everywhere)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
