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 configure-measure-group-properties.md #1133

Closed
wants to merge 1 commit into
base: live
from

Conversation

Projects
None yet
3 participants
@vstrien
Copy link

vstrien commented Oct 17, 2018

Description for DataAggregation property was outdated - for current SSAS versions, DataAggregation is a supported setting (according to the 20768 training). The sentence I added is a description from the 20768 trainer handbook.

Update configure-measure-group-properties.md
Description for DataAggregation property was outdated - for current SSAS versions, DataAggregation is a supported setting. The sentence I added is a description from the 20768 training.
@PRMerger14

This comment has been minimized.

Copy link
Contributor

PRMerger14 commented Oct 17, 2018

@vstrien : Thanks for your contribution! The author, @Minewiskan, has been notified to review your proposed change.

@Minewiskan

This comment has been minimized.

Copy link
Contributor

Minewiskan commented Oct 24, 2018

@vstrien Thanks for contributing. However, this property is actually not used. The engine ignores it. What description from the 20768 training are you referring to? Could you provide me with more detail about this training and who provided it?
Thanks

@vstrien

This comment has been minimized.

Copy link

vstrien commented Oct 25, 2018

@vstrien Thanks for contributing. However, this property is actually not used. The engine ignores it. What description from the 20768 training are you referring to? Could you provide me with more detail about this training and who provided it?
Thanks

Sure, it's in the Microsoft-provided Trainer Handbook for the 20768 course (as an MCT I have access to this handbook). It's on page 111, as displayed in this screenshot:
screenshot 2018-10-25 at 08 35 15

@Minewiskan

This comment has been minimized.

Copy link
Contributor

Minewiskan commented Oct 25, 2018

@vstrien Thanks for letting me know. You're the second person in two weeks who suggested the same change. Now I know where it's coming from. It will take a while to look into this. Let's leave this issue open while I do some digging. Thanks for your patience.

@Minewiskan

This comment has been minimized.

Copy link
Contributor

Minewiskan commented Oct 25, 2018

@vstrien I'm not able to get into the actual course, but I found it described here, https://www.microsoft.com/en-us/learning/course.aspx?cid=20768. This course is terribly outdated. I remember being consulted for help with creating the DAX module, probably seven or eight years ago. I strongly recommend not suggesting or presenting this course. You might want to bring it up with your contacts in MS Learning. I'll do what I can internally to get this course removed. I cannot merge your suggested change as stated earlier. If you could close this PR, that would be great. Again, thanks for the information. I'm sorry I cannot provide a better outcome.

@vstrien

This comment has been minimized.

Copy link

vstrien commented Oct 25, 2018

As requested by @Minewiskan, proposed update was incorrect.

@vstrien vstrien closed this Oct 25, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment