Skip to content

[R4R] change order creatation time and lastupdate time to nanosecond#545

Merged
cryptom-dev merged 2 commits intodevelopfrom
ordertime_dev
Apr 8, 2019
Merged

[R4R] change order creatation time and lastupdate time to nanosecond#545
cryptom-dev merged 2 commits intodevelopfrom
ordertime_dev

Conversation

@cryptom-dev
Copy link
Contributor

@cryptom-dev cryptom-dev commented Apr 8, 2019

Description

change order creatation time and lastupdate time to nanosecond

Rationale

query service process

Example

N/A

Changes

Preflight checks

  • build passed (make build)
  • tests passed (make test)
  • integration tests passed (make integration_test)
  • manual transaction test passed (cli invoke)

Already reviewed by

...

Related issues

... reference related issue #'s here ...

@cryptom-dev cryptom-dev self-assigned this Apr 8, 2019
@cryptom-dev cryptom-dev changed the title change order creatation time and lastupdate time to nanosecond [R4R] change order creatation time and lastupdate time to nanosecond Apr 8, 2019
app/app.go Outdated
func (app *BinanceChain) setUpgradeConfig() {
upgrade.Mgr.AddUpgradeHeight(upgrade.FixOrderSeqInPriceLevelName, app.upgradeConfig.FixOrderSeqInPriceLevelHeight)
upgrade.Mgr.AddUpgradeHeight(upgrade.FixDropFilledOrderSeqName, app.upgradeConfig.FixDropFilledOrderSeqHeight)
upgrade.Mgr.AddUpgradeHeight(upgrade.FixOrderTimestampName, app.upgradeConfig.FixOrderTimestampHeight)
Copy link
Contributor

@rickyyangz rickyyangz Apr 8, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for develop branch, I do not think we need these hard fork logic any more ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for develop branch, I do not think we need these hard fork logic any more ?

fixed

@cryptom-dev cryptom-dev merged commit 9c6c66b into develop Apr 8, 2019
@unclezoro unclezoro deleted the ordertime_dev branch May 10, 2022 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants