MPHEE: first pass at adding logging for applications

[martyphee]

MultiLang Daemon StdLogger call

Adding the ability for the clients to send logging messages through the normal stdout by having the KCL process new LogMessages

WIP right now

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

2022-06-17 13:31:49,176 [ShardRecordProcessor-0001] INFO  s.a.k.multilang.messages.LogMessage [NONE] - Processing record with key AN_ID_OF_A_POLICY 
2022-06-17 13:31:49,176 [multi-lang-daemon-0003] INFO  s.a.kinesis.multilang.LineReaderTask [NONE] - Starting: Reading next message from STDIN for shardId-000000000000 
2022-06-17 13:31:49,177 [ShardRecordProcessor-0001] ERROR s.a.k.multilang.messages.LogMessage [NONE] - Processing record with key AN_ID_OF_A_POLICY 

Python side maybe?

class KclLogger:
    class LogLevel(Enum):
        INFO = "info"
        DEBUG = "debug"
        ERROR = "error"
        WARN = "warn"

    def debug(self, message: str):
        self._write_action(self.LogLevel.DEBUG.value, message)

    def info(self, message: str):
        self._write_action(self.LogLevel.INFO.value, message)

    def warn(self, message: str):
        self._write_action(self.LogLevel.WARN.value, message)

    def error(self, message: str):
        self._write_action(self.LogLevel.ERROR.value, message)

    @staticmethod
    def _write_action(log_level: LogLevel, message: str):
        kclprocess.io_handler.write_action(KclLogger._create_message(log_level, message))

    @staticmethod
    def _create_message(log_level: LogLevel, message: str):
        return {
            "action": "log",
            "logLevel": log_level,
            "message": message
        }

[brendan-p-lynch]

Please update this draft if you still would like this to be added

[brendan-p-lynch]

We should not be adding wildcards for imports. This should explictly state all the things we need to import

[brendan-p-lynch]

This looks to be returning only a boolean, not a Function<String, Boolean>. Also everything returns true. What is the benefit of returning a value

[martyphee]

It's returning a function

return (m) -> {
                     log.debug(m);
                     return true;
                 };

Not sure a better return type since Function requires one.

[martyphee]

I haven't looked at this for a couple years now. If you find it useful I can fix things up but I'm not using kinesis anymore.

[martyphee]

It's returning a function

return (m) -> {
                     log.debug(m);
                     return true;
                 };

Not sure a better return type since Function requires one.