Skip to content

InfluxDBClient __exit__ function -> pylint warning #520

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

Closed
intelroman opened this issue Oct 20, 2022 · 2 comments · Fixed by #526
Closed

InfluxDBClient __exit__ function -> pylint warning #520

intelroman opened this issue Oct 20, 2022 · 2 comments · Fixed by #526
Labels
bug Something isn't working
Milestone

Comments

@intelroman
Copy link

intelroman commented Oct 20, 2022

Specifications

  • Client Version: Latest
  • InfluxDB Version: 2.4.0
  • Platform: Linux

Code sample to reproduce problem

    with InfluxDBClient.from_config_file("conf/influx.ini") as _client:
        with _client.write_api(write_options=WriteOptions(batch_size=5000,
                                                            flush_interval=10_000,
                                                            jitter_interval=2_000,
                                                            retry_interval=5_000,
                                                            max_retries=5,
                                                            max_retry_delay=30_000,
                                                            exponential_base=2,
                                                            )) as _write_client:

Expected behavior

Expect pylint to not show me any problem in the code.

   _base.py file 
don't have __enter__/ __exit__

Actual behavior

Pylint will complain
Context manager '_BaseClient' doesn't implement enter and exit.pylint(not-context-manager)

Additional info

@intelroman intelroman added the bug Something isn't working label Oct 20, 2022
@intelroman
Copy link
Author

@bednar
Copy link
Contributor

bednar commented Oct 20, 2022

Hi @intelroman,

thanks for using our client.

Is this something you might be willing to help with?

Regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants