search

StarRocks / demo

Apache License 2.0
79 stars 55 forks source link

Fix bracket position issue #62 #63

Open NewJerseyStyle opened 2 months ago

NewJerseyStyle commented 2 months ago

Fixing issue #62 Change MiscDemo/connect/python/connector.py Line 91 print("{}\t{}\t{}").format(siteid, citycode, pv) to print("{}\t{}\t{}".format(siteid, citycode, pv))

The code was from Beijing Dingshi Zongheng Technology Co., Ltd. It was trying to print some information in a given format. However this intention was not present correctly according to Python syntax. To make it better represent the intention and allow the Python interpreter to execute it correctly, the closing bracket of function call print has been moved to the end of the line.

It is not a original creation of mine, somehow it is done through me.

CLAassistant commented 2 months ago

CLA assistant check
All committers have signed the CLA.

NewJerseyStyle commented 2 months ago

I am not sure if my contributions in this pull request is my original creation. But it is an issue that should be fixed. Thanks

DanRoscigno commented 1 month ago

Thanks @NewJerseyStyle , can you please sign the CLA?

NewJerseyStyle commented 1 month ago

@DanRoscigno Thanks. I am trying to identify the complete details of the source and of any license or other restriction related to function and bracket in Python syntax. I will agree CLA once the document is ready.