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

Update R/Python/core/tikv-client modules to RC1 #263

Merged
merged 5 commits into from Mar 12, 2018
Merged

Conversation

Novemser
Copy link
Contributor

@Novemser Novemser commented Mar 11, 2018

  1. Fix statistics parameter for Python, R module.
  2. Update TiSpark version to 1.0-RC1

@sre-bot
Copy link
Contributor

sre-bot commented Mar 11, 2018

Hi contributor, thanks for your PR.

This patch needs to be approved by someone of admins. They should reply with "/ok-to-test" to accept this PR for running test automatically.

@Novemser Novemser changed the title Fix Python map database API Update R/Python/parent module to RC1 Mar 11, 2018
*
* @param tiSession TiSession used for selecting statistics information from TiKV
*/
* Manager class for maintaining table statistics information cache.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Accidentally introduced by scala formatter.

@Novemser Novemser self-assigned this Mar 11, 2018
@Novemser Novemser changed the title Update R/Python/parent module to RC1 Update R/Python/core/tikv-client modules to RC1 Mar 11, 2018
@Novemser
Copy link
Contributor Author

/run-all-tests

@Novemser
Copy link
Contributor Author

PTAL @ilovesoup @birdstorm

python/setup.py Outdated
@@ -2,7 +2,7 @@
setup(
name = 'pytispark',
packages = ['pytispark'],
version = '0.1.3',
version = '1.0.0-RC1',
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be 1.0.0-RC1 or 1.0-RC1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I saw both on Pypi

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The former could do more fined grained version control.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But we generally do not modify this module...

Copy link
Contributor

@ilovesoup ilovesoup left a comment

Choose a reason for hiding this comment

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

LGTM

@ilovesoup ilovesoup merged commit 0ef80af into master Mar 12, 2018
@ilovesoup ilovesoup deleted the fix_Python_mapping branch March 12, 2018 14:35
wfxxh pushed a commit to wanfangdata/tispark that referenced this pull request Jun 30, 2023
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.

None yet

4 participants