The Wayback Machine - https://web.archive.org/web/20200912024751/https://github.com/CovenantSQL/CovenantSQL/pull/357
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Miner support MQTT protocol #357

Open
wants to merge 16 commits into
base: develop
from
Open

[WIP] Miner support MQTT protocol #357

wants to merge 16 commits into from

Conversation

@laodouya
Copy link
Contributor

laodouya commented Jun 21, 2019

No description provided.

laodouya added 14 commits Jun 14, 2019
…eader of a database.
@laodouya laodouya changed the title [WIP] Feature/miner mqtt [WIP] Miner support MQTT protocol Jun 21, 2019
@@ -106,6 +106,14 @@ type DNSSeed struct {
BPCount int `yaml:"BPCount"`
}

type MQTTBrokerInfo struct {

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
exported type MQTTBrokerInfo should have comment or be unexported

@@ -1020,6 +1020,10 @@ func (c *Chain) AddResponse(resp *types.SignedResponseHeader) (err error) {
return c.ai.addResponse(c.rt.getHeightFromTime(resp.GetRequestTimestamp()), resp)
}

func (c *Chain) GetPeerLeaderID() proto.NodeID {

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
exported method Chain.GetPeerLeaderID should have comment or be unexported

"github.com/gogf/gf/g/container/gqueue"
)

type MQTTAPI string

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
exported type MQTTAPI should have comment or be unexported

type MQTTAPI string

const (
//Publish API

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
comment on exported const MQTTNewest should be of the form "MQTTNewest ..."

const (
//Publish API
MQTTNewest MQTTAPI = "newest"
DSNList MQTTAPI = "dsnlist"

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
exported const DSNList should have comment (or a comment on this block) or be unexported

type SubscribeEvent struct {
ClientID proto.NodeID
DatabaseID proto.DatabaseID
ApiName MQTTAPI

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
struct field ApiName should be APIName

Payload BrokerPayload
}

type MQTTClient struct {

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
exported type MQTTClient should have comment or be unexported

return dests
}

func NewMQTTClient(config *conf.MQTTBrokerInfo, dbms *DBMS) (c *MQTTClient) {

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
exported function NewMQTTClient should have comment or be unexported

}
}

func (c *MQTTClient) PublishDSN(apiName MQTTAPI, databaseID proto.DatabaseID, payload BrokerPayload, requestClient proto.NodeID) error {

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
exported method MQTTClient.PublishDSN should have comment or be unexported

return nil
}

func (c *MQTTClient) Close() {

This comment has been minimized.

@covenantdb

covenantdb Jun 21, 2019

[golint] reported by reviewdog 🐶
exported method MQTTClient.Close should have comment or be unexported

laodouya added 2 commits Jun 21, 2019
@laodouya laodouya force-pushed the feature/miner_mqtt branch from 87e35e5 to ac8847d Jun 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.