Suggestion: use_blob
default as True when user is admin
#894
Closed
MariusWirtz
started this conversation in
Ideas
Replies: 2 comments 2 replies
-
I have some reservations about this change. Moving towards blob based interaction makes physical storage play a role in how the application scales. that seems a bit dangerous especially for applications with high concurrency.
…Sent from my mobile device
On Apr 8, 2023 6:35 PM, Marius Wirtz ***@***.***> wrote:
Given the performance impact of using blobs with TM1py, it seems reasonable to use them by default, for admin users, in the following functions:
* write
* execute_mdx_csv
* execute_view_csv
* execute_mdx_dataframe
* execute_view_dataframe
In theory, this could be considered a breaking change in the upcoming 1.11 release.
However, given the revolutionary performance impacts of up to 10x in write and 2x in read, it seems a reasonable decision IMO.
Any other opinions on this one?
—
Reply to this email directly, view it on GitHub<#894>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AEK7GZWIF5U55LJE6FGHGSLXAHRZ3ANCNFSM6AAAAAAWXWXH4A>.
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
1 reply
-
I think it should always be false unless specified. I think builders that use TM1py have the assumption that data read/write is more direct and relies on TM1s API endpoint built for that reason. While this pattern is faster there could be considerable tradeoffs at scale. Plus I don’t think IBM is designing their future roadmap with this “hack” (hack in the positive sense) in mind. My suggestion is that we keep it as false, then raise this performance gap with IBM and ask them to close it.
|
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Given the performance impact of using blobs with TM1py, it seems reasonable to use them by default, for admin users, in the following functions:
write
execute_mdx_csv
execute_view_csv
execute_mdx_dataframe
execute_view_dataframe
In theory, this could be considered a breaking change in the upcoming 1.11 release.
However, given the revolutionary performance impacts of up to 10x in write and 2x in read, it seems a reasonable decision IMO.
Any other opinions on this one?
Beta Was this translation helpful? Give feedback.
All reactions