spdf/elektron
Archived
1
0
Fork 0
Code Issues 4 Pull requests 1 Projects Releases Packages Wiki Activity

WIP : Elektron Logging library #16

Merged
balandi1 merged 50 commits from master into master 2019-12-10 01:15:34 +00:00
Conversation 113 Commits 50 Files changed 61 +32 -32
22 changed files with 32 additions and 32 deletions
Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 3d201bf437 - Show all commits

4
def/taskUtils.go
View file

@ -26,8 +26,8 @@ import (
pradykaushik commented 2019-11-21 00:31:57 +00:00 (Migrated from github.com)
Copy link

I would alias this to elekLog as you seem to be using that prefix when aliasing imports of sub-packages.

I would alias this to _elekLog_ as you seem to be using that prefix when aliasing imports of sub-packages.
balandi1 commented 2019-11-21 19:30:07 +00:00 (Migrated from github.com)
Copy link

Okay. Done

Okay. Done
pradykaushik commented 2019-11-21 00:31:57 +00:00 (Migrated from github.com)
Copy link

I would alias this to elekLog as you seem to be using that prefix when aliasing imports of sub-packages.

I would alias this to _elekLog_ as you seem to be using that prefix when aliasing imports of sub-packages.
balandi1 commented 2019-11-21 19:30:07 +00:00 (Migrated from github.com)
Copy link

Okay. Done

Okay. Done
"github.com/mash/gokmeans"
"github.com/montanaflynn/stats"
log "github.com/sirupsen/logrus"
elekLog "github.com/spdfg/elektron/elektronLogging"
pradykaushik commented 2019-11-21 00:31:57 +00:00 (Migrated from github.com)
Copy link

I would alias this to elekLog as you seem to be using that prefix when aliasing imports of sub-packages.

I would alias this to _elekLog_ as you seem to be using that prefix when aliasing imports of sub-packages.
balandi1 commented 2019-11-21 19:30:07 +00:00 (Migrated from github.com)
Copy link

Okay. Done

Okay. Done
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
pradykaushik commented 2019-11-21 00:31:57 +00:00 (Migrated from github.com)
Copy link

I would alias this to elekLog as you seem to be using that prefix when aliasing imports of sub-packages.

I would alias this to _elekLog_ as you seem to be using that prefix when aliasing imports of sub-packages.
balandi1 commented 2019-11-21 19:30:07 +00:00 (Migrated from github.com)
Copy link

Okay. Done

Okay. Done
elekLog "github.com/spdfg/elektron/logging"
pradykaushik commented 2019-11-21 00:31:57 +00:00 (Migrated from github.com)
Copy link

I would alias this to elekLog as you seem to be using that prefix when aliasing imports of sub-packages.

I would alias this to _elekLog_ as you seem to be using that prefix when aliasing imports of sub-packages.
balandi1 commented 2019-11-21 19:30:07 +00:00 (Migrated from github.com)
Copy link

Okay. Done

Okay. Done
elekLogTypes "github.com/spdfg/elektron/logging/types"
pradykaushik commented 2019-11-21 00:31:57 +00:00 (Migrated from github.com)
Copy link

I would alias this to elekLog as you seem to be using that prefix when aliasing imports of sub-packages.

I would alias this to _elekLog_ as you seem to be using that prefix when aliasing imports of sub-packages.
balandi1 commented 2019-11-21 19:30:07 +00:00 (Migrated from github.com)
Copy link

Okay. Done

Okay. Done
)
// Information about a cluster of tasks.

pradykaushik commented 2019-11-21 00:31:57 +00:00 (Migrated from github.com)
Copy link

I would alias this to elekLog as you seem to be using that prefix when aliasing imports of sub-packages.

I would alias this to _elekLog_ as you seem to be using that prefix when aliasing imports of sub-packages.
balandi1 commented 2019-11-21 19:30:07 +00:00 (Migrated from github.com)
Copy link

Okay. Done

Okay. Done
pradykaushik commented 2019-11-21 00:31:57 +00:00 (Migrated from github.com)
Copy link

I would alias this to elekLog as you seem to be using that prefix when aliasing imports of sub-packages.

