Skip to content

Commit 70e956e

Browse files
author
yanxi0227
committed
add log
1 parent 2c57f54 commit 70e956e

File tree

5 files changed

+32
-6
lines changed

5 files changed

+32
-6
lines changed

core/src/main/java/com/dtstack/flink/sql/Main.java

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,9 @@ public static void main(String[] args) throws Exception {
173173
}
174174

175175
for (InsertSqlParser.SqlParseResult result : sqlTree.getExecSqlList()) {
176-
LOG.info("exe-sql:\n" + result.getExecSql());
176+
if(LOG.isInfoEnabled()){
177+
LOG.info("exe-sql:\n" + result.getExecSql());
178+
}
177179

178180
boolean isSide = false;
179181

@@ -203,7 +205,9 @@ public static void main(String[] args) throws Exception {
203205
sideSqlExec.exec(result.getExecSql(), sideTableMap, tableEnv, registerTableCache);
204206
}else{
205207
tableEnv.sqlUpdate(result.getExecSql());
206-
LOG.info("exec sql: " + result.getExecSql());
208+
if(LOG.isInfoEnabled()){
209+
210+
}LOG.info("exec sql: " + result.getExecSql());
207211
}
208212
}
209213
}
@@ -286,7 +290,9 @@ private static void registerTable(SqlTree sqlTree, StreamExecutionEnvironment en
286290

287291
Table regTable = tableEnv.fromDataStream(adaptStream, fields);
288292
tableEnv.registerTable(tableInfo.getName(), regTable);
289-
LOG.info("registe table {} success.", tableInfo.getName());
293+
if(LOG.isInfoEnabled()){
294+
LOG.info("registe table {} success.", tableInfo.getName());
295+
}
290296
registerTableCache.put(tableInfo.getName(), regTable);
291297
classPathSet.add(PluginUtil.getRemoteJarFilePath(tableInfo.getType(), SourceTableInfo.SOURCE_SUFFIX, remoteSqlPluginPath, localSqlPluginPath));
292298
} else if (tableInfo instanceof TargetTableInfo) {

core/src/main/java/com/dtstack/flink/sql/side/SideSqlExec.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,9 @@ public void exec(String sql, Map<String, SideTableInfo> sideTableMap, StreamTabl
106106

107107
if(pollSqlNode.getKind() == INSERT){
108108
tableEnv.sqlUpdate(pollSqlNode.toString());
109-
LOG.info("exec sql: " + pollSqlNode.toString());
109+
if(LOG.isInfoEnabled()){
110+
LOG.info("exec sql: " + pollSqlNode.toString());
111+
}
110112
}else if(pollSqlNode.getKind() == AS){
111113
AliasInfo aliasInfo = parseASNode(pollSqlNode);
112114
Table table = tableEnv.sql(aliasInfo.getName());
@@ -536,7 +538,9 @@ public void registerTmpTable(CreateTmpTableParser.SqlParserResult result,
536538
AliasInfo aliasInfo = parseASNode(pollSqlNode);
537539
Table table = tableEnv.sql(aliasInfo.getName());
538540
tableEnv.registerTable(aliasInfo.getAlias(), table);
539-
LOG.info("Register Table {} by {}", aliasInfo.getAlias(), aliasInfo.getName());
541+
if(LOG.isInfoEnabled()){
542+
LOG.info("Register Table {} by {}", aliasInfo.getAlias(), aliasInfo.getName());
543+
}
540544
localTableCache.put(aliasInfo.getAlias(), table);
541545
} else if (pollSqlNode.getKind() == SELECT){
542546
Table table = tableEnv.sqlQuery(pollObj.toString());

kafka09/kafka09-source/src/main/java/com/dtstack/flink/sql/source/kafka/table/KafkaSourceParser.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,9 @@ static void dealNestField(Matcher matcher, TableInfo tableInfo) {
6363
tableInfo.addField(mappingField);
6464
tableInfo.addFieldClass(fieldClass);
6565
tableInfo.addFieldType(fieldType);
66-
LOG.info(physicalField + "--->" + mappingField + " Class: " + fieldClass.toString());
66+
if(LOG.isInfoEnabled()){
67+
LOG.info(physicalField + "--->" + mappingField + " Class: " + fieldClass.toString());
68+
}
6769
}
6870

6971
@Override

kafka10/kafka10-source/src/main/java/com/dtstack/flink/sql/source/kafka/table/KafkaSourceParser.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@
2424
import com.dtstack.flink.sql.table.TableInfo;
2525
import com.dtstack.flink.sql.util.ClassUtil;
2626
import com.dtstack.flink.sql.util.MathUtil;
27+
import org.slf4j.Logger;
28+
import org.slf4j.LoggerFactory;
2729

2830
import java.util.Map;
2931
import java.util.regex.Matcher;
@@ -38,6 +40,8 @@
3840

3941
public class KafkaSourceParser extends AbsSourceParser {
4042

43+
private static final Logger LOG = LoggerFactory.getLogger(KafkaSourceParser.class);
44+
4145
private static final String KAFKA_NEST_FIELD_KEY = "nestFieldKey";
4246

4347
private static Pattern kafkaNestFieldKeyPattern = Pattern.compile("(?i)((@*\\w+\\.)*\\w+)\\s+(\\w+)\\s+AS\\s+(\\w+)$");
@@ -63,6 +67,9 @@ static void dealNestField(Matcher matcher, TableInfo tableInfo) {
6367
tableInfo.addField(mappingField);
6468
tableInfo.addFieldClass(fieldClass);
6569
tableInfo.addFieldType(fieldType);
70+
if(LOG.isInfoEnabled()){
71+
LOG.info(physicalField + "--->" + mappingField + " Class: " + fieldClass.toString());
72+
}
6673
}
6774

6875
@Override

kafka11/kafka11-source/src/main/java/com/dtstack/flink/sql/source/kafka/table/KafkaSourceParser.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@
2424
import com.dtstack.flink.sql.table.TableInfo;
2525
import com.dtstack.flink.sql.util.ClassUtil;
2626
import com.dtstack.flink.sql.util.MathUtil;
27+
import org.slf4j.Logger;
28+
import org.slf4j.LoggerFactory;
2729

2830
import java.util.Map;
2931
import java.util.regex.Matcher;
@@ -38,6 +40,8 @@
3840

3941
public class KafkaSourceParser extends AbsSourceParser {
4042

43+
private static final Logger LOG = LoggerFactory.getLogger(KafkaSourceParser.class);
44+
4145
private static final String KAFKA_NEST_FIELD_KEY = "nestFieldKey";
4246

4347
private static Pattern kafkaNestFieldKeyPattern = Pattern.compile("(?i)((@*\\w+\\.)*\\w+)\\s+(\\w+)\\s+AS\\s+(\\w+)$");
@@ -63,6 +67,9 @@ static void dealNestField(Matcher matcher, TableInfo tableInfo) {
6367
tableInfo.addField(mappingField);
6468
tableInfo.addFieldClass(fieldClass);
6569
tableInfo.addFieldType(fieldType);
70+
if(LOG.isInfoEnabled()){
71+
LOG.info(physicalField + "--->" + mappingField + " Class: " + fieldClass.toString());
72+
}
6673
}
6774

6875
@Override

0 commit comments

Comments
 (0)