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

[FSTORE-33] Improve HSFS parameter description #806

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

moritzmeister
Copy link
Contributor

This PR adds/fixes/changes...
... adds a clearer description for the host parameter of the connection object.

JIRA Issue: FSTORE-33

Priority for Review: -

Related PRs: -

How Has This Been Tested?

  • Unit Tests
  • Integration Tests
  • Manual Tests on VM

Checklist For The Assigned Reviewer:

- [ ] Checked if merge conflicts with master exist
- [ ] Checked if stylechecks for Java and Python pass
- [ ] Checked if all docstrings were added and/or updated appropriately
- [ ] Ran spellcheck on docstring
- [ ] Checked if guides & concepts need to be updated
- [ ] Checked if naming conventions for parameters and variables were followed
- [ ] Checked if private methods are properly declared and used
- [ ] Checked if hard-to-understand areas of code are commented
- [ ] Checked if tests are effective
- [ ] Built and deployed changes on dev VM and tested manually
- [x] (Checked if all type annotations were added and/or updated appropriately)

Copy link
Contributor

@bubriks bubriks left a comment

Choose a reason for hiding this comment

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

Looks good

@SirOibaf SirOibaf merged commit c130236 into logicalclocks:master Oct 3, 2022
davitbzh added a commit to davitbzh/feature-store-api that referenced this pull request Oct 20, 2022
* [FSTORE-11] Add mkdocs macros dependency for docs (logicalclocks#800)

* [HOPSWORKS-3296] Remove update of storage connector (logicalclocks#801)

* [FSTORE-329] CommitDTO type field is not returned anymore after Payar… (logicalclocks#805)

* [FSTORE-8] add query default argument in snowflake (logicalclocks#790)

* [FSTORE-8][append] Add unit test (logicalclocks#808)

* [FSTORE-33] Improve HSFS parameter description (logicalclocks#806)

* [FSTORE-331] BigQuery connector doesn't work if the application is ex… (logicalclocks#809)

* [FSTORE-331] BigQuery connector doesn't work if the application is executed on multiple nodes

* Change the BigQuery configuration so that it uses the credential field
  (base64 encoding of the content of the credential file)
  instead of providing the path in the credentialFile property.

* For Java, updated the tests to use JUnit5 to allow for the @tmpdir
  annotation

* Check if options is null before checking if it's empty

* [FSTORE-8][append] Java fix for snowflake connector (logicalclocks#810)

* [FSTORE-345] Update documentation to reflect supported methods in hsfs engines (logicalclocks#814)

* [HOPSWORKS-3345] Improve event time error message (logicalclocks#785)

* improve error message

* [FSTORE-348] Refactor Tutorials documentation page (logicalclocks#817)

* fix style (logicalclocks#820)

* [FSTORE-358] Add icon to external links in documentation navigation (logicalclocks#822)

* [FSTORE-311] Improved timestamp with timezone handling (logicalclocks#821)

* add timezone normalization to python engine

* set spark session timezone to utc

* stylecheck

* set spark timezone in java client

* style

* remove spark proerty validation because its internally set as session property

* [FSTORE-342] Add java tests to github pipeline (logicalclocks#819)

* [FSTORE-342] Add java tests to github pipeline

* Fix Java build

* Add m2 caching

* Remove failure ignore flag

* [FSTORE-346] Handle as_of in feature view (logicalclocks#816)

* handle as_of in feature view

* fix style

* add unit test

* fix style

* fix tests

* fix style

* fix style

* refactor is_time_travel

* address comments

* fix style

* [FSTORE-331][Append] Fix Storage Connector Tests on windows (logicalclocks#827)

* [FSTORE-383] Feature View method documentation (logicalclocks#831)

* [FSTORE-311][Append] Timezone-aware Timestamp conversion for PySpark client (logicalclocks#826)

* fixed timezones for pyspark clients in different timezone

* spark session timezone to utc

* [FSTORE-359] Rename Transformation Function Output Types and Fix Timezone-related issues (logicalclocks#829)

* remove unused import

* spark types for tf

* spark types for tf

* tmp

* merge rebase with upstream/master

* [HOPSWORKS-3342][Append] unit tests (#9)

* added unit tests for transformation functions

* fixed stylecheck

* fixed a small thing in docs

* infer_python_type

* fix tests

* fix tests

* fix checks

* removed legacy types cast from test

* add convert_column method, fix style

* updated builtin transformation functions

* style

(cherry picked from commit 6ad26c4cb72cfd286dbb3c7049cd72f0aed3ab78)

* made transformation functions timezone-safe

(cherry picked from commit 176c2da134517648d568f69784b1e13c56c580f1)

* style

* removed deprecated np types

* tests tf function returning none

* style

* fixed tests

* style

Co-authored-by: davitbzh <davit.bzhalava@gmail.com>

* [HOPSWORKS-3326][Append] Timezone-independent conversion of Hudi Timestamps and Timezone-Sensitive GH-Actions (logicalclocks#828)

* made conversion from utc explicit

* windows and timezone action

* rename action

* change runner to windows

* removed python variants from tz-local actions

* added java tz local action

* small rename

* added tests

* remove unused variable

* set correct timezone in java tests

* [FSTORE-332] implement method for get_or_create_feature_view (logicalclocks#813)

* get_or_create_feature_view

* uppercased Y

* updated comments

* format

* added some basic tests

Co-authored-by: Moritz Meister <8422705+moritzmeister@users.noreply.github.com>
Co-authored-by: Fabio Buso <buso.fabio@gmail.com>
Co-authored-by: Dhananjay Mukhedkar <55157590+DhananjayMukhedkar@users.noreply.github.com>
Co-authored-by: kennethmhc <kennethmhc@users.noreply.github.com>
Co-authored-by: davitbzh <davit.bzhalava@gmail.com>
Co-authored-by: davitbzh <44586065+davitbzh@users.noreply.github.com>
kennethmhc pushed a commit to kennethmhc/feature-store-api that referenced this pull request Nov 16, 2022
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.

3 participants