I would alias this to _elekLog_ as you seem to be using that prefix when aliasing imports of sub-packages.
balandi1 commented 2019-11-21 19:30:07 +00:00 (Migrated from github.com)
Copy link

Okay. Done

Okay. Done

2
elektronLogging/ElektronFormatter.go → logging/ElektronFormatter.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
"bytes"

2
elektronLogging/clsfnTaskDistOverheadLogger.go → logging/clsfnTaskDistOverheadLogger.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
"os"

2
elektronLogging/consoleLogger.go → logging/consoleLogger.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
"fmt"

2
elektronLogging/createLogDir.go → logging/createLogDir.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
"os"

4
elektronLogging/logger.go → logging/logger.go
View file

@ -1,4 +1,4 @@
pradykaushik commented 2019-12-06 22:03:56 +00:00 (Migrated from github.com)
Copy link

isEnabled() does not have to be part of the interface.

`isEnabled()` does not have to be part of the interface.
pradykaushik commented 2019-12-06 22:04:02 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
pradykaushik commented 2019-12-06 22:04:11 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
balandi1 commented 2019-12-09 15:41:29 +00:00 (Migrated from github.com)
Copy link

Yes. Removed it from there

Yes. Removed it from there
balandi1 commented 2019-12-09 15:42:13 +00:00 (Migrated from github.com)
Copy link

Done

Done
balandi1 commented 2019-12-09 15:42:23 +00:00 (Migrated from github.com)
Copy link

Done

Done
pradykaushik commented 2019-12-06 22:03:56 +00:00 (Migrated from github.com)
Copy link

isEnabled() does not have to be part of the interface.

`isEnabled()` does not have to be part of the interface.
pradykaushik commented 2019-12-06 22:04:02 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
pradykaushik commented 2019-12-06 22:04:11 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
balandi1 commented 2019-12-09 15:41:29 +00:00 (Migrated from github.com)
Copy link

Yes. Removed it from there

Yes. Removed it from there
balandi1 commented 2019-12-09 15:42:13 +00:00 (Migrated from github.com)
Copy link

Done

Done
balandi1 commented 2019-12-09 15:42:23 +00:00 (Migrated from github.com)
Copy link

Done

Done
package elektronLogging
pradykaushik commented 2019-12-06 22:03:56 +00:00 (Migrated from github.com)
Copy link

isEnabled() does not have to be part of the interface.

`isEnabled()` does not have to be part of the interface.
pradykaushik commented 2019-12-06 22:04:02 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
pradykaushik commented 2019-12-06 22:04:11 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
balandi1 commented 2019-12-09 15:41:29 +00:00 (Migrated from github.com)
Copy link

Yes. Removed it from there

Yes. Removed it from there
balandi1 commented 2019-12-09 15:42:13 +00:00 (Migrated from github.com)
Copy link

Done

Done
balandi1 commented 2019-12-09 15:42:23 +00:00 (Migrated from github.com)
Copy link

Done

Done
package logging
pradykaushik commented 2019-12-06 22:03:56 +00:00 (Migrated from github.com)
Copy link

isEnabled() does not have to be part of the interface.

`isEnabled()` does not have to be part of the interface.
pradykaushik commented 2019-12-06 22:04:02 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
pradykaushik commented 2019-12-06 22:04:11 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
balandi1 commented 2019-12-09 15:41:29 +00:00 (Migrated from github.com)
Copy link

Yes. Removed it from there

Yes. Removed it from there
balandi1 commented 2019-12-09 15:42:13 +00:00 (Migrated from github.com)
Copy link

Done

Done
balandi1 commented 2019-12-09 15:42:23 +00:00 (Migrated from github.com)
Copy link

Done

Done
import (
"os"
@ -6,7 +6,7 @@ import (
pradykaushik commented 2019-12-06 22:03:56 +00:00 (Migrated from github.com)
Copy link

isEnabled() does not have to be part of the interface.

`isEnabled()` does not have to be part of the interface.
pradykaushik commented 2019-12-06 22:04:02 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
pradykaushik commented 2019-12-06 22:04:11 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
balandi1 commented 2019-12-09 15:41:29 +00:00 (Migrated from github.com)
Copy link

Yes. Removed it from there

Yes. Removed it from there
balandi1 commented 2019-12-09 15:42:13 +00:00 (Migrated from github.com)
Copy link

Done

Done
balandi1 commented 2019-12-09 15:42:23 +00:00 (Migrated from github.com)
Copy link

Done

Done
pradykaushik commented 2019-12-06 22:03:56 +00:00 (Migrated from github.com)
Copy link

isEnabled() does not have to be part of the interface.

`isEnabled()` does not have to be part of the interface.
pradykaushik commented 2019-12-06 22:04:02 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
pradykaushik commented 2019-12-06 22:04:11 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
balandi1 commented 2019-12-09 15:41:29 +00:00 (Migrated from github.com)
Copy link

Yes. Removed it from there

Yes. Removed it from there
balandi1 commented 2019-12-09 15:42:13 +00:00 (Migrated from github.com)
Copy link

Done

Done
balandi1 commented 2019-12-09 15:42:23 +00:00 (Migrated from github.com)
Copy link

Done

Done
"time"
log "github.com/sirupsen/logrus"
. "github.com/spdfg/elektron/elektronLogging/types"
pradykaushik commented 2019-12-06 22:03:56 +00:00 (Migrated from github.com)
Copy link

isEnabled() does not have to be part of the interface.

`isEnabled()` does not have to be part of the interface.
pradykaushik commented 2019-12-06 22:04:02 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
pradykaushik commented 2019-12-06 22:04:11 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
balandi1 commented 2019-12-09 15:41:29 +00:00 (Migrated from github.com)
Copy link

Yes. Removed it from there

Yes. Removed it from there
balandi1 commented 2019-12-09 15:42:13 +00:00 (Migrated from github.com)
Copy link

Done

Done
balandi1 commented 2019-12-09 15:42:23 +00:00 (Migrated from github.com)
Copy link

Done

Done
. "github.com/spdfg/elektron/logging/types"
pradykaushik commented 2019-12-06 22:03:56 +00:00 (Migrated from github.com)
Copy link

isEnabled() does not have to be part of the interface.

`isEnabled()` does not have to be part of the interface.
pradykaushik commented 2019-12-06 22:04:02 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
pradykaushik commented 2019-12-06 22:04:11 +00:00 (Migrated from github.com)
Copy link

ditto

ditto
balandi1 commented 2019-12-09 15:41:29 +00:00 (Migrated from github.com)
Copy link

Yes. Removed it from there

Yes. Removed it from there
balandi1 commented 2019-12-09 15:42:13 +00:00 (Migrated from github.com)
Copy link

Done

Done
balandi1 commented 2019-12-09 15:42:23 +00:00 (Migrated from github.com)
Copy link

Done

Done
)
var config LoggerConfig

2
elektronLogging/loggerChain.go → logging/loggerChain.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
log "github.com/sirupsen/logrus"

2
elektronLogging/loggerConfig.go → logging/loggerConfig.go
View file

@ -1,4 +1,4 @@
pradykaushik commented 2019-12-06 22:05:07 +00:00 (Migrated from github.com)
Copy link

does this need to be exported outside the package?

does this need to be exported outside the package?
balandi1 commented 2019-12-09 16:03:06 +00:00 (Migrated from github.com)
Copy link

No, not needed. Changed it to getConfig .

No, not needed. Changed it to `getConfig` .
pradykaushik commented 2019-12-06 22:05:07 +00:00 (Migrated from github.com)
Copy link

does this need to be exported outside the package?

does this need to be exported outside the package?
balandi1 commented 2019-12-09 16:03:06 +00:00 (Migrated from github.com)
Copy link

No, not needed. Changed it to getConfig .

No, not needed. Changed it to `getConfig` .
package elektronLogging
pradykaushik commented 2019-12-06 22:05:07 +00:00 (Migrated from github.com)
Copy link

does this need to be exported outside the package?

does this need to be exported outside the package?
balandi1 commented 2019-12-09 16:03:06 +00:00 (Migrated from github.com)
Copy link

No, not needed. Changed it to getConfig .

No, not needed. Changed it to `getConfig` .
package logging
pradykaushik commented 2019-12-06 22:05:07 +00:00 (Migrated from github.com)
Copy link

does this need to be exported outside the package?

does this need to be exported outside the package?
balandi1 commented 2019-12-09 16:03:06 +00:00 (Migrated from github.com)
Copy link

No, not needed. Changed it to getConfig .

No, not needed. Changed it to `getConfig` .
import (
log "github.com/sirupsen/logrus"

2
elektronLogging/pcpLogger.go → logging/pcpLogger.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
"os"

2
elektronLogging/schedPolicySwitchLogger.go → logging/schedPolicySwitchLogger.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
"os"

2
elektronLogging/schedTraceLogger.go → logging/schedTraceLogger.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
"os"

2
elektronLogging/schedWindowLogger.go → logging/schedWindowLogger.go
View file

@ -1,4 +1,4 @@
package elektronLogging
package logging
import (
"os"

0
elektronLogging/types/types.go → logging/types/types.go
View file

4
pcp/pcp.go
View file

@ -25,8 +25,8 @@ import (
"time"
log "github.com/sirupsen/logrus"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
)
func Start(quit chan struct{}, logging *bool, pcpConfigFile string) {

4
powerCap/extrema.go
View file

@ -30,8 +30,8 @@ import (
"time"
log "github.com/sirupsen/logrus"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
"github.com/spdfg/elektron/pcp"
"github.com/spdfg/elektron/rapl"
)

4
powerCap/progressiveExtrema.go
View file

@ -32,8 +32,8 @@ import (
log "github.com/sirupsen/logrus"
"github.com/spdfg/elektron/constants"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
"github.com/spdfg/elektron/pcp"
"github.com/spdfg/elektron/rapl"
"github.com/spdfg/elektron/utilities"

4
scheduler.go
View file

@ -31,8 +31,8 @@ import (
sched "github.com/mesos/mesos-go/api/v0/scheduler"
log "github.com/sirupsen/logrus"
"github.com/spdfg/elektron/def"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
"github.com/spdfg/elektron/pcp"
"github.com/spdfg/elektron/powerCap"
"github.com/spdfg/elektron/schedulers"

4
schedulers/base.go
View file

@ -30,8 +30,8 @@ import (
sched "github.com/mesos/mesos-go/api/v0/scheduler"
log "github.com/sirupsen/logrus"
"github.com/spdfg/elektron/def"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
"github.com/spdfg/elektron/utilities"
"github.com/spdfg/elektron/utilities/schedUtils"
)

4
schedulers/helpers.go
View file

@ -27,8 +27,8 @@ import (
log "github.com/sirupsen/logrus"
"github.com/spdfg/elektron/constants"
"github.com/spdfg/elektron/def"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
"github.com/spdfg/elektron/utilities"
"github.com/spdfg/elektron/utilities/mesosUtils"
)

4
schedulers/schedPolicy.go
View file

@ -26,8 +26,8 @@ import (
sched "github.com/mesos/mesos-go/api/v0/scheduler"
log "github.com/sirupsen/logrus"
"github.com/spdfg/elektron/def"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
)
type SchedPolicyContext interface {

4
utilities/offerUtils/offerUtils.go
View file

@ -24,8 +24,8 @@ import (
mesos "github.com/mesos/mesos-go/api/v0/mesosproto"
log "github.com/sirupsen/logrus"
"github.com/spdfg/elektron/constants"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
)
func OfferAgg(offer *mesos.Offer) (float64, float64, float64) {

4
utilities/schedUtils/schedUtils.go
View file

@ -21,8 +21,8 @@ package schedUtils
import (
log "github.com/sirupsen/logrus"
"github.com/spdfg/elektron/def"
elekLog "github.com/spdfg/elektron/elektronLogging"
elekLogTypes "github.com/spdfg/elektron/elektronLogging/types"
elekLog "github.com/spdfg/elektron/logging"
elekLogTypes "github.com/spdfg/elektron/logging/types"
"github.com/spdfg/elektron/utilities"
)